Sahas saurav
@SahasSauravAll comments
- @MikeBish13Submitted almost 4 years ago@SahasSauravPosted over 3 years ago
great solution and pixel-perfect site ... Where did you learn scss any crash recommendation?
0 - @clowormSubmitted almost 4 years ago
Hello Everybody :)
I used this challenge as an opportunity to learn how to use React's
useContext
anduseReducer
together to manage the timer and theme states. Would love to hear any feedback!@SahasSauravPosted almost 4 years agogreat animation and responsive even i have also tried this challenge just check my solution link- https://next-pomodoro.vercel.app/ and please give feedback to me and my github repo https://github.com/SahasSaurav/next_pomodoro and even i also soved the state maagement with context api and useReducer just give a look to it
1 - @clowormSubmitted almost 4 years ago
Hi Everyone!
I put a lot of effort into getting the flip animation right. Let me know what you think!
I also added a bonus feature that lets you customize the date you're counting down to. Just pass in an ISO 8601 Date String into the date query param to use it.
Countdown to Christmas 2021: https://countdown.cloworm.vercel.app/?date=2021-12-25T05:00:00.000Z
Countdown to 2022: https://countdown.cloworm.vercel.app/?date=2022-01-01T05:00:00.000Z
@SahasSauravPosted almost 4 years agoHey, great job and really love the way you are using svg. Everything looks fabulous and responsive
1 - @MatthewKissingerSubmitted over 4 years ago
- With the desktop background image:
I was having difficulty having it only cover the bottom half of the page without repeating so I decided on having it cover the entire background starting at 380px. Any tips or hints to solve this problem would be appreciated.
@SahasSauravPosted over 4 years agouse background-size:contain background-repeat: no-repeat and background-position: bottom for background image
0 - @nikkuvSubmitted over 4 years ago
Any suggestions and feedback are welcome!
@SahasSauravPosted over 4 years agouse the max-width in form reduce the width in form element and decrease the line-height the h1 in larger screen
form{ max-width:550px }
0 - @solybarrSubmitted over 4 years ago
Hi I'm a little disappointed by this one, it was harder than I thought. I had trouble with the relative/absolute positioning of the Call to action that overlaps the footer. I couldn't center it the normal way with margin 0 auto because of the positioning. I couldn't size it correctly either. Thanks for any help.
@SahasSauravPosted over 4 years agothe reason behind the accessibility issue is each must be uniquely identity from other so use aria-label attribute and this aria-label attribute also help with screen reader how to describe about the link. <a href="www.facebook.com" aria-label="facebook">Facebook</a>
0 - @nikkuvSubmitted over 4 years ago
I'm improving a lot because of feedback !! so anyone who wants to give their feedback ...please!!
@SahasSauravPosted over 4 years agouse flexbox to align it in center both horizontal and vertical
0 - @muhammadkevinpfSubmitted over 4 years ago
Any feedback will be appreciated
@SahasSauravPosted over 4 years agousing aria_label to uniquely identify the each input and fix the image by using the object-fit:contain on img tags
0 - @eefborromeoSubmitted over 4 years ago
What is the best way to implement responsive images? I tried using
srcset
for this solution but I am a little unsure if I used it correctly or if there is a better way to implement responsive images.@SahasSauravPosted over 4 years agoYou see Kevin Powell video on YouTube for img srcset and you are prefectly img srcset https://youtu.be/2QYpkrX2N48
0 - @takamegSubmitted over 4 years ago
Not sure if this is what I am supposed to do with javascript in this challenge, although it's switching the contents. Any feedback/advice to improve my code is highly appreciated.
- @sarvagya2545Submitted over 4 years ago
This is my first time using scss! Please tell me what you think of my solution.
@SahasSauravPosted over 4 years agodesign is good but to get rid of accessibility issue use aria-label to uniqueidentify the each checkbox
1 - @hmnth07Submitted over 4 years ago
Open to feedbacks! :)
@SahasSauravPosted over 4 years agousing button type submit and in a link use aria-label attribute to uniquely identify the each link
0