Design comparison
Solution retrospective
I can't seem to get my javascript to work after much effort. Is there a way to fix it?
Community feedback
- @AdrianoEscarabotePosted almost 2 years ago
Hi gemeramargiela, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:
I noticed that you used a
button
in which case the best option would be ana
, because in my head when a person clicks on a button written READ MORE, he is not confirming a form, or something like, it will be redirected to another page, to Read More about!to solve this problem do this:
<a href="/" class="read-btn">READ MORE</a>
We have to make sure that all the content is contained in a reference region, designated with HTML5 reference elements.
native HTML5 reference elements:
<body> <header>This is the header</header> <nav>This is the nav</nav> <main>This is the main</main> <footer>This is the footer</footer> </body>
The rest is great!
I hope it helps... 👍
0 - @zeynabmvsPosted almost 2 years ago
use <a> tag for links in navigation and also for all the news titles and read more button.
when you click on the toggle button it refreshes the page and the nav-links are showing only for a second. change the following line in the index.html: <a class="toggle-button" href=""> to <a class="toggle-button">
it should fix the problem, let me know if it doesn't work, You can also check out my solution here: https://www.frontendmentor.io/solutions/responsive-mobile-first-news-home-page-using-grid-flexbox-sass-0qzsyCifGq
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord