Skip to content

fix(Page): Keep sidebar collapsed between sizes#2264

Merged
mcoker merged 2 commits intopatternfly:masterfrom
Venefilyn:patch-1
Sep 13, 2019
Merged

fix(Page): Keep sidebar collapsed between sizes#2264
mcoker merged 2 commits intopatternfly:masterfrom
Venefilyn:patch-1

Conversation

@Venefilyn
Copy link

What: This fixes the sidebar appearing and closing when resizing window from md to sm when sidebar is closed.

fixes #2252

@patternfly-build
Copy link
Collaborator

PatternFly-Next preview: https://patternfly-next-pr-2264.surge.sh

// Collapse nav
&.pf-m-collapsed {
@media screen and (min-width: $pf-global--breakpoint--md) {
@media screen {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you remove the wrapping @media query, and leave the max-width and overflow selectors?

Copy link
Author

Choose a reason for hiding this comment

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

Makes sense.. done :)

Copy link
Contributor

@mcoker mcoker left a comment

Choose a reason for hiding this comment

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

nice, thanks @spytec!

@mattnolting mattnolting self-requested a review September 13, 2019 21:03
Copy link
Collaborator

@mattnolting mattnolting left a comment

Choose a reason for hiding this comment

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

LGTM 👍 Thanks @spytec

@mcoker mcoker merged commit 306fdbc into patternfly:master Sep 13, 2019
@redallen
Copy link
Contributor

🎉 This PR is included in version 2.31.10 🎉

The release is available on:

Your semantic-release bot 📦🚀

@Venefilyn Venefilyn deleted the patch-1 branch September 23, 2019 08:41
rachael-phillips added a commit that referenced this pull request Sep 27, 2019
# 2019.08 release notes (2019-10-01)
Packages released:
- @patternfly/patternfly@2.x

## Components
- **Charts:** 
  - Adjust padding for pie and donut charts ([#2247](#2247))
  - Set mix-blend-mode for better color contrast ([#2239](#2239))
  - Update area chart opacity ([#2233](#2233))
- **Copy to clipboard:** Wrapped code in pre which allows the code to preserve line-breaks and spaces and also applies styling such as the monospace font. ([#2260](#2260))
- **Data list:** Make data list borders consistent with table ([#2289](#2289))
- **Data toolbar:** 
  - Updated clear button mods ([#2248](#2248))
  - Added in examples for expandable ([#2273](#2273))
- **Flex layout:** Wrapped flex items, update css ([#2214](#2214))
- **Wizard:**
  - Re-enabled wizard modal demos ([#2259](#2259))
  - Updated in page wizard demos to use dark sidebar/nav ([#2296](#2296))
- **Navigation:** Added styling updates to dark nav ([#2283](#2283))
- **Page:** Added ability to keep sidebar collapsed between sizes ([#2264](#2264))
- **Pagination:** Added compact variation ([#2275](#2275))
- **Radio:** Updated the radio component example to single components ([#2231](#2231))
- **Table:**
  - Fixed a11y issues in org documentation/html/table ([#2282](#2282))
  - Added empty and loading state table demos ([#2254](#2254))
- **Tooltip:** Added text align left modifier ([#2284](#2284))
- **Select:**
  - Added typeahead form wrapper, udpated css ([#2255](#2255))
  - Add top expanded example ([#2026](#2026))
- **Chip group:** Add overflow to group ([#2278](#2278))
- **Button:** Add control modifier ([#2005](#2005))

## Other
- **Shield:** Added missing components, missing var to brand docs js ([#2263](#2263))
redallen pushed a commit that referenced this pull request Oct 1, 2019
* Create 2019.08 release notes (2019-10-01)

Adding new release notes

* Update 2019.08 release notes (2019-10-01)

# 2019.08 release notes (2019-10-01)
Packages released:
- @patternfly/patternfly@2.x

## Components
- **Charts:** 
  - Adjust padding for pie and donut charts ([#2247](#2247))
  - Set mix-blend-mode for better color contrast ([#2239](#2239))
  - Update area chart opacity ([#2233](#2233))
- **Copy to clipboard:** Wrapped code in pre which allows the code to preserve line-breaks and spaces and also applies styling such as the monospace font. ([#2260](#2260))
- **Data list:** Make data list borders consistent with table ([#2289](#2289))
- **Data toolbar:** 
  - Updated clear button mods ([#2248](#2248))
  - Added in examples for expandable ([#2273](#2273))
- **Flex layout:** Wrapped flex items, update css ([#2214](#2214))
- **Wizard:**
  - Re-enabled wizard modal demos ([#2259](#2259))
  - Updated in page wizard demos to use dark sidebar/nav ([#2296](#2296))
- **Navigation:** Added styling updates to dark nav ([#2283](#2283))
- **Page:** Added ability to keep sidebar collapsed between sizes ([#2264](#2264))
- **Pagination:** Added compact variation ([#2275](#2275))
- **Radio:** Updated the radio component example to single components ([#2231](#2231))
- **Table:**
  - Fixed a11y issues in org documentation/html/table ([#2282](#2282))
  - Added empty and loading state table demos ([#2254](#2254))
- **Tooltip:** Added text align left modifier ([#2284](#2284))
- **Select:**
  - Added typeahead form wrapper, udpated css ([#2255](#2255))
  - Add top expanded example ([#2026](#2026))
- **Chip group:** Add overflow to group ([#2278](#2278))
- **Button:** Add control modifier ([#2005](#2005))

## Other
- **Shield:** Added missing components, missing var to brand docs js ([#2263](#2263))

* Rename 2019.08 release notes (2019-10-01) to 2019.08 release notes (2019-10-01).MD

Added .md file name

* Update 2019.08 release notes (2019-10-01).MD

Co-Authored-By: Michael Coker <35148959+mcoker@users.noreply.github.com>

* alphabetize components
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.

6 participants