add solution #5786
add solution #5786
Conversation
There was a problem hiding this comment.
Sorry, the pull request review was unsuccessful for the following reasons:
Warning
There are no files to review. Please make sure that you have added files to the pull request. Some files and directories may be ignored from the review or failed to load.
[CODE: 4]
Additional info:
Ignored files list:
src/index.htmlsrc/style.css
If the issue is within your control, please address it.
The next review attempt will automatically launch in 1 hour.
If you believe this is an error or need assistance, please contact the support team.
There was a problem hiding this comment.
Demo links leads to 404 page.
Looks like wasn't any deploys for this task.
https://github.com/diachkinainna/layout_moyo-header/actions
src/style.css
Outdated
| margin-left: 20px; | ||
| } | ||
|
|
||
| a.is-active { |
There was a problem hiding this comment.
| a.is-active { | |
| .is-active { |
redundant a
src/style.css
Outdated
| color: var(--active-link-color); | ||
| } | ||
|
|
||
| a.is-active::after { |
There was a problem hiding this comment.
| a.is-active::after { | |
| .is-active::after { |
same
DEMO LINK
TEST REPORT LINK
[x ] Header height is set in 1 place (for the links)
[ x] Content is vertically centered (for any header height)
[ x] CSS is used to show all letters in Uppercase (don't type them in HTML)
[x ] Logo is an image wrapped with a link
[x ] CSS Variable is used for a blue color
[x ] Pseudo-element is used for a blue line below the active link
[ x] Code follows all the Code Style Rules ❗️