Bug 453167 - GTK CSD window close icon (`window-close-symbolic`) does not match that of SSD apps
Summary: GTK CSD window close icon (`window-close-symbolic`) does not match that of SS...
Status: RESOLVED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: Icons (show other bugs)
Version: 5.24.4
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: visual-design
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-04-29 00:41 UTC by Eduardo Alvarado
Modified: 2025-01-04 18:46 UTC (History)
8 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.11
Sentry Crash Report:


Attachments
Note the orange arrow (436.95 KB, image/png)
2022-04-29 00:41 UTC, Eduardo Alvarado
Details
kirigami(?) apps with a fake pop-up window using the "window-close-symbolic" icon. On mouse hover the "window-close" icon then appears. (1.05 MB, image/png)
2022-06-12 13:08 UTC, Paul McAuley
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Eduardo Alvarado 2022-04-29 00:41:38 UTC
Created attachment 148449 [details]
Note the orange arrow

SUMMARY
Because the close button in the Breeze decoration is not a circled X, but a single X by default, some icons about closing things in the icon theme should match.

See attached image. As a side effect, applications using libadwaita (GTK 4) should gain consistency in their client-side decorations.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux
KDE Plasma Version: 5.24.4
KDE Frameworks Version: 5.93.0
Qt Version: 5.15.3
Kernel Version: 5.17.5-arch1-1 (64-bit)
Graphics Platform: Wayland
Processors: 12 × Intel® Core™ i5-10400 CPU @ 2.90GHz
Memory: 15.5 GiB of RAM
Graphics Processor: Mesa Intel® UHD Graphics 630
Comment 1 Eduardo Alvarado 2022-04-29 00:54:53 UTC
This would also affect the close icon in notifications.
Comment 2 Nate Graham 2022-05-02 18:33:09 UTC
> This would also affect the close icon in notifications.
That's unrelated to the issue in GTK apps.

*** This bug has been marked as a duplicate of bug 449876 ***
Comment 3 Nate Graham 2022-05-15 18:59:08 UTC
Artem, is there anything we can do here? Is this fixed by https://invent.kde.org/plasma/breeze-gtk/-/merge_requests/56?
Comment 4 Artem Grinev 2022-05-15 19:35:49 UTC
(In reply to Nate Graham from comment #3)
> Artem, is there anything we can do here? Is this fixed by
> https://invent.kde.org/plasma/breeze-gtk/-/merge_requests/56?

No, it fixes https://bugs.kde.org/show_bug.cgi?id=449876 for dialogs

Technically this particular bug is a Breeze Icons issue which should provide correct default gtk icons. Libadwaita uses them and not our injections of KWin's decorations. So maybe this should be moved there.
Comment 5 Nate Graham 2022-05-16 16:03:00 UTC
Cool. What icon should be changed, and to what?
Comment 6 Artem Grinev 2022-05-16 17:01:09 UTC
I suspect it's "window-close-symbolic"
Comment 7 Nate Graham 2022-05-16 19:19:47 UTC
Indeed, `window-close-symbolic` is circled.

If we change that to just be an uncircled X, I wonder if it could have any other consequences...
Comment 8 Paul McAuley 2022-06-12 13:08:26 UTC
Created attachment 149642 [details]
kirigami(?) apps with a fake pop-up window using the "window-close-symbolic" icon. On mouse hover the "window-close" icon then appears.

(In reply to Eduardo Alvarado from comment #1)
> This would also affect the close icon in notifications.

The close icon in notifications uses "window-close", not "window-close-symbolic".

I have been investigating this for my 3rd party icons, and the only applications I have found so far which use "window-close-symbolic" are (I think) kirigami-based ones which have a fake pop-up window. See the attached screenshot. When you hover the close button with the mouse in these fake pop-up windows, they then show the "window-close" icon.


(In reply to Nate Graham from comment #7)
> Indeed, `window-close-symbolic` is circled.
> 
> If we change that to just be an uncircled X, I wonder if it could have any
> other consequences...

An uncircled X would need to be larger than the current icon to match the other symbolic GTK CSD window icons. In the case of the kirigami(?) examples I mentioned above this would cause problems with the mouse-over of "window-close" being smaller. Either you could change the (also circled but red) "window-close" icon to be larger as well to match, or get the kirigami(?) apps to draw their own mouse-over highlight like GTK rather than using an icon.

Overall, I like that Adwaita GTK now uses the "symbolic" icons for windows and I now use Adwaita GTK rather than Breeze GTK for that reason. It partially solves the regression in Wayland in bug 437082, and allows third party icon themes to now have at least some consistency.
Comment 9 Bug Janitor Service 2022-07-03 21:57:43 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/breeze-icons/-/merge_requests/211
Comment 10 Bug Janitor Service 2022-07-03 22:04:45 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/breeze-icons/-/merge_requests/212
Comment 11 Nate Graham 2024-10-22 22:31:24 UTC
Will be fixed with https://invent.kde.org/frameworks/breeze-icons/-/merge_requests/349 now.
Comment 12 Nate Graham 2025-01-04 18:46:38 UTC
Git commit a22293658d0ec378bdf243781afbce5f26536a43 by Nate Graham.
Committed on 04/01/2025 at 18:46.
Pushed by ngraham into branch 'master'.

Make the close icons be black X symbols

Make the close icons be black X symbols

Black is is a more correct color (closing windows and tabs isn't
destructive, so it shouldn't use the red color), and omitting the
circle makes it match the default window decoration icon and feel
lighter and breezier.

This implements what was decided in
https://invent.kde.org/teams/vdg/issues/-/issues/62.
FIXED-IN: 6.11

M  +8    -11   icons/actions/16/window-close.svg
M  +12   -8    icons/actions/22/window-close.svg
M  +12   -8    icons/actions/32/window-close.svg

https://invent.kde.org/frameworks/breeze-icons/-/commit/a22293658d0ec378bdf243781afbce5f26536a43