Closed (fixed)
Project:
Tour
Version:
2.0.x-dev
Component:
User interface
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
19 Oct 2024 at 00:06 UTC
Updated:
4 Nov 2024 at 17:34 UTC
Jump to comment: Most recent
A followup to #3473594: Improve the tour block in the context of the navigation module. In the latest commits before the issue had to go in somehow the styling was dropped. The icon is missing and the menu item for the tour and no tour navigation block look like actual buttons instead of being in line with the rest of the navigation blocks. plus the the dividing line between navigation blocks is also missing. (i've moved the block in the navigation layout above the user block)
no tour

tour

Readd the styling for the tour navigation block
| Comment | File | Size | Author |
|---|---|---|---|
| no_tour.jpg | 10 KB | rkoller | |
| tour.jpg | 10.48 KB | rkoller |
Comments
Comment #2
smustgrave commentedWould do a cache clear or re-install navigation. Rendering fine for me but you may have some old config saved in navigation .
Comment #3
rkolleri always do a
drush crwhen i checked out a branch and pulled the latest changes in git. did that several times. i tried to remove the navigation block that turned up a WSOD. i then just did adrush pmu tourwent in addition to that into the navigation layout page and removed the now idle navigation block that was associated with tour. after that i'Ve installed the tour module again. but then ran into:Comment #4
smustgrave commentedI think you need to uninstall navigation
Comment #5
rkollertried that now as well. ran
drush pmu touranddrush pmu navigation, then ran adrush crand thendrush en navigationand after thatdrush en tourand anotherdrush cr., but i run into the same error still. :/ NO idea why.Comment #6
rkollerhm i've tried the following and simply removed the if else and only used the navigation_sdc case:
with that the error is gone. could it be that it stumbles across the if statement?
{% if navigation_sdc %}?Comment #7
smustgrave commentedCore version?
Comment #8
rkoller11.x
Comment #9
smustgrave commentedWould verify your 11.x has the SDC changes.
Comment #10
rkollerin which issue those SDC changes went in for core? so i am able to see which files changed and check if those changes are also in place for me (which should be the case cuz i am on the latest version of 11.x)
Comment #11
rkollerwas it that issue? #3458215: Migrate Toolbar button to SDC
Comment #12
smustgrave commentedYup
Comment #13
rkolleryes changes are in place. svgs moved around and for example toolbar-button.component.yaml got added and toolbar-button.pcss.css changed and so on.
Comment #14
rkolleryou dont run into that WSOD on 11.x?
Comment #15
smustgrave commentedin tour.module can you change line 302 to
Comment #16
rkollerahhhhhh i guess i understand now. yep your suggested change fixed the WSOD. it is working now. that means the problem was i was using a none stable version of core, dev in my case. does that mean it would also need an entry for alpha and beta versions as well?
Comment #17
smustgrave commentedI don't mind including that change just means that there is a potential for someone to be on 11.x-dev branch and not have the SDC changes. Think that would be a small hole but probably best case we can cover.
Comment #18
rkolleri agree, the odds someone on 11.x-dev but on a commit pre SDC who is using the tour module are rather low, very low.
Comment #20
smustgrave commented