Skip to content

fix: improve Windows custom-state icon contrast - #10622

Merged
Rello merged 3 commits into
nextcloud:masterfrom
jsk1004ha:fix/windows-custom-state-icon-contrast
Aug 27, 2026
Merged

fix: improve Windows custom-state icon contrast#10622
Rello merged 3 commits into
nextcloud:masterfrom
jsk1004ha:fix/windows-custom-state-icon-contrast

Conversation

@jsk1004ha

@jsk1004ha jsk1004ha commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Summary

Give the Windows locked and shared custom-state icons a light fill with a dark outline. The dual-tone silhouettes remain visible on both light and dark Explorer backgrounds instead of disappearing when a black icon is rendered on the dark theme.

  • Shared icon updated to Material Symbols person_add silhouette with light fill and dark outline.
  • Locked icon updated with light interior and dark outline.
  • All 18 tracked PNG sizes regenerated from updated SVG sources.
  • Added automated SVG and PNG contrast regression test (test/testcustomstateicons.cpp).

Fixes #6070.

Validation

  • Tested SVG and PNG contrast with automated regression test suite (TestCustomStateIcons).
  • Verified dual-tone silhouettes on dark/light themes.
  • Verified all 18 tracked PNG assets and SVG sources.
  • git diff --check

Assisted-by: ChatGPT:GPT-5.6-Pro

@Rello

Rello commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

@kra-mo can you check this?

@Rello Rello added the design Design, UI, UX, etc. label Aug 18, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

馃挕 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 49b27b6ecc

鈩癸笍 About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 馃憤.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread theme/cfapishellext_custom_states/0-locked.svg
Comment thread theme/cfapishellext_custom_states/0-locked.svg

@kra-mo kra-mo 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.

Fair, but the share icon should be a single uniform shape instead of this, which is how it renders for me:

Image

But even better would be to base it off of the same share icon that we use everywhere:

https://fonts.google.com/icons?icon.size=24&icon.color=%231f1f1f&icon.set=Material+Symbols&selected=Material+Symbols+Outlined:person_add:FILL@0;wght@400;GRAD@0;opsz@24&icon.query=person+add

@jsk1004ha
jsk1004ha force-pushed the fix/windows-custom-state-icon-contrast branch from 49b27b6 to d68ac17 Compare August 18, 2026 12:36

Copy link
Copy Markdown
Contributor Author

@kra-mo I updated the shared overlay in e6d9af861337b51f36f99d684fff66e2fe9e62ad to use a single Material Symbols Outlined person_add silhouette, while retaining the white fill and black outline required for light/dark Explorer contrast. I regenerated every tracked shared PNG size (24, 32, 40, 48, 64, 128, 256, 512, and 1024 px) and verified their dimensions plus the presence of both opaque light and dark pixels. The CustomStateIcons regression test added in the preceding commit still covers both SVGs and all 18 raster assets. The GitHub Actions runs are currently marked action_required, so they need the repository-side approval before executing.

Assisted-by: ChatGPT:GPT-5.6-Pro
Signed-off-by: Junseo Kim <129354802+jsk1004ha@users.noreply.github.com>
@jsk1004ha
jsk1004ha force-pushed the fix/windows-custom-state-icon-contrast branch from e6d9af8 to d8375f3 Compare August 19, 2026 08:06
@Rello

Rello commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

is it just me, or are there additional pixels in some icons?
Bildschirmfoto 2026-08-19 um 12 51 59
Bildschirmfoto 2026-08-19 um 12 51 50

@kra-mo

kra-mo commented Aug 20, 2026

Copy link
Copy Markdown
Member

@Rello it is not just you

@Rello

Rello commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

@jsk1004ha can you please check the icons again?

@Rello Rello added the needs info User feedback is required. Will be closed if not responded. label Aug 24, 2026
Assisted-by: Codex:gpt-5.6-sol
Signed-off-by: jsk1004ha <129354802+jsk1004ha@users.noreply.github.com>
@jsk1004ha

Copy link
Copy Markdown
Contributor Author

I checked and found an issue where the icon appeared broken, so I鈥檝e fixed it.

@github-actions github-actions Bot removed the needs info User feedback is required. Will be closed if not responded. label Aug 25, 2026

@kra-mo kra-mo 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.

Ok, one more thing is that I think it should use the filled variant of the icon, as that looks better as a badge.

Assisted-by: Codex:gpt-5.6-sol
Signed-off-by: jsk1004ha <129354802+jsk1004ha@users.noreply.github.com>
@Rello Rello added this to the 35.0.0 milestone Aug 26, 2026
@Rello
Rello merged commit f87793d into nextcloud:master Aug 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

design Design, UI, UX, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Lock icon color with Dark theme

4 participants