Maniphest T3208

Removing layout keeps 'Apply' button disabled in Keyboard Settings
Closed, ResolvedPublic
Actions

Tags

Subscribers

zmike, bu5hm4n, oz

Assigned To

zmike

Authored By

oz, Feb 22 2016

Description

Settings -> All -> Input -> Keyboard
Add any layout in Configuration tab -> Press Apply
Remove any layout
Observe that Apply button is still in disabled mode as if there were no changes

OS: Ubuntu 14.04.4 LTS, 3.13.0-77-generic

Details

Commits: rEFL23ac20bfce3f: Widgets: remove duplicate declaration of language,changed signal.
rEFL5affad30b4ee: Widgets: remove duplicate declaration of access,changed signal.
rE0de37612bbb0: force changed when adding or removing keyboard layouts in config
rEbbd4d2fbae50: force changed when adding or removing keyboard layouts in config

oz created this task.Feb 22 2016, 4:09 AM

Herald assigned this task to zmike. · View Herald TranscriptFeb 22 2016, 4:09 AM

Herald added a subscriber: zmike. · View Herald Transcript

oz updated the task description. (Show Details)Feb 22 2016, 5:11 PM

zmike closed this task as Resolved by committing rEbbd4d2fbae50: force changed when adding or removing keyboard layouts in config.Feb 26 2016, 1:46 PM

zmike added a commit: rEbbd4d2fbae50: force changed when adding or removing keyboard layouts in config.

The fix has side effect: after removing a layout "Up", "Down" and "Remove" buttons are kept active although there is no active selection.

zmike closed this task as Resolved by committing rE0de37612bbb0: force changed when adding or removing keyboard layouts in config.Mar 9 2016, 10:12 AM

zmike added a commit: rE0de37612bbb0: force changed when adding or removing keyboard layouts in config.

tasn added a commit: rEFL5affad30b4ee: Widgets: remove duplicate declaration of access,changed signal..Mar 24 2016, 3:13 PM

tasn added a commit: rEFL23ac20bfce3f: Widgets: remove duplicate declaration of language,changed signal..