@mprysak
Karol
@VillageR88All comments
- @VillageR88
The issue with body height still persists.
- @mprysak@VillageR88
Inside the stylesheet, change:
body { height: 100vh; }to
body { min-height: 100dvh; }Marked as helpful - P@BartoszMokrzyckiWhat are you most proud of, and what would you do differently next time?
I’d like to present my Quiz App project, which I built using HTML, SCSS, and JavaScript. This app allows users to select quiz subjects, answer questions, and receive immediate feedback on their answers. It also includes features like light/dark mode, a progress tracker, and a responsive design to ensure a smooth experience across devices.
If you have any feedback, suggestions, or ideas for new features, I would greatly appreciate it! Every piece of feedback is valuable and will help me grow as a developer. Thank you for checking out my project! 😊
@VillageR88Great effort! I'm currently on my phone, but I noticed an issue with the color transition.
Marked as helpful - @313raduWhat are you most proud of, and what would you do differently next time?
🔥 🔥 🔥
@VillageR88313radu, thumbs up for your effort in taking a simple challenge and elevating it to the next dimension—literally! 😆
But once again, the same responsive issues persist!
Marked as helpful - @kamilkoziol5@VillageR88
Kamil, I've been following your solutions since the first one, and visually, they are pure gems. Your UI/UX background is evident, making your attention to layout pure gold. However, as you dive deeper into challenges, you'll inevitably face more than just layout-based issues.
Your form element is missing labels, input IDs, and autocomplete. The input with the query should be a radio button with a proper name attribute. Also, the action doesn’t pass any information to an endpoint, but I assume this was more about the mockup.
- @313radu@VillageR88
Great effort, 313radu! You need to adjust the responsive layout, as some areas are inaccessible. Also, check for console leftovers and a meta issue.
Marked as helpful - @YacoubDweik@VillageR88
You're ready for the job market!
Just be sure to check the console for any errors.
Marked as helpful - P@sophia-see@VillageR88
Great effort! I recommend refactoring it for larger screens, as it currently appears overly stretched without clear boundaries.
Marked as helpful - P@sophia-see@VillageR88
Great job overall! 👏 However, on larger monitors, the layout feels a bit off. It would be worth adjusting the design for wider screens to maintain a polished look. Keep it up!
Marked as helpful - P@BartoszMokrzyckiWhat specific areas of your project would you like help with?
Hello, I am presenting my Solar System project. I encountered the following issues:
Data Synchronization Between Overview, Structure, and Surface When Changing Resolution (Desktop to Mobile): I am facing issues with keeping the data for Overview, Structure, and Surface in sync when the screen resolution changes, such as from desktop to mobile.
Any help or suggestions would be greatly appreciated!
@VillageR88Great job, Bartosz! If I were to tackle this challenge again, I’d opt for 100% SSG and prefetch all routes with images for instant loading and smoother navigation.
- @YacoubDweik@VillageR88
Interesting approach! I’d suggest keeping the max width at 1440px to avoid excessive head movement on larger monitors.
Marked as helpful - @xaoccc@VillageR88
Hi, I'm currently on my phone and noticed an issue with horizontal screen overflow. You might want to check on this.
- @ronilucylucyWhat challenges did you encounter, and how did you overcome them?
tbh responsiveness, i struggle a lot with fckn css.
What specific areas of your project would you like help with?oh. probably responsiveness. i need really help what i am doing wrong :D
@VillageR88I did this challenge again recently. Maybe you can check it out.
- @cendyz@VillageR88
I did quick resolve on your sollution
* { transition: background-color 150ms, color 150ms, border-color 150ms; }and I think it looks awesome.
Marked as helpful - @cendyz@VillageR88
This is really nice! Maybe a subtle transition effect for the theme change could make it smoother.
There are also some errors in the console.
Marked as helpful - P@BartoszMokrzycki@VillageR88
Hi Bartosz,
Great job! Here are some minor issues to address:
- missing main element
- incorrect use of <label for=FORM_ELEMENT>
- missing meta description
Keep up the good work!
Marked as helpful - @ronilucylucyWhat challenges did you encounter, and how did you overcome them?
tbh responsiveness, i struggle a lot with fckn css.
What specific areas of your project would you like help with?oh. probably responsiveness. i need really help what i am doing wrong :D
@VillageR88Hi,
I’m checking your solution on my phone right now, and I’ll provide more detailed feedback once I’m on my PC later. For now, I can say that the solution feels a bit overcomplicated and could benefit from some refactoring.
On mobile, some areas are going beyond the screen. A good starting point would be setting min-height: 100dvh to ensure proper scaling, then building from there. Additionally, this solution doesn’t seem to require JavaScript, as it adds an unnecessary layer of complexity.
Looking forward to diving deeper into your work soon!
Marked as helpful - P@NDK1195@VillageR88
- missing favicon.
- missing meta description.
- missing at least one <main> element.
Good job!
Marked as helpful
