Remapping DDC brightness control to contrast on external display? · Issue #767 · MonitorControl/MonitorControl · GitHub
Skip to content

Remapping DDC brightness control to contrast on external display? #767

Closed

Description

@waydabber

Discussed in #764

Originally posted by renaudguerin November 3, 2021

Before opening the issue, have you...?

  • Searched for existing issues

Is your feature request related to a problem? Please describe

My work setup is an Intel MBP -> CalDigit Thunderbolt hub -> USB-C to Displayport cable -> BenQ BL3201PT

This monitor doesn't respond to brightness controls at all with MonitorControl 3.1.1
It does respond to volume and contrast though, and the contrast menu slider works fine.
This means the brightness keys do nothing, and I have to reach for the menu instead.

Describe the solution you'd like

A preference checkbox to enable permanent remapping of Brightness keyboard controls to Contrast instead.
Or (probably cleaner) don't hardcode the mapping of keys -> feature name -> ID at all but make it configurable, with sane defaults out of the box.

Describe alternatives you've considered

As I was writing this I've discovered that 4.0.0 introduces a keyboard shortcut for contrast (Ctrl-Add-Cmd + brightness key)

This is an improvement, but it mostly benefits people who are already able to control brightness using the unmodified keys. In my case, the brightness keys are useless without this complex modifier, so I would much prefer direct access.

I appreciate the concerns around not cluttering the preferences UI, but IMHO some of the current settings are even more niche than this one (e.g. "Further lower brightness via software dimming")

Maybe a "batteries included but optional" approach would work here ? i.e. don't hardcode brightness/contrast/volume controls per se, make the brightness keys-> feature name ->IDs mapping configurable for advanced users, and ship with sane defaults that work for most people.

Anything else?

No response

Activity

  1. self-assigned this
    on Nov 4, 2021
  2. added
    doneWork on this issue is complete. Will be available on next release
    on Nov 4, 2021
  3. waydabber commented on Nov 4, 2021

    @waydabber
    MemberAuthor

    On the Intel codepath the control code override does not work indeed in v4.0.0. Thanks @renaudguerin

    This is now fixed, it will work in v4.0.1.

  4. linked a pull request that will close this issueVarious fixes and improvements for v4.0.1 #769on Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugIssue is a bug (e.g. Crash, …)doneWork on this issue is complete. Will be available on next release

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions