caiosba / drupal-demo-tour

Demo Tour module for Drupal 7
GNU General Public License v2.0
0 stars 0 forks source link

Implementing on a drupal 7 site #1

Open crivote opened 6 years ago

crivote commented 6 years ago

Thanks for your nifty module! I'm trying to adjust the appearance of the module to match the graphic style of the site and I have a doubt about the function of one of the temporal div elements you add to make the tour. What's the purpose of the .tour-step-background element? Seems to get the size of the featured element, but is set to display: none in tour.css line 45.

Also, may I suggest that the z-index of the .tour-step-backdrop class has the !important atribute? If the featured element has a z-index with a more specific css rule it supersedes your definition and gets under the tour-backdrop.

caiosba commented 6 years ago

Hi, thanks! Answers inline below:

What's the purpose of the .tour-step-background element?

It gets displayed by JavaScript later.

Also, may I suggest that the z-index of the .tour-step-backdrop class has the !important atribute?

Sure, go ahead :)