-
Notifications
You must be signed in to change notification settings - Fork 92
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
Warning timeout update #38
base: alpha
Are you sure you want to change the base?
Conversation
Add YUR Fit link in list view Update release notes Update Patreon Credits
Merge README to beta branch
Fix a quick typo
Remove unnecessary emergency mods list
Add new patron to credits
Add upload date in List View
Added timeouts in ms for warnings based on warning message length
transition: 0s; | ||
} | ||
} | ||
|
||
#resize-button, #minimize-button { | ||
&:hover { | ||
background: silver; | ||
background: rgba(128, 128, 128, 0.3); |
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.
Color literals like rgba(128, 128, 128, 0.3)
should only be used in variable declarations; they should be referred to via variable everywhere else.
@@ -125,14 +112,14 @@ | |||
|
|||
#close-button { | |||
&:hover { | |||
background: #e0021f; | |||
background: #ff1938; |
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.
Color literals like #ff1938
should only be used in variable declarations; they should be referred to via variable everywhere else.
@@ -89,21 +88,9 @@ | |||
} | |||
|
|||
&.theme-dark { | |||
background: #252525; | |||
background: rgba(0, 0, 0, 0.6); |
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.
Color literals like rgba(0, 0, 0, 0.6)
should only be used in variable declarations; they should be referred to via variable everywhere else.
background: url(../assets/update.png); | ||
background-repeat: no-repeat; | ||
transition: .5s; | ||
} | ||
} | ||
|
||
h5.scanning-text { |
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.
Avoid qualifying class selectors with an element.
text-align: center; | ||
|
||
&::before { | ||
&:first-child::before { |
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.
Rule set contains (12/10) properties
@@ -49,6 +49,11 @@ | |||
white-space: nowrap; | |||
text-overflow: ellipsis; | |||
} | |||
.upload-date { |
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.
Selector should have depth of applicability no greater than 2, but was 3
margin-bottom: 5px; | ||
} | ||
|
||
#dl-location, #theme-image-path { |
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.
Avoid using id selectors
Each selector in a comma sequence should be on its own single line
No description provided.