Last modified: 2013-06-04 08:37:46 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T50898, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 48898 - [INTERLANG] Clicking on the 'Apply Settings' does not close the ULS panel
[INTERLANG] Clicking on the 'Apply Settings' does not close the ULS panel
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
UniversalLanguageSelector (Other open bugs)
unspecified
All All
: High major (vote)
: ---
Assigned To: Amir E. Aharoni
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-05-28 15:21 UTC by Runa Bhattacharjee
Modified: 2013-06-04 08:37 UTC (History)
7 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Runa Bhattacharjee 2013-05-28 15:21:50 UTC
Clicking on the 'Apply Settings' button in the Language Settings panel presently does not close the settings panel. This is a regression.


Expected behaviour: Clicking on an active 'Apply Settings' button after making any changes should apply the changes and close the panel.
Comment 1 Runa Bhattacharjee 2013-05-29 07:48:32 UTC
The bug happens if the user presses the 'Apply Settings' button only after enabling or disabling the input methods using the enable/disable toggle button in the 'Input settings'.
Comment 2 Pau Giner 2013-05-29 13:10:27 UTC
Some clarifications related to this topic are added below:

* In order to keep the behaviour consistent, when changes are produced, the apply settings button should become enabled. Changes should be persisted after the user clicks on it, and the dialog closed.

* For changes that are previewed immediately (such as changing fonts), the changes take effect immediately and the cancel button is used to revert them. This behaviour is consistent with the previous one ("Apply settings" closes the dialog and changes are valid) but provides immediate feedback to the user. The "enable/disable input method tools" button should behave as a previewed change since some users would expect their effect to be immediate.
Comment 3 Amir E. Aharoni 2013-05-29 14:12:01 UTC
It appears that https://gerrit.wikimedia.org/r/#/c/65101/ broke it.
Comment 4 Gerrit Notification Bot 2013-06-03 07:41:30 UTC
Related URL: https://gerrit.wikimedia.org/r/66380 (Gerrit Change I9f5444ffd57ddf46d197ebdb23a18c0c4600b196)
Comment 5 Gerrit Notification Bot 2013-06-03 09:42:03 UTC
Related URL: https://gerrit.wikimedia.org/r/66391 (Gerrit Change I0c05a9a1be14c1036110f84c7a0080b98f93ee1d)

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links