Skip to content

Conversation

@ch0rizo
Copy link
Contributor

@ch0rizo ch0rizo commented Oct 29, 2025

Description

When description got to long it would squeeze the image to the right

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.

Description Before After
Image fixed Screenshot 2025-10-29 at 21 34 57 Screenshot 2025-10-29 at 21 34 46

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-1596

@ch0rizo ch0rizo requested a review from a team October 29, 2025 20:37
@ch0rizo ch0rizo self-assigned this Oct 29, 2025
@linear
Copy link

linear bot commented Oct 29, 2025

@github-actions github-actions bot added the review-needed Pull requests that need review label Oct 29, 2025
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.

Nice fixing the bug! I like that you moved the practical info as well. Perhaps it would look better if you removed the card and the title 'Praktisk info' and just had tags there? I feel like the title 'Praktisk info' doesn't add much, and that the card feels a bit out of place

@ch0rizo
Copy link
Contributor Author

ch0rizo commented Oct 29, 2025

Nice fixing the bug! I like that you moved the practical info as well. Perhaps it would look better if you removed the card and the title 'Praktisk info' and just had tags there? I feel like the title 'Praktisk info' doesn't add much, and that the card feels a bit out of place

I see what you mean. This is just a momentarily fix, and have plans to make this look better. I tested without the card, then it just look out of place. I think i'll merge this...

@ch0rizo ch0rizo merged commit 2119f35 into master Oct 30, 2025
7 checks passed
@ch0rizo ch0rizo deleted the hot-fix-image-lending branch October 30, 2025 00:51
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.

2 participants