Skip to content

Improve card readability#554

Open
hojjatabdollahi wants to merge 2 commits into
pop-os:masterfrom
hojjatabdollahi:hojjat/declutter
Open

Improve card readability#554
hojjatabdollahi wants to merge 2 commits into
pop-os:masterfrom
hojjatabdollahi:hojjat/declutter

Conversation

@hojjatabdollahi

Copy link
Copy Markdown

This is a suggestion. Feel free to close it.
Currently the cards look too dense and too cluttered.

  • The height of the cards is too short, the description gets too close to the bottom
  • We don't have a dimmed secondary text token in the theme. The body and caption text are way too similar making the cards unreadable.

Here's what I did:

  • I added a description_text_style that is dimmed a bit (similar to how it's done for the shortcut keys in the context menus in libcosmic).
  • Made the description text font size a bit smaller too.
  • Made the text ellipsize instead of just cut off.
  • and made the card taller.
  • Also looks like the recent changes introduced some formatting and clippy issues, I fixed those in a separate commit.

To me it looks cleaner and less headache-y. :D

Before:
Screenshot_2026-06-09_18-18-16

After:
Screenshot_2026-06-09_18-18-33


  • I have disclosed use of any AI generated code in my commit messages.
    • If you are using an LLM, and do not fully understand the changes it is making to the code base, do not create a PR.
    • In our experience, AI generated code often results in overly complex code that lacks enough context for a proper fix or feature inclusion. This results in considerably longer code reviews. Due to this, AI authored or partially authored PRs may be closed without comment.
  • I understand these changes in full and will be able to respond to review comments.
  • My change is accurately described in the commit message.
  • My contribution is tested and working as described.
  • I have read the Developer Certificate of Origin and certify my contribution under its conditions.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant