Develop #4910
Develop #4910
Conversation
olga-wukelan
commented
Dec 7, 2024
olga-wukelan
commented
- DEMO LINK
- TEST REPORT LINK
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:
readme.mdsrc/index.htmlsrc/index.scsssrc/styles/card.scsssrc/styles/header.scsssrc/styles/index.scsssrc/styles/main.scsssrc/styles/navigation.scsssrc/styles/stars.scsssrc/styles/variables.scss
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.
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:
readme.mdsrc/index.htmlsrc/index.scsssrc/styles/card.scsssrc/styles/header.scsssrc/styles/index.scsssrc/styles/main.scsssrc/styles/navigation.scsssrc/styles/stars.scsssrc/styles/variables.scss
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.
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:
readme.mdsrc/index.htmlsrc/index.scsssrc/styles/card.scsssrc/styles/header.scsssrc/styles/index.scsssrc/styles/main.scsssrc/styles/navigation.scsssrc/styles/stars.scsssrc/styles/variables.scss
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.
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:
readme.mdsrc/index.htmlsrc/index.scsssrc/styles/card.scsssrc/styles/header.scsssrc/styles/index.scsssrc/styles/main.scsssrc/styles/navigation.scsssrc/styles/stars.scsssrc/styles/variables.scss
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.
src/index.html
Outdated
| </header> | ||
| <!-- #endregion --> | ||
|
|
||
| <div class="main"> |
There was a problem hiding this comment.
it should be main tag (the reason part of the tests failed)
src/index.html
Outdated
| </div> | ||
|
|
||
| <div class="card__value"> | ||
| <h1 class="card__price">Price:</h1> |
There was a problem hiding this comment.
h1 is not a proper tag for all text - use p instead
h1 is a title of the entire page
src/index.html
Outdated
| src="../src/images/imac.jpeg" | ||
| alt="imac" | ||
| /> | ||
| <h1 class="card__title"> |
There was a problem hiding this comment.
| <h1 class="card__title"> | |
| <h2 class="card__title"> |
h1 is a title of the entire page
| > | ||
| <img | ||
| class="card__image" | ||
| src="../src/images/imac.jpeg" |
src/index.html
Outdated
| src="../src/images/imac.jpeg" | ||
| alt="imac" | ||
| /> | ||
| <h1 class="card__title"> |
