Skip to content

Emblems: update symbolic icons for gtk 4.22#1425

Open
garaevdi wants to merge 1 commit into
elementary:mainfrom
garaevdi:os9-fixup-emblems
Open

Emblems: update symbolic icons for gtk 4.22#1425
garaevdi wants to merge 1 commit 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!

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

Labels

None yet

Projects

Status: Needs Review

Development

Successfully merging this pull request may close these issues.

3 participants