Bug 499352 - [Request] Remove artificial limitation on minimum SDR brightness in HDR mode
Summary: [Request] Remove artificial limitation on minimum SDR brightness in HDR mode
Status: RESOLVED INTENTIONAL
Alias: None
Product: KScreen
Classification: Plasma
Component: kscreen-doctor (other bugs)
Version First Reported In: 6.2.5
Platform: Arch Linux Linux
: NOR wishlist
Target Milestone: ---
Assignee: kscreen-bugs-null@kde.org
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-01-31 05:54 UTC by Syboxez
Modified: 2025-02-19 15:56 UTC (History)
3 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Syboxez 2025-01-31 05:54:09 UTC
I was able to compile kscreen-doctor, changing the minimum value from 100 to 1, and it works just fine. Can this artificial limitation be removed from both the GUI and kscreen-doctor?
Comment 1 Syboxez 2025-01-31 06:03:57 UTC
Specifically, the value of `sdr-brightness`, if that was unclear.
Comment 2 Syboxez 2025-01-31 06:06:11 UTC
Not sure if it's possible to make that value a floating point, but that would be much preferred for nighttime use. Could lock any value below 1 behind a CLI to limit use to advanced users.
Comment 3 Zamundaaa 2025-02-11 16:56:53 UTC
Sorry, this is not what that setting is for. If you want the screen to go darker, use the brightness slider.
Comment 4 klaymorer 2025-02-19 07:20:16 UTC
It seems like the GUI lets you go down to 50 nits, while kscreen-doctor only lets you go to 100 nits. It seems like they should both be able to go to 50 nits. Also if you don't mind me asking here, is there any color difference between using a low SDR brightness with 100% screen brightness, and a higher SDR brightness with a lower display brightness?
Comment 5 Zamundaaa 2025-02-19 13:30:17 UTC
(In reply to klaymorer from comment #4)
> It seems like the GUI lets you go down to 50 nits, while kscreen-doctor only
> lets you go to 100 nits. It seems like they should both be able to go to 50
> nits.
I guess so.

> Also if you don't mind me asking here, is there any color difference
> between using a low SDR brightness with 100% screen brightness, and a higher
> SDR brightness with a lower display brightness?
There is no difference.
Comment 6 Bug Janitor Service 2025-02-19 13:32:04 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/libkscreen/-/merge_requests/240
Comment 7 Zamundaaa 2025-02-19 15:54:51 UTC
Git commit 4ced246d90d4a7f609b03a9d97174f04dda043b4 by Xaver Hugl.
Committed on 19/02/2025 at 13:31.
Pushed by zamundaaa into branch 'master'.

doctor: match the minimum allowed SDR luminance to what the GUI allows

M  +2    -2    src/doctor/doctor.cpp

https://invent.kde.org/plasma/libkscreen/-/commit/4ced246d90d4a7f609b03a9d97174f04dda043b4
Comment 8 Bug Janitor Service 2025-02-19 15:55:15 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/libkscreen/-/merge_requests/241
Comment 9 Zamundaaa 2025-02-19 15:56:58 UTC
Git commit 1b656fcf0247a76757728debc33b438b7f6fcb32 by Xaver Hugl.
Committed on 19/02/2025 at 15:54.
Pushed by zamundaaa into branch 'Plasma/6.3'.

doctor: match the minimum allowed SDR luminance to what the GUI allows


(cherry picked from commit 4ced246d90d4a7f609b03a9d97174f04dda043b4)

Co-authored-by: Xaver Hugl <xaver.hugl@gmail.com>

M  +2    -2    src/doctor/doctor.cpp

https://invent.kde.org/plasma/libkscreen/-/commit/1b656fcf0247a76757728debc33b438b7f6fcb32