[MGNLUI-2598] Rename/delete actions in favourites are difficult to locate Created: 17/Jan/14 Updated: 19/Jun/15 Resolved: 04/Jun/14 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | favorites |
| Affects Version/s: | 5.0.1, 5.2.1 |
| Fix Version/s: | 5.3 |
| Type: | Bug | Priority: | Major |
| Reporter: | Rainer Blumenthal | Assignee: | Christoph Meier |
| Resolution: | Fixed | Votes: | 2 |
| Labels: | aperto, next, support, usability, ux | ||
| 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
|
||||||||||||||||||||
| Release notes required: |
Yes
|
||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||
| Description |
|
You have to click on the symbol ... to get the possibility to rename / delete. We couldn't figure out how this works with 3 guys, working with Magnolia since many years. The usability is inacceptable. We found out how by accident, after 10 minutes. |
| Comments |
| Comment by Andreas Weder [ 23/Jan/14 ] |
|
I agree with you that the discoverability of the edit/delete actions is very bad currently. In fact, the Favorites UI has other short comings as well (it works differently from other UIs; DnD is unsatisfying; no other way to rearrange favorites than through DnD; the UI still doesn't match the visual design). There was a set of reasons that forced us to compromise on usability back when we implemented it. We eventually decided to make favorites easy and fast to use (just a click), and sacrificed making them easy to edit. Which means: the "edit"/"delete" features are there, but you have to learn or be taught how to access them. However, we do want to change that. My current plan is to make Favorites a focused effort (not before 5.3) and then also add some of the requested features at the same time (e.g. group favorites). Obviously, this would be a bigger effort, and that doesn't help you in the short or middle term. If we're looking for intermediate solutions, two options I see that also employ the means we currently have, would be:
I like "edit favorites" best, as it is very explicit, is in-line with the design and works everywhere. I'd have to give it some more thought, but I expect this to work. I would not want to show tools on hover, since that doesn't work on tablets and makes the interface noisy when moving the mouse (a short delay might help). What do you think? And how urgent do you think this issue has to be fixed? |
| Comment by Christoph Meier [ 22/May/14 ] |
|
We will implement the following:
|
| Comment by Christoph Meier [ 23/May/14 ] |
|
Functionality implemented and committed to onto branch " |
| Comment by Christoph Meier [ 26/May/14 ] |
|
TODO: Grey out the functionality "Edit favorites" if there is no favorite, yet. |
| Comment by Christoph Meier [ 30/May/14 ] |
|
Note: We want to have the favorites-app every time when (re-)appearing to be in its initial state - that means, all icons should be hidden, every field in non-editable mode. Concerning UI-Tests: info.magnolia.integrationtests.uitest.FavoriteUITest#ensureOnlyOneFavoriteIsSelected has been removed, since setting an item selected is no more possible (not necessesary) with the new approach to show/hide the delete- and edit-icons of all the items when clicking on the appropriate link/icon of the header-bar of the favorites-form. |
| Comment by Christoph Meier [ 30/May/14 ] |
|
Commits: m_ui ; on branch "
ce-bundle (for the UI-tests); on branch " |
| Comment by Christopher Zimmermann [ 03/Jun/14 ] |
|
Reopen:
|
| Comment by Christoph Meier [ 04/Jun/14 ] |
|
Re-resolved:
Impl. of FavoritesEntry#createItemdId i did not changed; to use jcr-item-uuid (if existing) makes sense, and usage of date+counter is safe (and if it's more than safe, it doesn't hurt) |
| Comment by Christopher Zimmermann [ 04/Jun/14 ] |
|
Coolio! Maybe this style??: 2. Please remove comment FavoritesPresenter#initializeViews |