Page MenuHomePhabricator

[Regression] Exiting edit collection description loses changes
Closed, ResolvedPublic


Go to one of your collections listed on Special:Gather that is not your watchlist and click edit.

Inside the edit screen go to edit the description (cog icon)
Type text in title/description
Click the back icon in the top left

Actual: You lose the changes - clicking the cog again the things you inputted are lost
Expected: it should warn you about losing save changes using a confirm prompt. When the user says no they are returned to the editor screen

Note: this is a good opportunity to generalise the code in EditOverlay and TalkOverlay

Event Timeline

Jdlrobson raised the priority of this task from to Needs Triage.
Jdlrobson updated the task description. (Show Details)
Jdlrobson added a project: Gather.
Jdlrobson moved this task to Needs triage on the Gather board.
Jdlrobson added a subscriber: Jdlrobson.

@rmoen do you think this is minor enough to pull into this sprint and fix/enhance?

In theory this should be relatively simple but I am not sure about urgency.

Jhernandez renamed this task from Exiting edit collection description loses changes to [Regression] Exiting edit collection description loses changes.Jul 8 2015, 10:13 AM
Jhernandez moved this task from Incoming to 2016-17 Q2 on the Readers-Web-Backlog board.

Jon, please edit the task description so that it's clear what the student has to do. Currently the instructions only contain the steps to reproduce the bug, hence the student might think the task is reproducing the bug.

This comment was removed by Haritha28.

Change 281248 had a related patch set uploaded (by Haritha28):
Exiting edit collection loses changes(WIP)

I've given some feedback on the patch!

This comment was removed by Haritha28.

@Jdlrobson : I have uploaded a new patch which seems to work under the conditions you have said. Please have look at it and say if any more changes have to be made.

@Jdlrobson : Hello! Could you review the new patch as per your comments.

@Jdlrobson : I have uploaded a new patch set, as you have said along with the changes suggested by Joaquin.

Change 281248 merged by jenkins-bot:
Exiting edit collection loses changes