Part of #1921152: META: Start providing tour tips for other core modules.

Problem/Motivation

Write tour integration for Menu admin page

Proposed resolution

Create tour yml files for required admin screens in admin/structure/menu.

Remaining tasks

Create patch for tour integration for primary Structure page

User interface changes

New tours

API changes

None

Technical pointers when creating tour tips

See: https://drupal.org/node/1921152#tour-tips-tech-note for tech notes on making tour tips.

CommentFileSizeAuthor
#77 interdiff_76-77.txt1.51 KBridhimaabrol24
#77 2040823-77.patch3.86 KBridhimaabrol24
#76 interdiff_74-76.txt807 bytesridhimaabrol24
#76 2040823-76.patch4.17 KBridhimaabrol24
#74 interdiff_68-74.txt4.06 KBridhimaabrol24
#74 2040823-74.patch4.11 KBridhimaabrol24
#72 interdiff_68-72.txt2.66 KBridhimaabrol24
#72 2040823-72.patch2.99 KBridhimaabrol24
#68 2040823-68.patch2.77 KBowenpm3
#66 2040823-66.patch2.77 KBravi.shankar
#64 interdiff_55-64.txt2.2 KBvacho
#64 TourForMenuAdmin-2040823-64.patch2.77 KBvacho
#55 TourForMenuAdmin-2040823-55.patch2.64 KBElectric Doorknob
#43 interdiff-2040823-41-43.txt553 bytesjorgegc
#43 tour-menu-2040823-43.patch2.03 KBjorgegc
#41 interdiff-2040823-39-41.txt3.54 KBjorgegc
#41 tour-menu-2040823-41.patch2.17 KBjorgegc
#39 tour-menu-2040823-39.patch2.68 KBjorgegc
#31 2040823-menu-tour-30.patch2.73 KBpameeela
#29 interdiff.txt1.55 KBpameeela
#29 2040823-menu-tour-29.patch2.53 KBpameeela
#23 2040823-23-tour-menu-page.interdiff.txt1.29 KBnick_schuch
#23 2040823-23-tour-menu-page.patch2.51 KBnick_schuch
#21 2040823-21-tour-menu-page.patch2.5 KBnick_schuch
#18 2040823-18-tour-menu-page.interdiff.txt2.22 KBnick_schuch
#18 2040823-18-tour-menu-page.patch2.58 KBnick_schuch
#15 2040823-15-tour-menu-page.patch2.56 KBpameeela
#13 interdiff-2040823-12-13.txt1.48 KBpameeela
#13 2040823-13-tour-menu-page.patch1.25 KBpameeela
#12 2040823-12-tour-menu-page.patch2.56 KBfloydm
#12 interdiff-2040823-11-12.txt454 bytesfloydm
#11 2040823-11-tour-menu-page.patch2.57 KBrli
#9 core-tour-2040823-9-menu-yml.patch1.25 KBclemens.tolboom
#4 tour-menu-2040823-4.patch1.19 KBnickgs
#1 tour-menu-2040823-1.patch1.21 KBnickgs

Issue fork drupal-2040823

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nickgs’s picture

FileSize
1.21 KB

First time core patch here... figured I could give this a go.

I think these tours, if used right, can provide a great experience for new Drupal users!

I hope this is helpful.

Thanks.

Nick

nickgs’s picture

Status: Active » Needs review

Not sure why my patch didn't get submitted for testing but switching this to needs review.

Thanks.

Nick

larowlan’s picture

Thanks so much for working on this - these tours are starting to take shape!

+++ b/core/modules/menu/config/tour.tour.menu.ymlundefined
@@ -0,0 +1,29 @@
+    body: 'Collection of tools for use with a particular menu. Rearrange menu link ordering, and various other configuration attributes. Click dropdown for available actions'

'Use the drop-button to perform operations on each menu including re-arranging menu link order, adding new menu links and changing the menu name'?

+++ b/core/modules/menu/config/tour.tour.menu.ymlundefined
@@ -0,0 +1,29 @@
+    attributes: ¶
...
+    attributes: ¶

Trailing white space

+++ b/core/modules/menu/config/tour.tour.menu.ymlundefined
@@ -0,0 +1,29 @@
+    body: 'Will allow you to create a new menu to store a collection of menu links'

Can we frame this along the lines of 'Use this button to add a new menu to store a collection of menu links'?

+++ b/core/modules/menu/config/tour.tour.menu.ymlundefined
@@ -0,0 +1,29 @@
+    body: 'This page lists the menus available on your site. Menus are a collection of links arranged in a hierarchy. The display of menus is accomplished by its corresponding block which is managed on the block administration page.'

Perhaps 'You can configure the placement of a menu on your site by visiting the Block administration page/'?

nickgs’s picture

FileSize
1.19 KB

Awesome, thanks for the review! I'm having fun learning how the tours work and creating these patches.

I think the changes you suggest make alot of sense. It simplifies the verbiage and makes it a bit more succinct. Question, do we want to include links in tour tips? Specifically for this issue, a link to the block administration page. Being that we have the link already on the page I am not sure we need it.

Attached is a rerolled patch incorporating your suggestions.

Thanks.

Nick

lisarex’s picture

I didn't get a chance to create the patch, (I assigned the issue to myself but I guess I wasn't fast enough ;))

Here's our copy for this page, for comparison

tip 1
(Immediately under the page title)
Title: “Edit a menu link”
Body: “Use this page to edit an item in a menu.”
tip 2
(Immediately under “Parent link”)
Title: “Select a parent”
Body: “Menus can have more than one layer. Select the menu item that is in the layer above this one.”
tip 3
(Immediately under “Weight”)
Title: “Order menu item by weight”
Body: “Within a menu layer, items are listed by weight.”
Tip 4
(Immediately under “Save”)
Title: “Save your changes”
Body: “Click Save to apply your changes to the menu item.”

lisarex’s picture

So, we have written tours for the following:
/admin/structure/menu/
/admin/structure/menu/admin/item/*/edit
/admin/structure/menu/manage/admin

Should this be a multipage tour?

nick_schuch’s picture

Issue tags: +Needs tests, +Tour

Tour needs tests since we now have #2028535

lisarex’s picture

Assigned: lisarex » Unassigned

Going to unassign myself since I don't know how much time I'll have to get back into this, but I will try to make it happen!

clemens.tolboom’s picture

+++ b/core/modules/menu/config/tour.tour.menu.yml
@@ -0,0 +1,29 @@
+      data-class: dropbutton-action

This was not working. The tip did not attach to the button.

Attached patch has step 3 fixed. I'm not sure why the selector was broken. I changed it into dropbutton-widget instead.

I changed the location too as that looks nicer.

As @nick_schuch adds this needs #2028535: Provide a TourTestBase class for use by core and contrib modules but I'm not sure whether that would have caught the selector problem.

The patch was build on http://tour.drutch.nl/ without using git commands. (no coding involved). I really could use some help building that site for the community :-/

rli’s picture

Assigned: Unassigned » rli

working on this.

rli’s picture

Adding the test now.

floydm’s picture

Fixed intro comment in the test that referenced views_ui.

pameeela’s picture

Just made a few small changes. New patch and interdiff attached.

clemens.tolboom’s picture

Status: Needs review » Needs work

Test is missing in #13

pameeela’s picture

Status: Needs work » Needs review
FileSize
2.56 KB

Oops! Updated patch attached (interdiff doesn't change).

Also realised I should have explained the changes. I made the first tip title sentence case to match the others, added a full stop in the second tip, and removed the hyphen from rearranging.

nielsonm’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs tests

Looks good, great job.

batigolix’s picture

Status: Reviewed & tested by the community » Needs work

This doesnt seem ready to me.

a) I think we should not use words like "this button" or "this page". The tour tips will make clear what element is being referred to

b) There should be a tip on the Settings tab, so the user knows what can be found there

c) "Use the drop-button to perform operations on each menu including rearranging menu link order, adding new menu links and changing the menu name." . There are only 2 options in the drop button. There is no option to change the menu. But this can be done via the Edit menu drop down option

d) "Add a new menu" could be better "Add a menu". "New" is redundant here.

e) "You can configure the placement of a menu by visiting the block administration page." --> This is called the Block layout page now (at least at this moment). Can we have a link on there so that the users does not have to search for this page? Maybe it is also good to mention that one should enable a menu block first.

f) Check the menu_help function in menu.module for some additional information that could be used in this tour

nick_schuch’s picture

This patches updates the path to a route and uses the basic class for test coverage.

nick_schuch’s picture

Issue tags: +Tour Content Review

Status: Needs review » Needs work

The last submitted patch, 18: 2040823-18-tour-menu-page.patch, failed testing.

nick_schuch’s picture

Status: Needs work » Needs review
FileSize
2.5 KB

Status: Needs review » Needs work

The last submitted patch, 21: 2040823-21-tour-menu-page.patch, failed testing.

nick_schuch’s picture

Status: Needs work » Needs review
FileSize
2.51 KB
1.29 KB

Here we go. Time for green!

Status: Needs review » Needs work

The last submitted patch, 23: 2040823-23-tour-menu-page.patch, failed testing.

nick_schuch’s picture

The last submitted patch, 23: 2040823-23-tour-menu-page.patch, failed testing.

nick_schuch’s picture

nick_schuch’s picture

Assigned: rli » Unassigned
Status: Needs work » Needs review
pameeela’s picture

FileSize
2.53 KB
1.55 KB

New patch and interdiff. Makes content updates and adds a tip for Settings tab per #17

Status: Needs review » Needs work

The last submitted patch, 29: 2040823-menu-tour-29.patch, failed testing.

pameeela’s picture

FileSize
2.73 KB

Updates test coverage to exclude nth-child element which is unsupported. Should sort the fail.

pameeela’s picture

Status: Needs work » Needs review
bendev’s picture

Hello,

I would like to start contributing to D8 and I found multilingual to be a nice one to start with.

I have installed a D8 and started to configure it for FR and EN.

The translation of content type is ok. I have some problems to translate the menu items. I arrived to this post. I would like to see the "Tour for menu admin" so that I can test it and see if it is clear an apply it. The tour module is enabled. Should I install this patch to be able to start (and actually see the tour" ?

thanks

rodrigoaguilera’s picture

Status: Needs review » Needs work
Issue tags: +Novice

@bendev
Yes, you can apply this patch and follow the tour

the test need annotation to be detected as such instead of getInfo() and at least a setUp function and some actual tests.
Is as simple as the following example

  /**
   * {@inheritdoc}
   */
  protected function setUp() {
    parent::setUp();
    $this->adminUser = $this->drupalCreateUser(array('administer content translation', 'access tour'));
    $this->drupalLogin($this->adminUser);
  }

  /**
   * Tests Content translation tour tip availability.
   */
  public function testContentTranslationTour() {
    $this->drupalGet('admin/config/regional/content-language');
    $this->assertTourTips();
  }

marking also as novice

jorgegc’s picture

I am working on the changes guys.

jorgegc’s picture

Status: Needs work » Needs review
FileSize
2.68 KB

Uploading patch for review.

benjy’s picture

Status: Needs review » Needs work
+++ b/core/modules/menu_ui/src/Tests/MenuTourTest.php
@@ -0,0 +1,71 @@
+  /**
+   * {@inheritdoc}
+   */
+  protected function setUp() {
+    parent::setUp();
+    $this->adminUser = $this->drupalCreateUser(array('administer menu', 'access tour'));
+    $this->drupalLogin($this->adminUser);
+  }
+
+  /**
+   * Tests menu tour tip availability.
+   */
+  public function testMenuOverviewTour() {
+    $this->drupalGet('admin/structure/menu');
+    $this->assertTourTips();
+  }

None of this is needed if $tips and $permissions are configured it all happens in the base class.

jorgegc’s picture

Status: Needs work » Needs review
FileSize
2.17 KB
3.54 KB

Made changes as per suggestions in #40 and also renamed the yaml file to replace the underscore with a hyphen.

benjy’s picture

+++ b/core/modules/menu_ui/src/Tests/MenuTourTest.php
@@ -0,0 +1,49 @@
+  /**
+   * An admin user with administrative permissions.
+   *
+   * @var \Drupal\user\UserInterface
+   */
+  protected $adminUser;

Don't think we need this.

jorgegc’s picture

Nice catch! Updated patch and interdiff.

benjy’s picture

Status: Needs review » Reviewed & tested by the community

Looks good.

webchick’s picture

Status: Reviewed & tested by the community » Postponed

So pursuant to #1921152-109: META: Start providing tour tips for other core modules., I want to make sure we have a plan/strategy in place before going too nuts on these Tours. Marking postponed as a result.

However, I will say that I think this area would definitely benefit from a Tour, especially deeper into the menu interface. "What is show as expanded?" etc. So we likely will end up just expanding this one a bit to cover more of the menu admin interface.

benjy’s picture

It's a bit of shame that @jorgegc spent most of his day on this issue if there was a known disagreement with them moving forward.

We should probably postpone all child issues to stop anymore wasted effort.

webchick’s picture

Agreed. :( I spoke to him about it in person. Very sorry, I just didn't feel comfortable committing one of these when there doesn't seem to be alignment between the docs/UX team on what we should be doing here.

I'm back at my apartment now; I'll go postpone the other issues to make sure the dependencies are clear.

mgifford’s picture

Version: 8.0.x-dev » 8.1.x-dev

Drupal 8.0.6 was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 is now available and sites should prepare to update to 8.1.0.

Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.1.x-dev » 8.2.x-dev

Drupal 8.1.9 was released on September 7 and is the final bugfix release for the Drupal 8.1.x series. Drupal 8.1.x will not receive any further development aside from security fixes. Drupal 8.2.0-rc1 is now available and sites should prepare to upgrade to 8.2.0.

Bug reports should be targeted against the 8.2.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.2.x-dev » 8.3.x-dev

Drupal 8.2.6 was released on February 1, 2017 and is the final full bugfix release for the Drupal 8.2.x series. Drupal 8.2.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.3.0 on April 5, 2017. (Drupal 8.3.0-alpha1 is available for testing.)

Bug reports should be targeted against the 8.3.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.4.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.3.x-dev » 8.4.x-dev

Drupal 8.3.6 was released on August 2, 2017 and is the final full bugfix release for the Drupal 8.3.x series. Drupal 8.3.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.4.0 on October 4, 2017. (Drupal 8.4.0-alpha1 is available for testing.)

Bug reports should be targeted against the 8.4.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.5.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.4.x-dev » 8.5.x-dev

Drupal 8.4.4 was released on January 3, 2018 and is the final full bugfix release for the Drupal 8.4.x series. Drupal 8.4.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.5.0 on March 7, 2018. (Drupal 8.5.0-alpha1 is available for testing.)

Bug reports should be targeted against the 8.5.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.6.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Electric Doorknob’s picture

Version: 8.5.x-dev » 8.6.x-dev
Assigned: Unassigned » Electric Doorknob
Status: Postponed » Active

I'm picking this up as part of our #Nashville2018 sprint.

Electric Doorknob’s picture

This patch builds on the previous patch for this issue, and attempts to use the tour to contextualize the purpose of the Menu Admin page for new site admins, give an example of its use, and lead toward the Add and Edit menu forms.

Further work is required, especially on the Menu Edit page and Link Add/Edit pages.

Electric Doorknob’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 55: TourForMenuAdmin-2040823-55.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

jhodgdon’s picture

Thanks for the patch! As I noted on #1921152-146: META: Start providing tour tips for other core modules., the new tour needs to go into the config/optional directory under menu_ui, not config/install.

Electric Doorknob’s picture

Status: Needs work » Active

Thanks for the feedback! I'll reroll this with some additional related Menu module tour items that I noted in 56

benjifisher’s picture

Status: Active » Needs work

Generally, the "Active" state means that there is not yet a patch on the issue. I think this issue should be marked NW.

Version: 8.6.x-dev » 8.7.x-dev

Drupal 8.6.0-alpha1 will be released the week of July 16, 2018, which means new developments and disruptive changes should now be targeted against the 8.7.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.7.x-dev » 8.8.x-dev

Drupal 8.7.0-alpha1 will be released the week of March 11, 2019, which means new developments and disruptive changes should now be targeted against the 8.8.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

volkswagenchick’s picture

Issue tags: +drupalnorth2019

Tagging for DrupalNorth 2019

vacho’s picture

Refactoring some the code for made more standard.

Version: 8.8.x-dev » 8.9.x-dev

Drupal 8.8.0-alpha1 will be released the week of October 14th, 2019, which means new developments and disruptive changes should now be targeted against the 8.9.x-dev branch. (Any changes to 8.9.x will also be committed to 9.0.x in preparation for Drupal 9’s release, but some changes like significant feature additions will be deferred to 9.1.x.). For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

ravi.shankar’s picture

Status: Needs work » Needs review
FileSize
2.77 KB

I have re-rolled patch #64 on Drupal 8.9.x-dev

Status: Needs review » Needs work

The last submitted patch, 66: 2040823-66.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

owenpm3’s picture

Status: Needs work » Needs review
FileSize
2.77 KB

I've added a patch with the config/optional noted in #58. Sounds like this could still need some work.

Status: Needs review » Needs work

The last submitted patch, 68: 2040823-68.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

ankushgautam76@gmail.com’s picture

Title: Write tour integration for Menu admin page » Add tour integration for Menu admin page

Version: 8.9.x-dev » 9.1.x-dev

Drupal 8.9.0-beta1 was released on March 20, 2020. 8.9.x is the final, long-term support (LTS) minor release of Drupal 8, which means new developments and disruptive changes should now be targeted against the 9.1.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

ridhimaabrol24’s picture

Assigned: Electric Doorknob » Unassigned
Status: Needs work » Needs review
FileSize
2.99 KB
2.66 KB

Patch added for Drupal 9.1

Status: Needs review » Needs work

The last submitted patch, 72: 2040823-72.patch, failed testing. View results

ridhimaabrol24’s picture

Status: Needs work » Needs review
FileSize
4.11 KB
4.06 KB

Fixed failed tests.

Status: Needs review » Needs work

The last submitted patch, 74: 2040823-74.patch, failed testing. View results

ridhimaabrol24’s picture

ridhimaabrol24’s picture

Fixing failed test case

ridhimaabrol24’s picture

Status: Needs work » Needs review

Version: 9.1.x-dev » 9.2.x-dev

Drupal 9.1.0-alpha1 will be released the week of October 19, 2020, which means new developments and disruptive changes should now be targeted for the 9.2.x-dev branch. For more information see the Drupal 9 minor version schedule and the Allowed changes during the Drupal 9 release cycle.

Version: 9.2.x-dev » 9.3.x-dev

Drupal 9.2.0-alpha1 will be released the week of May 3, 2021, which means new developments and disruptive changes should now be targeted for the 9.3.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.0-rc1 was released on November 26, 2021, which means new developments and disruptive changes should now be targeted for the 9.4.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.4.x-dev » 9.5.x-dev

Drupal 9.4.0-alpha1 was released on May 6, 2022, which means new developments and disruptive changes should now be targeted for the 9.5.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.5.x-dev » 10.1.x-dev

Drupal 9.5.0-beta2 and Drupal 10.0.0-beta2 were released on September 29, 2022, which means new developments and disruptive changes should now be targeted for the 10.1.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

smustgrave’s picture

Status: Needs review » Needs work

Patch or MR doesn't apply anymore
The last patch or MR doesn't apply to the target branch, please reroll the code so that it can be reviewed by the automated testbot.

rpayanm made their first commit to this issue’s fork.

rpayanm’s picture

Status: Needs work » Needs review
smustgrave’s picture

Status: Needs review » Needs work

Appears there were some errors.

Rajeshreeputra made their first commit to this issue’s fork.

Version: 10.1.x-dev » 11.x-dev

Drupal core is moving towards using a “main” branch. As an interim step, a new 11.x branch has been opened, as Drupal.org infrastructure cannot currently fully support a branch named main. New developments and disruptive changes should now be targeted for the 11.x branch, which currently accepts only minor-version allowed changes. For more information, see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

quietone’s picture

Status: Needs work » Postponed
Issue tags: -Tour

This extension is being deprecated, see #3336033: [Meta] Tasks to deprecate Tour module. It will be removed from core and moved to a contrib project, #3376099: [11.x] [Meta] Tasks to remove Tour.

This is now Postponed. The status is set according to two policies. The Remove a core extension and move it to a contributed project and the Extensions approved for removal policies.