Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fixes horizontal scroll on mobile menu #251

Merged
merged 6 commits into from
Jun 29, 2020

Conversation

caylahamann
Copy link
Contributor

Description

Fixes the horizontal scroll on mobile menu caused by a <pre> tag.

Reviewer Notes

Go to a page such as "Intro to NerdStorage" and horizontally decrease your window size, and then horizontally scroll to the right to see the bug.

Related Issue(s) / Ticket(s)

If there are any related GitHub Issues or JIRA tickets, add links to them here.

Screenshot(s)

Before

Screen Shot 2020-06-25 at 2 56 01 PM

After

Screen Shot 2020-06-25 at 2 58 27 PM

@caylahamann caylahamann added the enhancement New feature or request label Jun 25, 2020
@jerelmiller
Copy link
Contributor

@caylahamann The build should be fixed when #235 gets merged

@zstix
Copy link
Contributor

zstix commented Jun 25, 2020

I'm still seeing a scrollbar on the mobile menu, I'll try to identify the source of the issue 🤔
Screen Shot 2020-06-25 at 3 56 10 PM

@caylahamann
Copy link
Contributor Author

I'm still seeing a scrollbar on the mobile menu, I'll try to identify the source of the issue 🤔
Screen Shot 2020-06-25 at 3 56 10 PM

@zstix do you know on what page you are seeing the issue? I've tried reproducing in chrome and firefox and I can't seem to see a bottom scroll bar :(

@zstix
Copy link
Contributor

zstix commented Jun 26, 2020

@caylahamann So sorry for the slow responses. I'm seeing this on the home page (and most of the other pages too) when the menu is tall enough to the vertical scroll. Happy to hop on a quick call if you would like.

Copy link
Contributor

@zstix zstix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not seeing this on any mobile devices 💯

@caylahamann caylahamann merged commit 6ef2b48 into master Jun 29, 2020
@caylahamann caylahamann deleted the cayla/mobile-horizontal-scroll branch June 29, 2020 17:09
@nr-opensource-bot
Copy link
Contributor

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants