Latest solutions
REST Countries API with color theme switcher
#fetch#react#react-router#redux#sass/scssSubmitted 4 months ago
Latest comments
- @mahmoudAcmSubmitted over 1 year ago@KorneyChervonenkoPosted 2 months ago
I think you shouldn’t use "display: none" to hide titles in menu in mobile layout. It’s worth to read: https://stackoverflow.com/questions/62107074/how-to-hide-a-text-and-make-it-accessible-by-screen-reader/62109988
0 - @webdev1264Submitted over 2 years ago@KorneyChervonenkoPosted 2 months ago
I think you shouldn’t use "display: none" to hide titles in menu when @media (max-width: 768px). It’s worth to read: https://stackoverflow.com/questions/62107074/how-to-hide-a-text-and-make-it-accessible-by-screen-reader/62109988
0 - @adram3l3chSubmitted over 2 years ago@KorneyChervonenkoPosted 2 months ago
I think you shouldn’t use "display: none" to hide <div class="title">…</div> in menu when @media (max-width: 800px). It’s worth to read: https://stackoverflow.com/questions/62107074/how-to-hide-a-text-and-make-it-accessible-by-screen-reader/62109988
0 - P@CasperTheChildSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
yes
What challenges did you encounter, and how did you overcome them?no
What specific areas of your project would you like help with?Is my accessibility correct?
@KorneyChervonenkoPosted 3 months agoHello friend. This challenge can be solved without JS. Look at HTML DETAILS if you are interested. https://developer.mozilla.org/en-US/docs/Web/HTML/Element/details
Marked as helpful1 - @catherineisonlineSubmitted almost 3 years ago@KorneyChervonenkoPosted 4 months ago
HI! Your solution looks nice though there are a couple of things you can improve:
- Region selector provides only 5 options despite the fact that there are 8 of them in the country database
- Clicking on India in boundary countries for some reason derives information about the British Indian Ocean Territory instead of India.
- Clicking on the Back button just jumps to the homepage instead of returning to the previously viewed country.
0 - P@Marjolein-Kasman-de-JongSubmitted 10 months ago@KorneyChervonenkoPosted 10 months ago
Hello friend, instead of .addEventListener('click', () => {displayAdvice();}) why don't just do .addEventListener('click', displayAdvice) ?
0