GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Remapping DDC brightness control to contrast on external display? · Issue #767 · MonitorControl/MonitorControl

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

New issue
Copy link
New issue
Copy link
Closed
#769
Closed
Remapping DDC brightness control to contrast on external display?#767
#769
Copy link
Assignees
waydabber
Labels
bugIssue is a bug (e.g. Crash, …)Issue is a bug (e.g. Crash, …)doneWork on this issue is complete. Will be available on next releaseWork on this issue is complete. Will be available on next release

Description

@waydabber
waydabber
opened on Nov 4, 2021
Issue body actions

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. waydabber
    added
    Type: Question
    bugIssue is a bug (e.g. Crash, …)Issue is a bug (e.g. Crash, …)
    and removed
    Type: Question
    on Nov 4, 2021
  2. waydabber
    self-assigned this
    on Nov 4, 2021
  3. waydabber
    added
    doneWork on this issue is complete. Will be available on next releaseWork on this issue is complete. Will be available on next release
    on Nov 4, 2021
  4. waydabber commented on Nov 4, 2021

    @waydabber
    waydabber
    on Nov 4, 2021
    MemberAuthor
    More actions

    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.

  5. waydabber
    linked a pull request that will close this issueVarious fixes and improvements for v4.0.1 #769on Nov 4, 2021
  6. waydabber
    closed this as completedin #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

  • @waydabber
    waydabber

Labels

bugIssue is a bug (e.g. Crash, …)Issue is a bug (e.g. Crash, …)doneWork on this issue is complete. Will be available on next releaseWork 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

    You are visitor 01402503 — GeezerVison does not store the pages you look up.
    Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.