[#7904] - [UX] Module menu assignment improvement
- Closed
- 3 Oct 2015
- Medium
- Build: staging
- Easy Test
- # 7904
- Diff
- Devportobello:UXMenuAssignement
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Summary
When you create/edit module, you can select page where module only show.
All menuitem are listed without knowing if item are published or not and without language if multilang enabled.
Test
- Create many menuitem as parent and many child
- Unpublish some menuitem
- Create custom module and goto menu assignment, select "Only on the pages selected"
- Unable to know if menuitem published or not
- Set many menuitem in different language
- Enable plugin "System - Language Filter "
- Goto module, all menuitem dont include langcode or flag
- Apply patch
- Now you can know if menuitem is unpublished and language
Preview
Version 1
Version 2
Version 3 - actual commit
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Administration UI/UX |
| Easy | No | ⇒ | Yes |
| Title |
|
||||||
| Title |
|
||||||
Works as intended, but would suggest maybe adding something to clarify what the red 'x' means - I have a hunch people won't know that means the menu item is unpublished.
Maybe something like this?
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7904.
Confirmed that it works as intended.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7904.
| Status | Pending | ⇒ | Ready to Commit |
RTC. Thanks.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7904.
| Labels |
Added:
?
|
||
We could easily get a tooltip at no big cost for the unpublished icon if judged necessary.
Code would be:
<?php if ($link->published == 0) : ?><span title="<?php echo JText::_('JUNPUBLISHED'); ?>" class="icon icon-unpublish hasTooltip"></span><?php endif;?>
Or we could use a label after the menu title instead of the icon similar to what we do in frontend for unpublished items. <span class="label><?php echo JText::_('JUNPUBLISHED'); ?></span>
I agree the icon alone would be not clear enough.
Should the 'delete' icon be used as an 'unpublish' icon? Unpublish is reversible and, normally, delete isn't so it could cause confusion.
Should the 'delete' icon be used as an 'unpublish' icon? Unpublish is reversible and, normally, delete isn't so it could cause confusion.
It's the icon that is on every "unpublish" button in backend and in the state icon in the backend lists indicating unpublished items. The delete buttons have the trash icon. So it is the correct icon. I'm just not sure if it's self-explanatory in the context of that view.
I agree, "red X" doesnt really clear.
Updated, like frontend.
Perhaps just changing the color of the text or a strike would be better with a tooltip?
Sent from my iPhone
On Sep 21, 2015, at 3:18 AM, Devportobello notifications@github.com wrote:
I agree, "red X" doesnt really clear.
Updated, like frontend.—
Reply to this email directly or view it on GitHub.
I have thought:
Can do all additional labels After the Menu title?
because in the big menu it already to much work for Eyes, and if start placing the icons/labels on the front of the title then it can be a "hard" appearance.
Bad:
- Title1
- * Title2
- Title3
- * Title4
Good:
- Title1
- Title2 *
- Title3
- Title4 *
Then it can be simple text label (after the lang flag) instead of icon.
Updated.
Added each screen per version.
Last iteration looks best to me. Thanks!
| Milestone |
Added: |
||
| Milestone |
Added: |
||
rdeutz
- | Milestone |
Added: |
||
| Milestone | |||
| Labels |
Added:
?
|
||
roland-d
- | Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-10-03 16:34:53 |
| Closed_By | ⇒ | roland-d |
| Labels |
Removed:
?
|
||
Nice. Tested OK here.