Last modified: 2014-06-01 11:11:20 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 T19496, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 17496 - JS: Automatically hide "Your preferences have been saved." message when changing preferences
JS: Automatically hide "Your preferences have been saved." message when chang...
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
User preferences (Other open bugs)
unspecified
All All
: Low enhancement (vote)
: ---
Assigned To: kunalgrover05
http://en.wikipedia.org/wiki/Special:...
: easy
Depends on:
Blocks: 31882
  Show dependency treegraph
 
Reported: 2009-02-14 16:20 UTC by sylvain.brunerie
Modified: 2014-06-01 11:11 UTC (History)
9 users (show)

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


Attachments
Patch to remove success box on changing any preference or switching tab (1.98 KB, patch)
2014-01-16 07:13 UTC, kunalgrover05
Details

Description sylvain.brunerie 2009-02-14 16:20:40 UTC
 
Comment 1 Chad H. 2009-02-14 16:23:09 UTC
Empty bug report. Suggest WONTFIX based on summary alone. Needs further info from reporter as to why this should be done (I don't see any reason not to confirm that they were saved).
Comment 2 sylvain.brunerie 2009-02-14 16:31:40 UTC
After saving preferences, a message is displayed that says "Your preferences
have been saved.". But if I keep doing changes to my preferences, and I leave
my computer, then when I come back in front of my screen, I don't remember
whether or not my changes have been saved. Some JavaScript code could
automatically hide this message when changing preferences.

(Sorry, I didn't mean to type Enter so early...)
(And sorry for my bad English)
Comment 3 Chad H. 2009-02-14 16:35:18 UTC
Ah ok! That makes a lot more sense. I originally understood this was "Using JS
to hide preferences" always. Didn't see a reason there :)
Comment 4 sylvain.brunerie 2009-02-15 11:48:12 UTC
:)
Comment 5 Chad H. 2009-08-02 16:17:22 UTC
Giving this to Andrew, low priority.
Comment 6 Aryeh Gregor (not reading bugmail, please e-mail directly) 2009-08-02 16:22:08 UTC
I suggest we not make messages mysteriously disappear after a timeout.  That's confusing.  People might not be done reading them, or they might have taken the action, switched to another tab/window, and only come back half an hour later.  No message should disappear after a time period has elapsed, unless maybe it's intrinsically time-sensitive and no longer makes sense.
Comment 7 Andrew Garrett 2009-08-02 16:25:50 UTC
I'd hide it whenever a preference is changed.
Comment 8 Aryeh Gregor (not reading bugmail, please e-mail directly) 2009-08-02 17:15:42 UTC
That would be sensible.
Comment 9 Jared Zimmerman (WMF) 2014-01-14 09:52:36 UTC
If we're going to show and hide this after a timer if prefer it was a floating fixed element rather than an in-line one, A. So the user sees it, and B. So it isn't making the content of the page refactor when it appears and disappears. If reccomend 4 seconds for the time on screen. 

To address the issue of "knowing if it saved" I'd recommend disabling the save button if no changes have been made since last save. I don't think this is absolute needed but it's another reminder that there is nothing you need to do.
Comment 10 kunalgrover05 2014-01-15 13:01:24 UTC
Hi I am new here and wanted to work on this bug.
I implemented it as this-Whenever a user changes the preferences tab or reloads or clicks on any section of the preferences tabs, the message is hidden. Is it a good implementation?
I liked Jared's idea as well. Can anyone suggest how I should implement it?
Comment 11 Tony Thomas 2014-01-15 13:04:07 UTC
(In reply to comment #10)
> Hi I am new here and wanted to work on this bug.
> I implemented it as this-Whenever a user changes the preferences tab or
> reloads
Great. I will assign the same to you.
Comment 12 kunalgrover05 2014-01-16 07:13:26 UTC
Created attachment 14325 [details]
Patch to remove success box on changing any preference or switching tab

I was unable to submit the patch to gerrit directly because of ISP issues.
As suggested by sumanah on IRC, I tried to submit it using the online gerrit uploader but it gave me an error as well.
So I posted the patch here directly.
Can someone please review it and tell if I should modify it.
Comment 13 kunalgrover05 2014-01-16 12:15:56 UTC
Submitted a patch successfully here
https://gerrit.wikimedia.org/r/#/c/107837/
Comment 14 Gerrit Notification Bot 2014-01-16 12:18:02 UTC
Change 107837 had a related patch set uploaded by 01tonythomas:
Patch to remove success box on changing any preference or switching tab

https://gerrit.wikimedia.org/r/107837
Comment 15 Tony Thomas 2014-01-16 12:19:18 UTC
(In reply to comment #12)
> Created attachment 14325 [details]
> Patch to remove success box on changing any preference or switching tab
You should go through 
www.mediawiki.org/wiki/Gerrit/Tutorial
next time to submit a patch using git-review.
Comment 16 kunalgrover05 2014-01-16 13:55:39 UTC
Thank you Tony. I was unable to submit a patch because my proxy doesn't allow SSH to Gerrit port. I will try and submit it through git-review next time by some other internet connection.
Comment 17 kunalgrover05 2014-01-16 13:56:13 UTC
Thank you Tony. I was unable to submit a patch because my proxy doesn't allow SSH to Gerrit port. I will try and submit it through git-review next time by some other internet connection.

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


Navigation
Links