-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
SCSS variable cleanup in prep for Docsy upgrade #2440
SCSS variable cleanup in prep for Docsy upgrade #2440
Conversation
@@ -1,138 +1,24 @@ | |||
.registry { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The use of these styles are nowhere to be found, including in the commit that introduced them:
background-color: $utilities; | ||
} | ||
|
||
.badge-tracer { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no tracer component, so this style was never used.
background-color: $dark-blue; | ||
} | ||
|
||
.badge-instrumentation { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is missing the .badge
context, and so was never used.
$twitter-blue: #00aced; | ||
$gitter-magenta: #ec1363; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These two component variables are not used.
$google_font_name: 'Noto Sans'; | ||
$google_font_family: 'Noto+Sans:300,400,600,700'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Redefining $google_font_*
has no effect when $td-enable-google-fonts
is false
$enable-gradients: false; | ||
$enable-rounded: true; | ||
$enable-shadows: false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dropping these, falling back on Docsy defaults.
5d0acd2
to
e566cc1
Compare
Preview:
In particular, you'll note that badge colors have not changed: