Maniphest T19496

Automatically hide "Your preferences have been saved." message when changing preferences
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	• bzimport
	Feb 14 2009, 4:20 PM

Description

Author: sylvain.brunerie

Description:

Version: unspecified
Severity: enhancement
URL: http://en.wikipedia.org/wiki/Special:Preferences
See Also: T67984: Preferences saved notification should use overlay style toaster notification rather than altering the page

Details

Reference: bz17496

	Subject	Repo	Branch	Lines +/-
	Use notification on successful preferences save	mediawiki/core	master	+30 -2
	Preferences: Fix issues from 6d3c65b5b0	mediawiki/core	master	+24 -17

Customize query in gerrit

Related Objects
Search...

Status	Assigned	Task
Open	None	T64559 Redesign Special:Preferences (tracking)
Open	None	T33882 Ensure consistency of how special pages [et al.] switches can be saved
Resolved	Sn1per	T19496 Automatically hide "Your preferences have been saved." message when changing preferences

Event Timeline

• bzimport raised the priority of this task from to Low.Nov 21 2014, 10:29 PM

• bzimport added projects: MediaWiki-Core-Preferences, good first task.

• bzimport set Reference to bz17496.

• bzimport created this task.Feb 14 2009, 4:20 PM

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).

sylvain.brunerie wrote:

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)

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 :)

sylvain.brunerie wrote:

Giving this to Andrew, low priority.

ayg wrote:

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.

I'd hide it whenever a preference is changed.

ayg wrote:

That would be sensible.

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.

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?

(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.

Created attachment 14325
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.

Attached:

fix-successbox.patch1 KBDownload

Submitted a patch successfully here
https://gerrit.wikimedia.org/r/#/c/107837/

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

(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.

Attached:

fix-successbox.patch1 KBDownload

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.

• werdna unsubscribed.Dec 10 2014, 6:16 PM

Can we add it as a GCI task? The previous patch is outdated.

• demon unsubscribed.Dec 10 2014, 6:22 PM

CCing @Aklapper just in case we still have time for GCi.

Qgil removed Kunalgrover05 as the assignee of this task.Jan 12 2015, 12:21 PM

Qgil set Security to None.

@01tonythomas, @Kunalgrover05: Would you mentor such a GCi task? If yes feel free to create a task in Google Melange, or I can do that.

I think polishing https://gerrit.wikimedia.org/r/#/c/107837/ a bit would do the same. Will create the GCI task.

He7d3r renamed this task from JS: Automatically hide "Your preferences have been saved." message when changing preferences to Automatically hide "Your preferences have been saved." message when changing preferences.Jan 12 2015, 10:06 PM

He7d3r updated the task description. (Show Details)

He7d3r added a project: JavaScript.

Change 185125 had a related patch set uploaded (by Sn1per):
Hide saved prefs message on change

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

Patch-For-Review

The change by @Sn1per use the postedit box to show the success message. It fades out on switching tabs and looks good. Can someone do a quick review, as the GCI ends in 24 hours ?
Attaching a quick preview

I have changed the savedprefs message from a postEdit box to a mw.notification because of feedback. The notification box is more consistent with other settings changes e.g. add to watch list. The disappearance on settings/tab change functionality is unchanged. Here is a screenshot :-)