Skip to content

Conversation

ch0rizo
Copy link
Contributor

@ch0rizo ch0rizo commented Aug 28, 2025

Description

Added a pin for pinned job listings

Result

If you've made visual changes, please check the boxes below and include images showing the changes. Descriptions are appreciated.

  • Changes look good on both light and dark theme.
  • Changes look good with different viewports (mobile, tablet, etc.).
  • Changes look good with slower Internet connections.

Caution

Make sure your images do not contain any real user information.

Screenshot 2025-08-29 at 13 48 23

Testing

  • I have thoroughly tested my changes.

Please describe what and how the changes have been tested, and provide instructions to reproduce if necessary.


Resolves ABA-1488

Copy link

linear bot commented Aug 28, 2025

@github-actions github-actions bot added the review-needed Pull requests that need review label Aug 28, 2025
Copy link
Contributor

@Viljen789 Viljen789 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing change! Great work!

Copy link
Contributor

@itsisak itsisak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!

Just a thought, but maybe a Tooltip explaining that "this ad is pinned to the top" could be a nice addition to avoid confucion. Maybe users would think its a button for them to pin ads they like?

Copy link
Contributor

@falbru falbru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

Copy link
Contributor

@viljeno viljeno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review-needed Pull requests that need review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants