Staging #588
Staging #588
Conversation
New Crowdin translations
Update @types/chrome to the latest version 🚀
Update web-ext to the latest version 🚀
Update @types/node to the latest version 🚀
![accesslint[bot]](https://avatars.githubusercontent.com/in/1502?s=60&v=4){kind=small}
| (keydown)="handleKeydown()" | ||
| (blur)="handleBlur()" | ||
| (keydown.enter)="handleSubmit()" | ||
| > |
There was a problem hiding this comment.
Looks like there's a label missing for this input. That makes it hard for people using screen readers or voice control to use the input. If you don't want a visual label, try an aria-label attribute.
![hound[bot]](https://avatars.githubusercontent.com/in/3598?s=60&v=4){kind=small}
| } | ||
| .connection-indicator--uncertain { | ||
| color: var(--orange-color); | ||
| } |
There was a problem hiding this comment.
Files should end with a trailing newline
| } | ||
| .connection-indicator--connected { | ||
| color: var(--green-color); | ||
| } |
There was a problem hiding this comment.
Rule declaration should be followed by an empty line
| &:hover { | ||
| opacity: 1; | ||
| } | ||
| } |
There was a problem hiding this comment.
Rule declaration should be followed by an empty line
| display: inline-block; | ||
| opacity: .6; | ||
| transition: all .3s ease; | ||
| &:hover { |
There was a problem hiding this comment.
Rule declaration should be preceded by an empty line
| .url-box__connection-indicator { | ||
| display: inline-block; | ||
| opacity: .6; | ||
| transition: all .3s ease; |
There was a problem hiding this comment.
.3 should be written with a leading zero as 0.3
| padding: 0 10px; | ||
| width: 40px; | ||
| text-align: center; | ||
| & ~ .url-box__input { |
There was a problem hiding this comment.
Rule declaration should be preceded by an empty line
Selector url-box__input should be written in lowercase with hyphens
Unnecessary parent selector (&)
| border-color: var(--primary-color); | ||
| } | ||
| } | ||
| .url-box__input-prepend { | ||
| position: absolute; |
There was a problem hiding this comment.
Properties should be ordered align-items, bottom, display, left, padding, position, text-align, top, width, z-index
| border-color: var(--primary-color); | ||
| } | ||
| } | ||
| .url-box__input-prepend { |
There was a problem hiding this comment.
Selector url-box__input-prepend should be written in lowercase with hyphens
|
|
||
| &:hover { | ||
| border-color: var(--theme-off-border-color); | ||
| } | ||
| &:focus, &:active { | ||
| &:focus, &:active, &:focus-within { |
There was a problem hiding this comment.
Each selector in a comma sequence should be on its own single line
| @@ -2,7 +2,7 @@ $url-input-height: 50px; | |||
|
|
|||
| .url-box { | |||
| // max-width: 900px; | |||
| margin: 0 auto 20px; | |||
| margin: 0 auto 10px; | |||
There was a problem hiding this comment.
Properties should be ordered display, margin, width
| &:hover { | ||
| text-decoration: underline; | ||
| } | ||
| .response-time{ |
There was a problem hiding this comment.
Merge rule .response-time with rule on line 19
Opening curly brace { should be preceded by one space
| .response-status-code{ | ||
| background: rgba(var(--rgb-blue), .1); | ||
| } | ||
| .response-time{ | ||
| display: inline-block; | ||
| background: rgba(var(--rgb-black), .5); |
There was a problem hiding this comment.
.5 should be written with a leading zero as 0.5
| background: rgba(var(--rgb-cerise), .5); | ||
| // color: var(--green-color); | ||
| border-radius: 4px; | ||
| padding: 0 !important; |
There was a problem hiding this comment.
!important should not be used
Line should be indented with spaces, not tabs
|
|
||
| .fancy-input-content mark { | ||
| // background: var(--cerise-color); | ||
| background: rgba(var(--rgb-cerise), .5); |
There was a problem hiding this comment.
.5 should be written with a leading zero as 0.5
| } | ||
| } | ||
|
|
||
| .fancy-input-content mark { |
There was a problem hiding this comment.
Merge rule .fancy-input-content mark with rule on line 41
| position: relative !important; | ||
| // padding: 0; | ||
| border-radius: 0; | ||
| font: inherit; |
There was a problem hiding this comment.
Line should be indented with spaces, not tabs
| display: block !important; | ||
| position: relative !important; | ||
| // padding: 0; | ||
| border-radius: 0; |
There was a problem hiding this comment.
Line should be indented with spaces, not tabs
|
|
||
| .fancy-input-element { | ||
| display: block !important; | ||
| position: relative !important; |
There was a problem hiding this comment.
!important should not be used
Line should be indented with spaces, not tabs
| } | ||
|
|
||
| .fancy-input-element { | ||
| display: block !important; |
There was a problem hiding this comment.
!important should not be used
Line should be indented with spaces, not tabs
Properties should be ordered border-radius, display, font, overflow-x, overflow-y, position
| // word-wrap: break-word !important; | ||
| white-space: pre; | ||
| color: transparent !important; | ||
| overflow: hidden !important; |
There was a problem hiding this comment.
!important should not be used
Line should be indented with spaces, not tabs
No description provided.