Skip to content

Conversation

@ncounter
Copy link
Contributor

@ncounter ncounter commented Nov 26, 2025

Before (arrows and text highlight the current problems)

image

After (arrows and text highlight the improvements)

image

@ncounter ncounter added the review-app Apply this label if you want a review app started label Nov 26, 2025
@github-actions github-actions bot added the Frontend Things related to the OBS RoR app label Nov 26, 2025
@obs-bot
Copy link
Collaborator

obs-bot commented Nov 26, 2025

@danidoni
Copy link
Contributor

I like it so much, so ordered!

@ncounter ncounter force-pushed the overview-side-links-refactoring branch from d9a30d8 to ae645c3 Compare November 27, 2025 10:06
@hennevogel
Copy link
Member

@hellcp-work
Copy link
Contributor

There's a plan to get rid of quality categories and replace them with labels after rollout

@ncounter
Copy link
Contributor Author

There's a plan to get rid of quality categories and replace them with labels after rollout

I'm glad to hear that, thanks for the insight.
In the meantime, though, I'd fix it either way.

@ncounter ncounter force-pushed the overview-side-links-refactoring branch from ae645c3 to aae02a4 Compare December 3, 2025 13:29
@ncounter ncounter force-pushed the overview-side-links-refactoring branch from aae02a4 to c6d9367 Compare December 9, 2025 08:46
@ncounter
Copy link
Contributor Author

ncounter commented Dec 9, 2025

https://www.smashingmagazine.com/2014/09/balancing-line-length-font-size-responsive-web-design/#line-length-measure-and-reading

I gave it double thinking about it and, while I strongly agree with the article content, I also believe that is something very important for daily reading activities like newspaper, posts, comments, and so on. Here instead we're dealing with a Project description a user will read once, and probably never again. I wonder if it is worth to find other suitable temporary solution actually. Opinions?

Anyway, next is to put this change behind a feature flag to not enforce it to everyone drastically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Frontend Things related to the OBS RoR app review-app Apply this label if you want a review app started

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants