Skip to content

Comments

[RFC] - UI: rework module's UI (sliders, combobox, checkbox).#20388

Open
TurboGit wants to merge 5 commits intomasterfrom
po/ui-sliders
Open

[RFC] - UI: rework module's UI (sliders, combobox, checkbox).#20388
TurboGit wants to merge 5 commits intomasterfrom
po/ui-sliders

Conversation

@TurboGit
Copy link
Member

@TurboGit TurboGit commented Feb 23, 2026

With the recent change to get more contrast on sliders it is now possible to refine them without loosing readability:

  • Make the slider text font smaller.
  • Make the slider thiner.
  • Remove some padding.
Before after
Capture d’écran du 2026-02-23 12-14-47 Capture d’écran du 2026-02-23 12-12-29

Also, with this PR the sections are standing out better.

@TurboGit TurboGit changed the title UI: rework sliders. [RFC] - UI: rework sliders. Feb 23, 2026
@jenshannoschwalm
Copy link
Collaborator

For me this is really an improvement as it gives a lot of vertical room and has less "missing characters" for longer text.

Not so sure for people with visibility impairments though. Would we want a preference option for this like "dense module layout" ?

@TurboGit
Copy link
Member Author

An option could be good indeed but there is also a CSS change which makes things harder. I'll see if we can improve this. And I have also noticed that the neutral point of the slider is not visible anymore. So this needs some more work.

@wpferguson
Copy link
Member

I have old eyes and thus some issues especially with contrast. The new sliders look fine to me, even on my 4K laptop screen (vs my 32 inch monitor).

@TurboGit
Copy link
Member Author

TurboGit commented Feb 23, 2026

And with the neutral point indicator, on the top now as there is no space left below the slider:

Before after
Capture d’écran du 2026-02-23 12-14-47 image

@zisoft
Copy link
Collaborator

zisoft commented Feb 23, 2026

My first thought was "great!". And it looks really great in the Colorbalance RBG module where ony sliders are involved.

But in other modules with mixed controls it looks unbalanced, all other controls have bigger text:

Bildschirmfoto 2026-02-23 um 19 25 14 Bildschirmfoto 2026-02-23 um 19 19 04

The checkbox above the exposure slider now looks huge. The proportions are no longer consistent.

And the handles are much smaller now as well, making it harder to hit them with the mouse.

@TurboGit
Copy link
Member Author

Indeed, this needs more work to have consistent text size.

@TurboGit
Copy link
Member Author

image

@TurboGit
Copy link
Member Author

@zisoft : Let me know if you see some more issues. TIA.

@zisoft
Copy link
Collaborator

zisoft commented Feb 23, 2026

I use the circle for the slider handles, they are cut off at the bottom:

Bildschirmfoto 2026-02-23 um 21 49 37

It looks more consistent now, but I think it is very small on a high resolution monitor.
And the checkbox seems too big in comparison.

Bildschirmfoto 2026-02-23 um 21 48 51

@TurboGit TurboGit force-pushed the po/ui-sliders branch 5 times, most recently from 602707d to 5b8739a Compare February 24, 2026 07:55
@TurboGit
Copy link
Member Author

TurboGit commented Feb 24, 2026

@zisoft : The condensed mode can now be selected as an option. The circle should not be cut anymore and checkbox size should also be adjusted now.

@TurboGit TurboGit added scope: UI user interface and interactions scope: codebase making darktable source code easier to manage labels Feb 24, 2026
@TurboGit TurboGit modified the milestone: 5.6 Feb 24, 2026
@TurboGit TurboGit self-assigned this Feb 24, 2026
@zisoft
Copy link
Collaborator

zisoft commented Feb 24, 2026

Yes, looks much better now:

Bildschirmfoto 2026-02-24 um 09 11 54 Bildschirmfoto 2026-02-24 um 09 12 36

The history is nearly unreadable small and the font baseline does not match the numbers:

Bildschirmfoto 2026-02-24 um 09 20 29 Bildschirmfoto 2026-02-24 um 09 13 16

With the recent change to get more contrast on sliders it is now possible
to refine them without loosing readability:

- Make the text font smaller.
- Make the slider thiner.
- Remove some padding.
@TurboGit TurboGit changed the title [RFC] - UI: rework sliders. [RFC] - UI: rework module's UI (sliders, combobox, checkbox). Feb 24, 2026
@TurboGit
Copy link
Member Author

@zisoft : Another iteration, fixes the history too small.

@zisoft
Copy link
Collaborator

zisoft commented Feb 24, 2026

This is how it looks now:

Bildschirmfoto 2026-02-24 um 13 00 17

The font size of the history is better now, baseline alignment ist still off.

I think there may be many other places to tweak, see the text of the checkboxes in the color picker section which is barely readable.

@TurboGit
Copy link
Member Author

The font size of the history is better now, baseline alignment ist still off.

Which OS? Font? I don't have this baseline alignment issue on my side.

@zisoft
Copy link
Collaborator

zisoft commented Feb 24, 2026

macOS, of course :)

No special font, just darktable default settings elegant-grey

Without the condensed setting everything looks perfect:

Bildschirmfoto 2026-02-24 um 14 12 09

@jenshannoschwalm
Copy link
Collaborator

Here it looks almost good now with all themes.

  1. geotagging still has an issue as
Bildschirmfoto vom 2026-02-24 15-21-53
  1. Couldn't see a message for "restart required" after changing preference of condensed mode

@TurboGit
Copy link
Member Author

Couldn't see a message for "restart required" after changing preference of condensed mode

Indeed, I suspect a bug that is not due to this PR.

@TurboGit
Copy link
Member Author

@jenshannoschwalm @zisoft : Last changes should fix all issues except maybe the size of the font in the entry geotaging & export for example. See #20395.

@zisoft
Copy link
Collaborator

zisoft commented Feb 24, 2026

More tests on macOS:

History and exposure module are looking great now, baseline offset is fixed:
Bildschirmfoto 2026-02-24 um 17 32 43
Bildschirmfoto 2026-02-24 um 17 33 34

Geotagging module right after darktable launch, no Lua in my dev environment:
Bildschirmfoto 2026-02-24 um 17 36 20

After switching to darkroom and back to lighttable. The controls are now on the same line with the label, maybe that's the cause?

Bildschirmfoto 2026-02-24 um 17 34 18

Couldn't see a message for "restart required" after changing preference of condensed mode

Indeed, I suspect a bug that is not due to this PR.

That's strange because I always get that message after changing the setting. No matter if I turn it on or off:
Bildschirmfoto 2026-02-24 um 17 37 36

Besides the geotagging bug everything is perfect now!

@TurboGit
Copy link
Member Author

Indeed without Lua, when I switch back to lighttable from darkroom I also have the size of entry being wrong.

This does two things:
- Change the settings of bauhaus
- Load a specific CSS file: condensed.css
Others CSS are just patial CSS handled internally.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: codebase making darktable source code easier to manage scope: UI user interface and interactions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants