Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve the useful links on the public frontpage #3793

Merged

Conversation

ivarnakken
Copy link
Member

@ivarnakken ivarnakken commented Apr 16, 2023

Description

Introduce a ghost button style

Sort of a mixture of dark and danger.


Improve the useful links on the public frontpage

They are all given images, which helps to seperate them. The description
is now also easier to catch.

Result

Before
image

After
image
image

Screen.Recording.2023-04-16.at.17.46.04.mov

Testing

  • I have thoroughly tested my changes.

The ghost button style does not affect other buttons.

The useful link cards work on light and dark theme, and they wrap nicely on smaller viewports.

@ivarnakken ivarnakken added enhancement Pull requests that make enhancements, instead of just purely new features review-needed Pull requests that need review new-feature Pull requests that introduce a new feature labels Apr 16, 2023
@ivarnakken ivarnakken requested a review from a team April 16, 2023 15:49
@ivarnakken ivarnakken self-assigned this Apr 16, 2023
@linear
Copy link

linear bot commented Apr 16, 2023

ABA-395 Improve "nyttige lenker" on public frontpage

PR made some art for us a while back for this.

Sort of a mixture of `dark` and `danger`.
@ivarnakken ivarnakken force-pushed the ivarnakken/aba-395-improve-nyttige-lenker-on-public branch from 39563a8 to 4e27143 Compare April 16, 2023 15:52
@ivarnakken ivarnakken changed the title Introduce a ghost button style Improve the useful links on the public frontpage Apr 16, 2023
Copy link
Member

@LudvigHz LudvigHz left a comment

Choose a reason for hiding this comment

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

Cool! This is massively needed 😍

Just some small comments

app/components/Button/index.tsx Show resolved Hide resolved
app/routes/overview/components/PublicFrontpage.tsx Outdated Show resolved Hide resolved
They are all given images, which helps to seperate them. The description
is now also easier to catch.
@ivarnakken ivarnakken force-pushed the ivarnakken/aba-395-improve-nyttige-lenker-on-public branch from 4e27143 to 91b0deb Compare April 16, 2023 18:25
href={item.link}
key={item.title}
rel="noreferrer"
target={item.isInternalLink ? '_self' : '_blank'}
Copy link
Member

Choose a reason for hiding this comment

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

Not a fan of this over using Link for the internal ones but I can let it slide.

@ivarnakken ivarnakken added approved Pull requests that have been approved ready-to-merge Pull requests that have been approved and are ready to be merged and removed review-needed Pull requests that need review labels Apr 16, 2023
@ivarnakken ivarnakken merged commit 6df20b6 into master Apr 17, 2023
@ivarnakken ivarnakken deleted the ivarnakken/aba-395-improve-nyttige-lenker-on-public branch April 17, 2023 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved enhancement Pull requests that make enhancements, instead of just purely new features new-feature Pull requests that introduce a new feature ready-to-merge Pull requests that have been approved and are ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants