Skip to content

Nav SCSS to MergeStyles Part 2: Style Conversion#4056

Merged
oengusmacinog-zz merged 6 commits intomicrosoft:masterfrom
oengusmacinog-zz:nav-scss2ms-pt2
Feb 21, 2018
Merged

Nav SCSS to MergeStyles Part 2: Style Conversion#4056
oengusmacinog-zz merged 6 commits intomicrosoft:masterfrom
oengusmacinog-zz:nav-scss2ms-pt2

Conversation

@oengusmacinog-zz
Copy link
Copy Markdown
Collaborator

@oengusmacinog-zz oengusmacinog-zz commented Feb 21, 2018

Pull request checklist

  • Include a change request file using $ npm run change

Description of changes

Convert all SCSS to MergeStyles in Nav component.

import * as stylesImport from './Nav.scss';
const styles: any = stylesImport;
import { AnimationClassNames, mergeStyles } from '../../Styling';
// import * as stylesImport from './Nav.scss';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Make sure these commented lines are removed before merging.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Done. Good catch!

@oengusmacinog-zz oengusmacinog-zz merged commit 4b74ee8 into microsoft:master Feb 21, 2018
@oengusmacinog-zz oengusmacinog-zz deleted the nav-scss2ms-pt2 branch February 21, 2018 22:44
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 31, 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.

3 participants