[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: |
|
||||||||
| Issue Links: |
|
||||||||
| 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. |
| 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 ( |
| 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. |