In T214968#5723307, @Salgo60 wrote:When the popup has been displayed it disappear is there a possibility to display it again.,...??
Description
Description
Related Objects
Related Objects
- Mentioned In
- T214968: Create gadget for feedback after an edit
- Mentioned Here
- T214968: Create gadget for feedback after an edit
Event Timeline
Comment Actions
There's currently no way of showing the popup again. I think it could be tricky to find a good way to add a button that is at the same time not in the way of other GUI components and makes it clear what it does. An alternative could be to not have the popup disappear automatically if the user interacts with it (click, mouse over etc.)
Comment Actions
Thanks or maybe if possible access it from the version list....
My slow brain reacts when the popup has disappeared and would like to look at it again ;-)
Lesson learned is if you see be prepared to read it ;-)
Comment Actions
@Sebastian_Berlin-WMSE It's possible to make the script add a portlet link to the toolbox, and have that link .show() the dialog again. The portlet link is added when the dialog closes (both when the user closes it as well as from the setTimeout). This guarantees that there will always be a dialog to show when clicking this link. It also guarantees that the portlet link is only shown after the user has made an edit.
Proposed diff: https://sv.wikipedia.org/w/index.php?oldid=46879235&diff=46881200
If you think that this is a good idea, I can implement this.
Comment Actions
Looks mostly good. A few things:
- Adding and removing the portlet link seems like odd behaviour to me. If nothing else it shifts the toolbar around. I'd have the link added at load and just not do anything if the popup is visible.
- If you show the popup after it's been hidden it won't go away by itself. This is reasonable, but it means that the user needs to click the "More" button in order to get the "Close" button. I'd add the "Close" button in the non-expanded view.
- You don't need to use find() in close(). $( '#gadget-articleQuality a' ) gets you the link.
Comment Actions
@Sebastian_Berlin-WMSE @Nirmos is this an implementation that still looks reasonable or has the need changed since this was first discussed?
Comment Actions
I think it is still reasonable and an improvement. I haven't seen any followup on my review.