- Issue created by @charles belov
- First commit to issue fork.
- πΊπΈUnited States smustgrave
Oops not working on this one, wrong ticket.
- Status changed to Postponed
3 months ago 8:10pm 6 September 2024 - πΊπΈUnited States smustgrave
Correct me if I'm wrong but for this to work we would need something like π Add an "alt text" to tour tips describing the situation and the highlighted element Active added?
Else that overview page would just be
title
bodyExcluding selector value because I imagine that wouldn't be useful to anyone.
- πΊπΈUnited States charles belov San Francisco, CA, US
Agreed. This is dependent on π Add an "alt text" to tour tips describing the situation and the highlighted element Active and I'm adding it as a related issue.
- πΊπΈUnited States smustgrave
Blocker landed.
Decided we will stay using the label (now title) + body field, with the assumption the site admins are using descriptive titles.
- πΊπΈUnited States thejimbirch Cape Cod, Massachusetts
With the alt text approach, I feel like you would basically have to write the tour twice.
An alternative approach could be to parse the tour yaml file completely. Looking at a simple tour:
langcode: en status: true dependencies: { } id: seo label: SEO routes: - route_name: entity.dashboard.canonical route_params: id: seo tips: seo_dashboard: id: seo_dashboard plugin: text label: 'SEO Dashboard' weight: -10 position: 'auto' selector: null body: '<p>This dashboard gathers all the SEO-related features of Drupal with descriptions and links to configuring them.</p>' dashboard_section: id: dashboard_section plugin: text label: 'Dashboard Section' weight: -10 position: 'auto' selector: .block-dashboard-text-block body: '<p>In each of these sections, you will find links to the installed SEO feature with instructions on how to use them.</p>' guided_tour: id: guided_tour plugin: text label: 'Guided Tour' weight: -10 position: 'auto' selector: null body: '<p>Let''s continue the tour by visiting each of the SEO features of the site. First up, the <a href="/admin/config/search/seo-checklist?tour=1">SEO Checklist</a>.</p>'
You could loop through the tips and output something like this:
<h2>tip.*.label</h2> tip.*.body
- πΊπΈUnited States thejimbirch Cape Cod, Massachusetts
Drupal core also has Help Topics. Could that be part of the solution?
- πΊπΈUnited States charles belov San Francisco, CA, US
+1 for parsing the yaml tour as it ensures the steps and labels are in sync and no duplicate effort.
- πΊπΈUnited States smustgrave
Actually a -1 for that approach.
1. We have removed all tours from the contrib module and placed into recipes
2 The yml could easily not match what's in the database so what appears in the yml could be different then what appears on the actual tour. - πΊπΈUnited States charles belov San Francisco, CA, US
Then could the separate window could be generated from the database? The source doesn't matter; what matters is that it be the same source.
- πΊπΈUnited States smustgrave
@thejimbirch we actually dropped the alt idea. We tweaked the tip form some to make it clear that site builders really should be using descriptive titles. With that assumption we can use title and body. But do need to determine where this link should go
Don't want to distract from the actual content so think at the bottom probably would be best
Thought about a hot key but people may not know about it then. But rough idea, probably would shrink the text and change to "View full tour"
- πΊπΈUnited States thejimbirch Cape Cod, Massachusetts
Nice idea. Maybe a external link style icon next to the close icon with the hover/title text you have there?