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 739891 - Mapping of display-less tablets sometimes stick to older settings
Mapping of display-less tablets sometimes stick to older settings
Status: RESOLVED FIXED
Product: gnome-settings-daemon
Classification: Core
Component: wacom
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Carlos Garnacho
gnome-settings-daemon-maint
Depends on:
Blocks:
 
 
Reported: 2014-11-10 15:41 UTC by Carlos Garnacho
Modified: 2014-11-11 14:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
common: Refactor output-from-settings input device configuration. (3.12 KB, patch)
2014-11-10 15:42 UTC, Carlos Garnacho
committed Details | Review
common: Unset output-from-settings if no matching output is found (1001 bytes, patch)
2014-11-10 15:42 UTC, Carlos Garnacho
committed Details | Review
common: Update output-from-settings when the outputs change (1.35 KB, patch)
2014-11-10 15:43 UTC, Carlos Garnacho
committed Details | Review

Description Carlos Garnacho 2014-11-10 15:41:24 UTC
I'm attaching some patches that:
- Make sure that display-less tablets can be set back to mapping to the full desktop extent
- Make sure that the output is looked up again from settings when the connected outputs change

The first patch is a minor refactor for these 2 fixes.
Comment 1 Carlos Garnacho 2014-11-10 15:42:51 UTC
Created attachment 290353 [details] [review]
common: Refactor output-from-settings input device configuration.

The code that handled this when creating a GsdInputInfo and whenever
the settings changed were almost the same, put it in a common place.
Comment 2 Carlos Garnacho 2014-11-10 15:42:57 UTC
Created attachment 290354 [details] [review]
common: Unset output-from-settings if no matching output is found

This ensures that either an output is guessed for the device again, or for
display-less tablets, the default matrix spanning full desktop extents is
reinstaurated.
Comment 3 Carlos Garnacho 2014-11-10 15:43:01 UTC
Created attachment 290355 [details] [review]
common: Update output-from-settings when the outputs change

Outputs being connected/disconnected should update the input devices that
are mapped to these through config, so they are remapped/unmapped correctly
after the change.
Comment 4 Rui Matos 2014-11-11 14:22:24 UTC
Review of attachment 290353 [details] [review]:

nice
Comment 5 Rui Matos 2014-11-11 14:25:04 UTC
Review of attachment 290354 [details] [review]:

looks good
Comment 6 Rui Matos 2014-11-11 14:30:33 UTC
Review of attachment 290355 [details] [review]:

makes sense
Comment 7 Rui Matos 2014-11-11 14:34:03 UTC
Carlos, this seems like it should go in for 3.14.2 too, right?
Comment 8 Carlos Garnacho 2014-11-11 14:48:47 UTC
(In reply to comment #7)
> Carlos, this seems like it should go in for 3.14.2 too, right?

Yep :), will push to both master and gnome-3-14
Comment 9 Carlos Garnacho 2014-11-11 14:49:38 UTC
Attachment 290353 [details] pushed as 6597c36 - common: Refactor output-from-settings input device configuration.
Attachment 290354 [details] pushed as 7f3955b - common: Unset output-from-settings if no matching output is found
Attachment 290355 [details] pushed as f0e5d24 - common: Update output-from-settings when the outputs change