Clarity: Remove overwrite.scss from main.scss

Created on 17 Feb 2017  路  6Comments  路  Source: vmware/clarity

Select one ... (check one with "x")

[x ] bug
[ ] feature request
[ ] enhancement

Expected behavior

Users should be able to build our main.scss with their own overwrites. For example:

main.user.scss:

@import "overwrite.clarity";
@import "path/to/node_modules/clarity-ui/main";

Actual behavior

Right now as we've included overwrites.scss in our main.scss, users' overwrites of our default variables are being nullified. So users' overwrites of default variable values wouldn't take effect.

Environment details

  • Angular version: 2.0.X

  • Clarity version:

  • OS and version:

  • Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Most helpful comment

@KashMoneyMillionaire
We have the code behind to support using/theming Clarity in this way but we are presently working through the process internally to validate it and then document it for our users.

All 6 comments

Is there any documentation supporting this behavior (even if it's beta)? I didn't see it in Docs -> Color Palette, but maybe I'm blind.

Specifically, I'm looking for the variables to overwrite and how to incorporate this into the angular-cli build.

@KashMoneyMillionaire
We have the code behind to support using/theming Clarity in this way but we are presently working through the process internally to validate it and then document it for our users.

Any updates on this?

@mfp22
Yes, we are very close to having a dark theme ready for review. This effort will include a more simplified approach to theming Clarity.

Subscribe to #968 to keep track of the effort. Thanks!

Awesome!! Is there another issue for that I can watch?

Hi there 馃憢, this is an automated message. To help Clarity keep track of discussions, we automatically lock closed issues after 14 days. Please look for another open issue or open a new issue with updated details and reference this one as necessary.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

mrmokwa picture mrmokwa  路  28Comments

reddolan picture reddolan  路  99Comments

gracesnoh picture gracesnoh  路  41Comments

reddolan picture reddolan  路  27Comments

gitnarendra picture gitnarendra  路  27Comments