Make Tour link more visible

Created on 2 December 2020, about 4 years ago
Updated 20 May 2024, 7 months ago

Problem/Motivation

The Tour link in the upper right corner is easily missed. I never noticed it until today, where I had to find it. And even when actively looking for it, I had to search the text on the page for "Tour", to discover it.

This is sad, because it really is a great feature.

Steps to reproduce

Activate the Tour module and visit for example /admin/structure/views/view/frontpage.

Proposed resolution

Make the link stand out more, by extending the text, and making the background blue.

Remaining tasks

Update the code.

User interface changes

Tour link text extended, and with blue background.

API changes

None

Proposed solution

Currently

Tour link, more visible

๐Ÿ“Œ Task
Status

Fixed

Version

2.0

Component
Tourย  โ†’

Last updated 13 days ago

Created by

๐Ÿ‡ฉ๐Ÿ‡ฐDenmark ressa Copenhagen

Live updates comments and jobs are added and updated live.
  • Needs usability review

    Used to alert the usability topic maintainer(s) that an issue significantly affects (or has the potential to affect) the usability of Drupal, and their signoff is needed. When adding this tag, make it easy to review the issue. Make sure the issue summary describes the problem and the proposed solution. Screenshots usually help a lot! To get sign-off on issues with the "Needs usability review" tag, post about them in the #ux channel on Drupal Slack, and/or attend a UX meeting to demo the patch and get direct feedback from designers/UX folks/product management on next steps. If an issue represents a significant new feature, UI change, or change to the general "user experience" of Drupal, use Needs product manager review instead. See the scope of responsibilities for product managers.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • ๐Ÿ‡ฉ๐Ÿ‡ฐDenmark ressa Copenhagen

    I just installed the Honeypot module, and by chance (I forget where) I saw that it offered a Tour. It made me remember that the Tour link is hard to find, and it took me some time to scan the entire screen, when I finally found the "Tour" link, hidden in the upper right corner. It's so sad that it sits hidden up there ...

  • Status changed to Needs review almost 2 years ago
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia gauravvvv Delhi, India

    I have attached patch for 10.1.x. please review

    Patch #18, no longer applies

     git apply -v 161.patch
    Checking patch core/modules/tour/tour.module...
    Checking patch core/themes/claro/css/theme/toolbar.icons.theme.css...
    Checking patch core/themes/claro/css/theme/toolbar.icons.theme.pcss.css...
    Checking patch core/modules/contextual/css/contextual.toolbar.css...
    error: while searching for:
    }
    .toolbar .toolbar-bar .contextual-toolbar-tab .toolbar-item {
      margin: 0;
    }
    .toolbar .toolbar-bar .contextual-toolbar-tab .toolbar-item.is-active {
      background-image: -webkit-linear-gradient(rgb(78, 159, 234) 0%, rgb(69, 132, 221) 100%);
    
    error: patch failed: core/modules/contextual/css/contextual.toolbar.css:12
    error: core/modules/contextual/css/contextual.toolbar.css: patch does not apply
  • ๐Ÿ‡ฉ๐Ÿ‡ฐDenmark ressa Copenhagen

    Thanks @Gauravvvv, I updated the existing merge request target branch from 9.2 to 10.1, and rebased it by clicking "Rebase without pipeline":

    To rebase a merge requestโ€™s branch without triggering a CI/CD pipeline, select Rebase without pipeline from the merge request reports section. This option is available when fast-forward merge is not possible but a conflict-free rebase is possible.

    So the current MR !161 should now work with Drupal 10.1.

  • Status changed to Needs work almost 2 years ago
  • ๐Ÿ‡ฉ๐Ÿ‡ฐDenmark ressa Copenhagen

    Perhaps we can now look at the review by @himanshu_sindhwani in #17 ๐Ÿ“Œ Make Tour link more visible Needs work , about not removing the transparency?

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States smustgrave

    Just FYI ๐ŸŒฑ [Policy] Remove tour module from core Fixed

  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia dsandhya

    plz review the path

  • ๐Ÿ‡ณ๐Ÿ‡ฟNew Zealand quietone
  • Status changed to Fixed 8 months ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States smustgrave

    Included with new 2.0.x branch.

  • ๐Ÿ‡ฉ๐Ÿ‡ฐDenmark ressa Copenhagen

    Thanks for taking over the maintenance of the Tour module @smustgrave, I really appreciate it.

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States smustgrave

    No problem! Really hoping to bring some new life to the module. Probably should start a 2.0.0 meta soon

  • ๐Ÿ‡ฉ๐Ÿ‡ฐDenmark ressa Copenhagen

    Cool, I am looking forward to version 2, and trying it out! Have a nice day.

  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024