After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 784761 - keyboard: Consider multiple bindings for default value
keyboard: Consider multiple bindings for default value
Status: RESOLVED OBSOLETE
Product: gnome-control-center
Classification: Core
Component: Keyboard
unspecified
Other All
: Normal normal
: ---
Assigned To: Rui Matos
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-07-10 17:52 UTC by Florian Müllner
Modified: 2021-06-09 16:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
keyboard: Consider multiple bindings for default value (1.83 KB, patch)
2017-07-10 17:52 UTC, Florian Müllner
reviewed Details | Review

Description Florian Müllner 2017-07-10 17:52:05 UTC
As mentioned in https://bugzilla.gnome.org/show_bug.cgi?id=673078#c31 and #c32, it may make sense to consider multiple bindings for determining whether a value is at its default.

Pros:
 - it *is* more correct
 - it simplifies the code (unless we want to allow permutations)

Contras:
 - if the user is unaware that non-primary bindings are a thing, it's
   confusing if setting an item to the default shortcut (as shown in
   the UI) isn't always the same as resetting it

I'll leave this here for discussion, if we decide that we want this, I'll write a proper commit message :-)
Comment 1 Florian Müllner 2017-07-10 17:52:10 UTC
Created attachment 355288 [details] [review]
keyboard: Consider multiple bindings for default value
Comment 2 Georges Basile Stavracas Neto 2018-01-26 22:06:30 UTC
Review of attachment 355288 [details] [review]:

The patch looks good to me, and actually removed more lines of code than added =)

But we need to have some more feedback here about the feature itself.
Comment 3 Georges Basile Stavracas Neto 2018-01-26 22:07:02 UTC
CCing Allan. Could you please share your thoughts here?
Comment 4 André Klapper 2021-06-09 16:27:10 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org.
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version, then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a new bug report at
  https://gitlab.gnome.org/GNOME/gnome-control-center/-/issues/

Thank you for your understanding and your help.