Last modified: 2013-11-21 11:47:33 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 T55772, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 53772 - ULS should indicate when display preferences change is complete
ULS should indicate when display preferences change is complete
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
UniversalLanguageSelector (Other open bugs)
unspecified
All All
: High normal (vote)
: ---
Assigned To: Santhosh Thottingal
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-09-04 19:36 UTC by Chris McMahon
Modified: 2013-11-21 11:47 UTC (History)
9 users (show)

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


Attachments

Description Chris McMahon 2013-09-04 19:36:25 UTC
In Display Settings/Fonts, change the font in the selectbox
Quickly click for a random page in the wiki

New font is likely not displayed

Changing preferences for fonts takes some time, on the order of several seconds.  Suggest a spinner or other acknowledgement to the user that the preferences have been changed successfully.
Comment 1 Gerrit Notification Bot 2013-09-04 19:39:59 UTC
Change 82654 had a related patch set uploaded by Cmcmahon:
changing prefs takes a long time, see Bug 53772

https://gerrit.wikimedia.org/r/82654
Comment 2 Siebrand Mazeland 2013-09-05 21:08:57 UTC
Could it be this occurs because the changes to the DOM are made first, and only then the setting is saved in the local storage? If so, it may be possible to fix this by saving the preference first.
Comment 3 Chris McMahon 2013-09-09 20:18:11 UTC
A spinner or something while the settings are saved would be useful here I think
Comment 4 Siebrand Mazeland 2013-10-29 14:08:16 UTC
Tracked in mingle at https://mingle.corp.wikimedia.org/projects/internationalization/cards/3814.
Comment 5 Siebrand Mazeland 2013-11-12 13:22:48 UTC
Now tracked at https://mingle.corp.wikimedia.org/projects/internationalization/cards/3822
Comment 6 Siebrand Mazeland 2013-11-12 15:24:53 UTC
The preferred solution here is to ensure that the preference is saved as soon as possible, and that no spinner is needed.
Comment 7 Niklas Laxström 2013-11-12 21:27:55 UTC
I don't understand what your point is. The preference is saved immediately and the time it takes to save the preference for logged in users is dominated mainly by network latency and then server side processing. IMHO optimization of either is not in scope for this bug. We cannot know beforehand whether spinner is needed.
Comment 8 Pau Giner 2013-11-14 09:48:20 UTC
We have been using the spinner to indicate that data will be loaded there. Here, the dialog is about to close and a spinner would probably only add distraction.

I propose to communicate the storing without adding new elements by doing all the following:
- Button label change to ("Applying changes" or "Applying settings")
- Button become disabled
- Cursor changes to "progress"

I made a quick example to visualise it at http://www.youtube.com/watch?v=ac0Bhy0P1I0
Comment 9 Siebrand Mazeland 2013-11-14 10:28:41 UTC
(In reply to comment #8)

> I made a quick example to visualise it at
> http://www.youtube.com/watch?v=ac0Bhy0P1I0

That looks fine, Pau. One possibly drawback is that the button size is changing, which is also a distracting element. This could be circumvented if the button width would initially be that of the widest width needed for any string that might be used in that button.
Comment 10 Gerrit Notification Bot 2013-11-14 12:07:50 UTC
Change 95353 had a related patch set uploaded by Santhosh:
Visual indication while saving the settings

https://gerrit.wikimedia.org/r/95353
Comment 11 Gerrit Notification Bot 2013-11-21 11:43:10 UTC
Change 95353 merged by jenkins-bot:
Visual indication while saving the settings

https://gerrit.wikimedia.org/r/95353

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


Navigation
Links