search

udifuchs / icc-brightness

Control OLED display brightness by applying ICC color profiles.
MIT License
171 stars 25 forks source link

Multi-monitor support #12

Open nomego opened 5 years ago

nomego commented 5 years ago

Right now, everything is setup to just pick the first display, so it doesn't work with multiple monitors. Could you adapt it to act on all displays?

Also, for me with colormgr 1.4.3 (not sure since when it works), I can do:

$ colormgr get-devices-by-kind display --filter object-path --value-only
/org/freedesktop/ColorManager/devices/xrandr_Dell_Inc__DELL_U2718Q_FN84K78E04TL_patrik_1000
/org/freedesktop/ColorManager/devices/xrandr_Dell_Inc__DELL_U2718Q_FN84K83Q0MRL_patrik_1000

if you want to drop/simplify the get_object_path() helper.

martinflack commented 4 years ago

It might be nice to make the affected display configurable explicitly, configurable as the "first OLED" display, or configurable as the "attached display" (for laptops; instead of external displays).

Pajn commented 4 years ago

Having the same problem, my colormgr output if it helps:

❯ colormgr get-devices-by-kind display --filter object-path --value-only
/org/freedesktop/ColorManager/devices/xrandr_Lenovo_Group_Limited_LEN_P27u_10_1295471686_rse_1000
/org/freedesktop/ColorManager/devices/xrandr_Lenovo_Group_Limited_LEN_P27u_10_1295471193_rse_1000
/org/freedesktop/ColorManager/devices/xrandr_eDP_1_rse_1000