files icon indicating copy to clipboard operation
files copied to clipboard

Selection icon doesn't follow accent color

Open cassidyjames opened this issue 5 years ago • 8 comments

What Happened

I changed my accent color and the file selection :heavy_check_mark: didn't follow that color.

Expected Behavior

The selection icon should match the accent color. We can probably use a symbolic icon, and then style it inside of Files?

Platform Information

elementary OS 6 daily with stylesheet from master.

cassidyjames avatar Nov 02 '20 22:11 cassidyjames

Since we'll have to do this across multiple repos, probably the best bet would be to use a Gtk.CheckButton and add Granite.STYLE_CLASS_BADGE. Then we can add the appropriate styles to Greenfield

danirabbit avatar Nov 02 '20 22:11 danirabbit

Note that the current selection helper is rendered by a CellRenderer - it is not a widget itself. The whole view is the widget which draws everything inside - except the overlaybar. I could try adding a style-class when rendering it. It might also be possible to make the helper an overlay widget I suppose.

jeremypw avatar Nov 03 '20 10:11 jeremypw

Adding a style class doesn't seem to affect the rendering of an icon :disappointed: It would be possible to colorize the icon but I do not know how to access the current accent color from the theme.

jeremypw avatar Nov 03 '20 11:11 jeremypw

It would be possible to colorize the icon but I do not know how to access the current accent color from the theme.

Doesn't the settings daemon store the user's colour now?

hanaral avatar Jan 31 '21 11:01 hanaral

Issue present in eOS 8, wayland session.

Accent color is Yellow We get Yellow backround on clicked files, with blue checkbox

It is a bit jarring

teamcons avatar Sep 22 '24 18:09 teamcons

This is hard to fix elegantly while we are using an icon with a fixed color hard-coded in (selection-checked) to render the select marker with an IconRenderer. It will probably have to wait until we move away from views rendered with Gtk.CellRenderer to the new dynamic widgets in Gtk4 which use widgets for the items.

jeremypw avatar Sep 23 '24 16:09 jeremypw

This is hard to fix elegantly while we are using an icon with a fixed color hard-coded in (selection-checked) to render the select marker with an IconRenderer. It will probably have to wait until we move away from views rendered with Gtk.CellRenderer to the new dynamic widgets in Gtk4 which use widgets for the items.

Thank you for your answer When would this be planned, if i may ask ?

teamcons avatar Sep 29 '24 16:09 teamcons

Although an experimental branch using Gtk4 and dynamic views was produced a while ago (see #2084), the diff with the main branch made it impractical to review. So the intention is to reduce the diff as much as possible and the first port will have conventional views with CellRenderers. Hopefully that can be accomplished during the lifetime of OS8. Once that is done then work on dynamic views can be started.

jeremypw avatar Sep 30 '24 11:09 jeremypw