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

Converted Components and Samples over to SASS #279

Merged
merged 19 commits into from Jan 26, 2016

Conversation

gokunymbus
Copy link
Contributor

No description provided.

John Miller and others added 16 commits January 11, 2016 23:04
- Updated component helper to include name of Pre-processor
- Removed outdated Component Example html
- Converted all Icons to Mixins so they can be used throughout SASS.
- Converted all components over to SASS
- FIxed All errors in Sass Files.
- Created a couple helper tasks for converting over mixins
…into miljo/less-sass-components

# Conflicts: [RESOLVED]
#	gulp/ComponentSamples.js
#	gulp/FabricBuild.js
#	gulp/FabricComponents.js
#	gulp/modules/ComponentHelper.js
#	gulp/modules/Plugins.js
#	src/less/_Fabric.Animations.less
- Working on fixing an issue where the LESS compiler should not be running but it is anyway.
- Removing build config from each file
- Now working for Samples
- Removed Sass linting unti gup-sass-lint fixes a big bug.
- Setup a config for Sass-lint, keeping until we get Sass linting working.
- Fixed two icons that were not working.
- Had to fix up the circle icon on button.
@mikewheaton
Copy link
Contributor

I'm having trouble getting the build system to use Sass rather than LESS. I've run gulp ConfigureEnvironment-setSassMode, which looks like it should be all that's necessary to switch the build system over to Sass. I then made changes to some components' Sass files, but running gulp gives me the same output in fabric.components.css. What am I missing?

@@ -0,0 +1,108 @@
// Copyright (c) Microsoft. All rights reserved. Licensed under the MIT license. See LICENSE in the project root for license information.
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems like this styling is from a previous version of SearchBox.

@mikewheaton
Copy link
Contributor

I did a comparison of the dist/ output from 'master' and this branch. There are some differences (mostly in source order and things like changing border: 0 to border: none but I don't see any major regressions. Should be fine to merge in once we fix the SearchBox regression @wdo3650 noticed.

gokunymbus added a commit that referenced this pull request Jan 26, 2016
Converted Components and Samples over to SASS
@gokunymbus gokunymbus merged commit 22f3b94 into master Jan 26, 2016
@mikewheaton mikewheaton deleted the miljo/less-sass-components branch February 9, 2016 03:24
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.

None yet

4 participants