Skip to content

[Internal_Review] Refactor: Modern GNOME inspired Aesthetic & Modular QSS#16 - #22

Merged
Akash6222 merged 19 commits into
mainfrom
style_and_theme
Aug 13, 2026
Merged

[Internal_Review] Refactor: Modern GNOME inspired Aesthetic & Modular QSS#16#22
Akash6222 merged 19 commits into
mainfrom
style_and_theme

Conversation

@demonkillerr

Copy link
Copy Markdown
Member

CI and Hardware review for #16

@demonkillerr demonkillerr added this to the v0.2.x milestone Mar 15, 2026
@demonkillerr demonkillerr added the enhancement New feature or request label Mar 15, 2026
@demonkillerr

Copy link
Copy Markdown
Member Author

CI pass

@demonkillerr

demonkillerr commented Mar 15, 2026

Copy link
Copy Markdown
Member Author
image

I think installed here looks a bit weird @sumitst05. Probably on my KDE machine only.

I feel like this should be more pronounced.. this is nothing but a mere styling nuance.

@demonkillerr

demonkillerr commented Mar 15, 2026

Copy link
Copy Markdown
Member Author
image

Another small quirk on the search tab. Can be made more coherent. I think it just looks a little out of place

@Akash6222

Copy link
Copy Markdown
Member

I'll test this week and add my review. Thanks for being so patient @sumitst05

@demonkillerr

Copy link
Copy Markdown
Member Author

@Akash6222 @sumitst05 @harshau007

I have made the following changes:

  • Deduplicate conflicting rules (package-card, status-badge, repo-badge, view-title, log-viewer were each defined twice with different values)
  • Consolidate the mixed gray/zinc palettes into one documented scale in base.qss
  • Style previously-default surfaces: QMenu dropdowns, checkbox/radio indicators (new SVG check + combo chevron in resource/icons), message boxes, horizontal scrollbars, generic QProgressBar
  • Add primary-btn (Search, Update All, Install) and danger-btn (Uninstall) button roles with the ALG blue accent
  • Align Home/Search titles to the shared view-title style instead of programmatic fonts; drop leftover no-op font code in updates_widget
  • Set WA_StyledBackground on UpdateItem so its card styling actually paints
  • Fix stale 0.2.28 version string in the About dialog (branch ships 0.2.30)

I still believe that the "installed" can be made a bit better, but I am open to keep it as it, as long as you feel it is fine. I am still testing on KDE, so please review this in your respective environments and let me know.

Sorry for the fact that I took a while to review this.

@Akash6222 Akash6222 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

GTG

@Akash6222

Copy link
Copy Markdown
Member

Hey @demonkillerr, I fixed a few issues and made some minor changes.

I found 2 issues that can be addressed later on. Here are those two issues: #26 and #25. I haven't found any other issues in Plasma, XFCE, or GNOME yet.

Soooooo, I have approved the PR.

@demonkillerr

Copy link
Copy Markdown
Member Author

looks good just now

@Akash6222
Akash6222 merged commit efa69af into main Aug 13, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants