Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Updated: Comment #0
Problem/Motivation
The help text for Menu item description is not that clear. Currently it reads 'Shown when hovering over the menu link.'
Proposed resolution
Update the help text to be more descriptive. This help text is derived from the node hierarchy module:
'Shown when hovering over the menu item. Hold your mouse over this link for a demonstration.'
Comment | File | Size | Author |
---|---|---|---|
#23 | interdiff-2079143-19-22-D7.txt | 3.49 KB | joyceg |
#23 | 2079143-22-D7.patch | 3.26 KB | joyceg |
#22 | 2079143-22.patch | 3.26 KB | joyceg |
#22 | interdiff-2079143-19-22.txt | 3.49 KB | joyceg |
#21 | interdiff-2079143-18-21.txt | 3.38 KB | joyceg |
Comments
Comment #1
pameeela CreditAttribution: pameeela commentedPatch attached.
Comment #2
larowlan+1 from me, but tagging for the UX team, can we get a screenshot.
Comment #3
pameeela CreditAttribution: pameeela commentedNew patch attached, updates both refs (node and menu page). Also screenshots.
Node page: /node/add/article
Menu page: /admin/structure/menu/manage/main/add
Comment #4
pameeela CreditAttribution: pameeela commentedOops, tag added back.
Comment #5
Bojhan CreditAttribution: Bojhan commentedI am not sure this is necessary, I have never seen a user get confused by this?
Comment #6
pameeela CreditAttribution: pameeela commentedI don't think it is a major problem, but I think it is an improvement anyway. The confusion I have seen is around the hover - 'Shown when hovering over the menu item' could be wrongly interpreted to mean the description will replace the menu title text on hover along with the hover style change of the link changing colour, for example.
So I thought providing a contextual demo of the title text behaviour would likely resolve any possible misunderstandings.
Comment #7
lostkangaroo CreditAttribution: lostkangaroo commentedupdating the title to be a touch more descriptive
Comment #8
pameeela CreditAttribution: pameeela commentedThanks @lostkangaroo - what are your thoughts on the patch?
Comment #9
lostkangaroo CreditAttribution: lostkangaroo commentedDunno really but I can see where this would help clarify something a person unfamiliar with the sublities of link attributes might not understand. Its not harmful so why not. A few more comments from UX experts would be nice.
Comment #9.0
lostkangaroo CreditAttribution: lostkangaroo commentedChange a to the to match patch
Comment #12
er.pushpinderrana CreditAttribution: er.pushpinderrana commentedUnable to apply above patch. Please review updated patch.
Comment #13
joyceg CreditAttribution: joyceg commentedThis needs to be implemented in Drupal 7.x too.
Comment #14
joyceg CreditAttribution: joyceg commented@er.pushpinderrana , not able to apply this patch.
Comment #15
joyceg CreditAttribution: joyceg commentedWorking on this issue.
Comment #16
joyceg CreditAttribution: joyceg commentedAdding the patch and screenshot of the modified description .
Patch applied successfully.
Comment #18
joyceg CreditAttribution: joyceg commentedSome issues with the previous patch.
Updated the patch.
Comment #19
joyceg CreditAttribution: joyceg commentedAdding patch for D7.x
Comment #20
therealssj CreditAttribution: therealssj commentedI might be nit picking but 'Hold your cursor' would be better than 'Hold your mouse'.
Comment #21
joyceg CreditAttribution: joyceg commentedComment #22
joyceg CreditAttribution: joyceg commentedImproved description.
Comment #23
joyceg CreditAttribution: joyceg commentedComment #24
joyceg CreditAttribution: joyceg commented#22, the patch was uploaded with D8 tests, it was a D7 patch.
It has been rectified in #23.
So, D7: #23
D8: #21
Comment #27
therealssj CreditAttribution: therealssj commentedComment #28
Ramya Bala CreditAttribution: Ramya Bala commentedI have applied this patch and tested .It is working perfectly.
Comment #29
joyceg CreditAttribution: joyceg commentedComment #30
alexpottAs this as a string change and we're in 8.1.x release candidate period moving to 8.2.x
Comment #31
catchSame question as Bojhan in #5 - not sure if we're adding extra unnecessary text here, so tagging for usability review since that comment was nearly 3 years ago.
Also the patch file has the patch in it.
Comment #32
yoroy CreditAttribution: yoroy at Roy Scholten commentedI don't think it is necessary to add this. We should not fix things based on "might be confusing for the user".
Then again, there is a way to add this without adding more text:
Shown when hovering over the menu link.