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

fixing official example page and datepicker/calendar components using… #5108

Merged
merged 2 commits into from
Jun 6, 2018

Conversation

lorejoh12
Copy link
Contributor

@lorejoh12 lorejoh12 commented Jun 5, 2018

… box-sizing: content-box on the calendar component to prevent scroll bars

Pull request checklist

Description of changes

adding explicit css check for box-sizing: content-box to the Calendar root component so that the calendar component and datepicker components don't have unnecessary scrollbars.

Focus areas to test

(optional)

Microsoft Reviewers: Open in CodeFlow

… box-sizing: content-box on the calendar component to prevent scroll bars
@lorejoh12 lorejoh12 requested a review from dzearing June 5, 2018 21:51
@dzearing dzearing merged commit 4353952 into microsoft:master Jun 6, 2018
Markionium added a commit to Markionium/office-ui-fabric-react that referenced this pull request Jun 6, 2018
* master:
  Part 2 of demo page refactor (microsoft#5089)
  Update jest.js
  fixing official example page and datepicker/calendar components using… (microsoft#5108)
  Don't run prettier and tslint in parallel as it might cause conflicts (microsoft#5100)
  FocusTrapZone - restore last focused descendant element (microsoft#5103)
  Coachmark/TeachingBubble: Fix SCSS selectors for buttons and Close Icon (microsoft#4835)
  HoverCard: IE11 fix (microsoft#5105)
Markionium added a commit to Markionium/office-ui-fabric-react that referenced this pull request Jun 6, 2018
* master:
  Charting (microsoft#4954)
  Deprecation lint rule! (microsoft#5109)
  Implement selection for selected items list (microsoft#5036)
  Ignore common/changes and don't prettify json files (microsoft#5112)
  Part 2 of demo page refactor (microsoft#5089)
  Update jest.js
  fixing official example page and datepicker/calendar components using… (microsoft#5108)
  Don't run prettier and tslint in parallel as it might cause conflicts (microsoft#5100)
  FocusTrapZone - restore last focused descendant element (microsoft#5103)
  Coachmark/TeachingBubble: Fix SCSS selectors for buttons and Close Icon (microsoft#4835)
  HoverCard: IE11 fix (microsoft#5105)
Markionium added a commit to Markionium/office-ui-fabric-react that referenced this pull request Jun 6, 2018
* master: (31 commits)
  Charting (microsoft#4954)
  Deprecation lint rule! (microsoft#5109)
  Implement selection for selected items list (microsoft#5036)
  Ignore common/changes and don't prettify json files (microsoft#5112)
  Part 2 of demo page refactor (microsoft#5089)
  Update jest.js
  fixing official example page and datepicker/calendar components using… (microsoft#5108)
  Don't run prettier and tslint in parallel as it might cause conflicts (microsoft#5100)
  FocusTrapZone - restore last focused descendant element (microsoft#5103)
  Coachmark/TeachingBubble: Fix SCSS selectors for buttons and Close Icon (microsoft#4835)
  HoverCard: IE11 fix (microsoft#5105)
  FocusTrapZone bug allows breaking out of the trap (microsoft#4898)
  Applying package updates.
  Update ISSUE_TEMPLATE.md
  Experiment/Nav component: hide nav group header if all the links under it are hidden (microsoft#5095)
  Add optional prop to not dismiss Callout on focus loss (microsoft#5092)
  Experiments: moves ShimmerTile from Shimmer to Tile. (microsoft#5090)
  Run jest in parallel on Windows (microsoft#5096)
  Applying package updates.
  Major bump jest-serializer-merge-styles
  ...
@lorejoh12 lorejoh12 deleted the jolore/contentBoxFix branch February 22, 2019 21:42
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Regression]Datepicker: unnecessary scrollbars
2 participants