Problem/Motivation
The UI standard page for Contextual links needs to be updated: https://www.drupal.org/docs/develop/user-interface-standards/contextual-...
Proposed resolution
Rewrite the page and add missing screenshots.
Check whether core modules follow this standard.
Remaining tasks
| Task | Novice task? | Contributor instructions | Complete? |
|---|---|---|---|
| No | Change the order of the text to first say how button should be used, then why, and then follow it up by a good example. | Yes | |
| Yes | Add screenshots if needed. | Yes | |
| Yes | Add a one sentences summary that's useful here. | Yes | |
| Yes | Add the correct link for the Drupal 8 API. | Yes | |
| Double check | Yes | Check several modules whether they use the contextual links this way. Add in a comment which pages you checked. | No |
| Double check issues | Yes | If you find a module that does not follow this standard: make an issue to fix this. | Optional |
Comments
Comment #2
ivan berezhnov commentedComment #4
joshmillerComment #5
msankhala commented@Ivan What is this CSKYiv18 tag used for. I see you add this tag to many issues. :)
Comment #6
gramie commentedI'm working on this at DrupalCon Nashville2018 in the Mentored Sprint. If anyone wants to talk about this, come and talk to me!
Comment #7
gramie commentedChanged the layout to follow standard of other documentation (how-why-examples)
Added link to Drupal8 API
The recommendation says to avoid using a single task, but this is very common on a standard Drupal installation (for example, by the blocks and views modules), so maybe this should be softened or explained more.
Comment #8
sk33lz commentedThe screenshot used on this page is a good visual reference, but is low quality and looks blurry. The most recent updates by @gramie look great, but we should update the screenshot before we call this done for better accessibility.
Comment #9
SharraRosichan commented+Nashville2018 Working on this at a spring - fixed Vertical Tabs.
Comment #10
gramie commentedI've changed the screenshot to one from a standard Drupal install. Hope it's clearer.
Comment #11
joshmillerOne remaining task to do.
Comment #12
joshmillerComment #23
quietone commentedThe remaining task here is "Check several modules whether they use the contextual links this way. Add in a comment which pages you checked.' Unfortunately, that is not enough detail for me. I did test the contextual links on a fresh install of Drupal 11.x today. I checked all the contextual links on the front page and the one node I created. They all appeared correct to me.
There is at least one issue to fix the display of contextual links, #2983655: Contextual links are not displayed correctly. So, I think it is safe to close this issue as fixed. I am sure someone will correct that if it is wrong.