Skip to content

Emblems: update symbolic icons for gtk 4.22#1425

Merged
danirabbit merged 4 commits into
elementary:mainfrom
garaevdi:os9-fixup-emblems
Jun 24, 2026
Merged

Emblems: update symbolic icons for gtk 4.22#1425
danirabbit merged 4 commits into
elementary:mainfrom
garaevdi:os9-fixup-emblems

Conversation

@garaevdi

Copy link
Copy Markdown
Contributor

Part four of a grand symbolic icons update. That one is a relatively small one, most deletions are a cleanup of some unused gradients. Tested with gtk 4.22, 4.14 and 3.24, but I didn't have time to create showcase images sowwy

@danirabbit danirabbit added this to OS 9 May 31, 2026
@ryonakano ryonakano moved this to Needs Review in OS 9 May 31, 2026
@ryonakano

Copy link
Copy Markdown
Member

Haven't tested yet, but the design of emblems/symbolic/emblem-default-symbolic.svg looks like different. Is this expected?

image

@garaevdi
garaevdi force-pushed the os9-fixup-emblems branch from c16d87c to fb41572 Compare May 31, 2026 22:05
@garaevdi

Copy link
Copy Markdown
Contributor Author

Haven't tested yet, but the design of emblems/symbolic/emblem-default-symbolic.svg looks like different. Is this expected?
image

Nope, seems like a mistake on my end, fixed!

@ryonakano
ryonakano requested review from a team and ryonakano May 31, 2026 22:40
@ryonakano

Copy link
Copy Markdown
Member

After with GTK 4.22.4

Icon Name Screenshot
emblem-default image
emblem-disabled-symbolic N/A
emblem-documents-symbolic image
emblem-downloads-symbolic N/A
emblem-enabled-symbolic N/A
emblem-error-symbolic N/A
emblem-favorite-symbolic image
emblem-important-symbolic image
emblem-mixed-symbolic N/A
emblem-photos-symbolic image
emblem-readonly-symbolic image
emblem-shared-symbolic image
emblem-symbolic-link-symbolic image
emblem-synchronized-symbolic N/A
emblem-system-symbolic image
emblem-unreadable-symbolic image
emblem-warning-symbolic N/A

Before with GTK 4.22.4

Icon Name Screenshot
emblem-default image
emblem-disabled-symbolic N/A
emblem-documents-symbolic image
emblem-downloads-symbolic N/A
emblem-enabled-symbolic N/A
emblem-error-symbolic N/A
emblem-favorite-symbolic image
emblem-important-symbolic image
emblem-mixed-symbolic N/A
emblem-photos-symbolic image
emblem-readonly-symbolic image
emblem-shared-symbolic image
emblem-symbolic-link-symbolic image
emblem-synchronized-symbolic N/A
emblem-system-symbolic image
emblem-unreadable-symbolic image
emblem-warning-symbolic N/A

@ryonakano

Copy link
Copy Markdown
Member

Looks like some emblem icons are missing in IconBrowser. I opened elementary/iconbrowser#76.

Before with GTK 4.22.4

Icon Name Screenshot
emblem-git-modified-symbolic image
emblem-git-new-symbolic image
emblem-music-symbolic image
emblem-ok-symbolic image
emblem-videos-symbolic image

Before with GTK 4.22.4

Icon Name Screenshot
emblem-git-modified-symbolic image
emblem-git-new-symbolic image
emblem-music-symbolic image
emblem-ok-symbolic image
emblem-videos-symbolic image

@ryonakano

Copy link
Copy Markdown
Member

After with GTK 4.14.5

Icon Name Screenshot
emblem-default image
emblem-disabled-symbolic N/A
emblem-documents-symbolic image
emblem-downloads-symbolic N/A
emblem-enabled-symbolic N/A
emblem-error-symbolic N/A
emblem-favorite-symbolic image
emblem-git-modified-symbolic image
emblem-git-new-symbolic image
emblem-important-symbolic image
emblem-mixed-symbolic N/A
emblem-music-symbolic image
emblem-ok-symbolic image
emblem-photos-symbolic image
emblem-readonly-symbolic image
emblem-shared-symbolic image
emblem-symbolic-link-symbolic image
emblem-synchronized-symbolic N/A
emblem-system-symbolic image
emblem-unreadable-symbolic image
emblem-videos-symbolic image
emblem-warning-symbolic N/A

@ryonakano ryonakano 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.

LGTM. Thanks!

@wpkelso wpkelso 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.

Hi @garaevdi, thank you so much for doing this series of PRs, I'm sorry it's taken me so long to get around to reviewing this one.

@ryonakano Thank you as well for doing the testing between versions, it's been very helpful 😃

Comment thread emblems/symbolic/emblem-unreadable-symbolic.svg
@ryonakano
ryonakano requested a review from wpkelso June 24, 2026 11:16
@danirabbit
danirabbit merged commit 0078423 into elementary:main Jun 24, 2026
3 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in OS 9 Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants