Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Tour layout #202

Closed
2 of 5 tasks
erudman21 opened this issue Feb 3, 2018 · 2 comments
Closed
2 of 5 tasks

Tour layout #202

erudman21 opened this issue Feb 3, 2018 · 2 comments
Assignees
@hql287
Labels
bug

Comments

@erudman21
Copy link

Please answer these question so I can help you faster.

Is it a bug or a feature request?

  • It's a bug.
  • I'm requesting a new feature.

Which version are you using?

  • macOS
  • Windows
  • Linux

Which is the version/distribution of the OS?

Windows 10

Describe your issue

Unnecessary scrollbar and part of slide image showing underneath slider for multiple slides in the tour. Only part of the 'next' and 'skip' buttons are clickable for these slides as well.

Ex:
screenshot 5

I think the issue could be fixed by adding overflow hidden to the Wrapper styled component.
@hql287
Copy link
Owner

hql287 commented Feb 3, 2018

@erudman21: Omg this looks terrible!! Thanks for reporting this. WIll fix soon!

@hql287 hql287 added the bug label Feb 3, 2018
@hql287 hql287 self-assigned this Feb 3, 2018
@hql287 hql287 mentioned this issue Mar 2, 2018
Merged
hql287 pushed a commit that referenced this issue Mar 2, 2018
@hql287
Fixed the multiple scrollbars bug for linux/windows (#236)
484929f 
Fixed #182 & #202
@hql287
Copy link
Owner

hql287 commented Mar 2, 2018

Fixed with #236

@hql287 hql287 closed this as completed Mar 2, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@hql287 hql287

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hql287 @erudman21