gjb2048 / moodle-theme_essential

The Essential Moodle Theme
https://gjb2048.github.io/moodle-theme_essential/
GNU General Public License v3.0
91 stars 120 forks source link

User tour with backdrop alignment off #831

Closed mnods closed 7 years ago

mnods commented 7 years ago

essential 3.2.1.1 moodle 3.2.3 user tour for /course/view.php% one tour step points to ID custom_menu_courses backdrop off - points to correct ID, with it on the tour step gets aligned to centre, highlighting the wrong element. course__about_moodle4mac

I also have some weirdness happening in a different step, with or without backdrop - text in the step not displaying properly - looks like it needs a line height. course__about_moodle4mac

neither issue appeared when using clean or boost.

gjb2048 commented 7 years ago

Thanks for the report.

How do I replicate the problem please? I've not used user tours before.

mnods commented 7 years ago

site admin > appearance > user tours create a new tour give it a name in the Apply to URL match field use /course/view.php% show with backdrop = YES leave others default save head back to main user tours page - admin/tool/usertours/configure.php click on the item you just created add a new step title "user menu" css selector = .usermenu title = User Menu content = This is the user menu which displays on all of your pages. It contains links to your Dashboard, Messages, Profile, and your Preferences.

head to a course main page and the user tour should appear. you can reset to start again via the rest link in the footer.

mnods commented 7 years ago

... so you could add new steps to this tour for various other selectors eg #custom_menu_courses and others in main tour settings, alter backdrop setting to see difference in placement of the tour step on the course page.

gjb2048 commented 7 years ago

Thanks, added to list of jobs, probably sometime in July.

mnods commented 7 years ago

nice on gareth, thanks.

gjb2048 commented 7 years ago

Target type = Selector?

gjb2048 commented 7 years ago

Caused by:

.navbar .navbar-inner {
    line-height: 0;
}
gjb2048 commented 7 years ago

Not in M3.3.

gjb2048 commented 7 years ago

Testing instructions do not state how to replicate first issue.

gjb2048 commented 7 years ago

Cannot replicate either issue in M3.1.

gjb2048 commented 7 years ago

Can replicate first issue in M3.2.

gjb2048 commented 7 years ago

In M3.3 the first issue is off but not by much.

gjb2048 commented 7 years ago

Actually ditto for M3.2.

gjb2048 commented 7 years ago

Ok I think that the first issue is down to user tours being confused about the navbar not being at the top but changing position. Not sure I can solve that.

gjb2048 commented 7 years ago

In M3.2 the #custom_menu_courses issue jumps around from when you've done 'Next' step to reloading with it showing. Or rather if you've scrolled down the page.

gjb2048 commented 7 years ago

Ok I can replicate the first issue with being 'off' in the Clean theme if the '.langmenu' is targeted on the navbar as it's within a '.collapse' -> core issue for that. So far in M3.2 and M3.3 but not M3.1.

gjb2048 commented 7 years ago

Not actually sure why 'line-height: 0' is there, removing for now in M3.2 and M3.3.

gjb2048 commented 7 years ago

Ok, up to you to raise MDL tracker issue.