Skip to content

fix(navbar): fixing navbar disabled item opacity (#UIM-396) #440

Conversation

Margar1ta
Copy link
Contributor

No description provided.

@PositiveJS
Copy link
Contributor

Preview docs changes for 5280239 at https://positive-js.github.io/mosaic-previews/pr440-5280239/

@pimenovoleg pimenovoleg requested a review from lskramarov March 18, 2020 16:55
@Margar1ta Margar1ta changed the title fix(navbar): fixing navbar disabled item opacity fix(navbar): fixing navbar disabled item opacity (#UIM-396) Mar 18, 2020
@@ -50,7 +50,7 @@
&[disabled] {
.mc-navbar-title,
.mc-icon {
opacity: map-get($navbar, item_state-disabled_opacity);
opacity: 0.3;
Copy link
Contributor

Choose a reason for hiding this comment

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

На синке обсуждали, нужно вернуть параметр в токены.
И возможно добавить в токены другие параметры opacity.

@PositiveJS
Copy link
Contributor

Preview docs changes for 071a308 at https://positive-js.github.io/mosaic-previews/pr440-071a308/

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@PositiveJS
Copy link
Contributor

Preview docs changes for 23fe28a at https://positive-js.github.io/mosaic-previews/pr440-23fe28a/

@pimenovoleg pimenovoleg merged commit a939bb4 into positive-js:master Mar 25, 2020
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.

4 participants