add task solution #4435
add task solution #4435
Conversation
|
I understand the problem with the text, how it can be corrected? |
There was a problem hiding this comment.
Great work! https://www.w3schools.com/css/css3_backgrounds.asp
The icon disappears on the focus. Consider fixing it
You can use Pixel perfect extension to compare your solution with Figma and pass tests
|
the magnifying glass doesn't disappear, but I don't understand where the error is with the text size |
|
ok) |
src/style.css
Outdated
| font-size: 14px; | ||
| border-radius: 4px; | ||
| border: 1px solid #e1e7ed; | ||
| box-sizing: border-box; |
There was a problem hiding this comment.
this property is better to define in * selector and remove everywhere else
| background-size: 11px 11px; | ||
| } | ||
|
|
||
| input:hover { |
There was a problem hiding this comment.
do not add styles using tag selector
Instead, add a common input class to both inputs and add all shared styles using this class (instead of repeating them in small-input and biginput)
src/style.css
Outdated
| outline: none; | ||
| color: inherit; | ||
| } | ||
| git add readme.md |
| box-shadow: 0 3px 8px 0 rgba(61, 78, 97, 0.2); | ||
| } | ||
|
|
||
| input:focus { |
There was a problem hiding this comment.
one of the reason tests fail might be overwritten by standard browser styles for placeholder
Add correct styles (font family - inherit, color, font weight) using pseudoelement:
.input::placeholder
https://developer.mozilla.org/en-US/docs/Web/CSS/::placeholder
No description provided.