[MGNLCDEP-7] Deleting a node may issue a very long and unusable alert message with hundreds of dependencies. Created: 17/Dec/10  Updated: 19/Jan/11  Resolved: 21/Dec/10

Status: Closed
Project: Content Dependencies
Component/s: None
Affects Version/s: 1.0.1
Fix Version/s: 1.0.2

Type: Bug Priority: Major
Reporter: Federico Grilli Assignee: Federico Grilli
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File unusable-long-alert.png     PNG File warning-deps-redone.png     PNG File warning-deps.png    
Issue Links:
duplicate
is duplicated by MGNLCDEP-12 Delete node dialog cannot display all... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:

 Description   

This can be reproduced on the demo-project by deleting the /demo-project/news-and-events page. See also attached screenshot. Often the OK and Cancel buttons stay hidden and completely unusable. We need a different way to show deps to the user.



 Comments   
Comment by Boris Kraft [ 17/Dec/10 ]

I don't think this deserves status "critical". The use case with the demo project is probably rare (if a page has so many dependencies, you probably are aware of that before you delete it, no?) . I agree that there should be a better way if there are too many dependencies.

Comment by Federico Grilli [ 17/Dec/10 ]

Agree. A "major" status perhaps suits it better.

Comment by Federico Grilli [ 20/Dec/10 ]

This is what I was able to achieve using jQuery and plugin called jquery-impromptu to replace the default js alert message.
I am aware that the warning style needs improvement (not to say that it sucks altogether), therefore I'd need opinions and help from our usability experts

Comment by Philipp Bärfuss [ 21/Dec/10 ]

It is critical in the sense that if you run into this situation you won't see the buttons and you miss the bigger part of the message. If a user is not used to use the ESC key in such situations he won't be able to get out of this situation anymore.

Alert message: why are we not just abbreviating the list with something like: .. and 20 mores

As soon we have the reference list in the dialog (MGNLCDEP-8) this should be sufficient.

Comment by Federico Grilli [ 21/Dec/10 ]

Eventually we've chosen not to use the jQuery plugin as it would be inconsistent with the way Magnolia UI currently shows alerts and messages in general (mostly js alerts). So Philipp's proposal was implemented. Here's the result.

Generated at Mon Feb 12 00:11:38 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.