Skip to content

Conversation

@alexeykostevich
Copy link

@alexeykostevich alexeykostevich commented Apr 7, 2020

  • Fixes the issue with the sidenav "show" button being scrolled with the main content.
  • Fixes the issue with the sidenav being overlapped by the main content. Creates a new stacking context for a user's content to allow safely use z-index in documented components (fixes styleguide rendering issue ember-learn/ember-styleguide#286).
  • Fixes minor issues with formatting and spelling.

@alexeykostevich alexeykostevich changed the title Fix layout issues Fix layout issues with the sidenav Apr 7, 2020
@alexeykostevich
Copy link
Author

@mansona , please, let me know if you have any concerns about the PR or its relevancy.

@MelSumner MelSumner requested a review from mansona April 26, 2020 15:28
Copy link

@MelSumner MelSumner left a comment

Choose a reason for hiding this comment

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

This seems like it will fix the issues we were seeing with the overlapping content, although it would be great to figure it out (as an iteration) without using z-index or absolute positioning. :) Thank you for contributing @alexeykostevich ! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

styleguide rendering issue

2 participants