@TalkingBaboon
Latest solutions
Latest comments
- @cyberspatial
Beautifully implemented solution.
Your code has few issues!
- For the <a> tag you have mistakenly combined button and <a> tag together.
- U should use alternative text for the image in case the browser doesn't load the image. < img src="" alt="image-description-in-brief" >
Good Luck!
Marked as helpful - @Chesarfield@cyberspatial
Solution is implemented very well. Further I would like to add:
-
You have used section inside a div. instead u can use div. Section is used to separate the content on the page, But here you are using it to separate the content inside a div , semantically it can be improved by replacing section with div.
-
U have used heading even for a rate and annual plan. Instead of using <h4> for annual plan and rate u can use <span> or <p>.
-
You can remove the accessibility issues by changing <div class="container"> to <main class="container"> and <div class="attribution"> to <footer class="attribution">.
Good luck!
Marked as helpful -
- @RusuEduard@cyberspatial
Good solution. Few things I want to add
- For a Desktop site , you had used flex to center the content, but in the given design it was left aligned so u could use " text-align:left", U can also change using the flex-box if the flex-direction is row, then u can use "justify-content:flex-start", if the flex-direction is column then you can use align-items:flex-start.
- You could remove your accessibility issues by changing <div class="card"> to <main class="card"> and changing <div class="attribution"> to < footer class="attribution">
Marked as helpful - @matthewohreluy@cyberspatial
Beautifully implemented the solution.
You could remove your accessibility issues by changing <div class="container"> to <main class="container"> and by changing the <div class="attributes"> to <footer class="attributes ">.
Your image alternative text is blank, make sure to mention there something in case the image doesn't load.
Marked as helpful
