Skip to content
OverviewSolutions
9
Comments
7
Yejin Han (한예진)
@Yejin-Han

All comments

  • Amjad Shadid•790
    @amjadsh97Submitted 8 months ago

    Dark mode: "Can you share best practices or recommended approaches for managing the theme in these projects?"

    Perfect Pixel: "In your experience, how important is achieving pixel-perfect design in web development projects? Are there situations where it's acceptable to deviate from perfect pixel alignment, and if so, what factors contribute to those decisions?"

    Component Organization: "When structuring and organizing components in a project, what strategies or best practices do you recommend? Are there specific patterns or architectural principles that can help maintain a clear and scalable component structure over time?"

    REST Countries API with color theme switcher solution

    2
    Yejin Han (한예진)•270
    @Yejin-HanPosted 3 months ago

    You did a great job! I felt that the completeness was high because even the effects such as hover, which were not specified in the style guide were all implemented. As a person who is still studying, I can't answer the requirements you wrote above, but I will briefly tell you the problems I've found.

    In a responsive design, elements touch the left and right ends at certain screen widths. It would be better if you could give a little margin around the contents. Also, the detail page doesn't bind the data to the Native name, and the text layout is sometimes broken. Furthermore, I think it would be good to separate the files for the main page and the detail page. And I suggest you to use Router to designate a route.

    Thank you! :)

    0
  • Bakhtiar•380
    @b4khtiarSubmitted 4 months ago
    What challenges did you encounter, and how did you overcome them?

    Trying to utilize inline form validation without using javascript. There's some problems that needs fix, like: if I use :invalid to mark invalid input, the input shows invalid at initial state, I use :user-invalid to fix that, but the :user-invalid does not initialize when user submit untouched input.

    Accessible mortgage calculator

    1
    Yejin Han (한예진)•270
    @Yejin-HanPosted 3 months ago

    First of all, you did a great job! I learned @component and provide/inject.

    What I've thought a little awkward was that both the error message shown by the input required attribute and set by you separately appear, so it makes uncomfortable to see. I don't know because of that or not, the error messages and the error styling came up a little late. I think it's better to let only one of error messages come out.

    Thank you. Happycoding! :)

    0
  • Yejin Han (한예진)•270
    @Yejin-HanPosted 3 months ago

    First of all, you really did a great job! :) To close the modal, not only clicking the 'Start New Order' button, but also clicking the outside part of the modal or pressing the ESC key. It was the part where I missed it while working on it so I really liked it. Also, it was interesting to use posthtml plugin, I've seen it for the first time. However, it doesn't seem to be compatible with the size of the tablet screen, so I think you'd better try it. Also, the rendered image looks the same across all screen widths.

    Marked as helpful
    0
  • c4todev•20
    @c4todevSubmitted 9 months ago

    Hello there! Making a results summary component is fun! I used React.js and Tailwind CSS for the project. Working with gradient colors in tailwind was challenging, but it was very instructive. I am really looking forward for feedbacks!

    Happy coding!

    Mobile first responsive component (React.js and Tailwind CSS)

    1
    Yejin Han (한예진)•270
    @Yejin-HanPosted 3 months ago

    Interesting work! I'm almost newbie to React so your code is very fun :) And also well done for trying tailwind css to make gradient!

    But I think you missed hover effect on Continue button. Please add the effect on it!

    Happy coding!

    0
  • tortaruga•450
    @tortarugaSubmitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    im happy with the final result, next time i'd try to make the js more DRY

    What challenges did you encounter, and how did you overcome them?

    i had some issues with the layout and making it look good on medium-sized screens, in the end i think it looks fine enough even though it's very simple and maybe not the prettiest

    What specific areas of your project would you like help with?

    any feedback as to if i'm doing a good job with accessibility and how to improve

    room homepage with bem and sass

    1
    Yejin Han (한예진)•270
    @Yejin-HanPosted 4 months ago

    First of all, thank you for your hard work. It must have taken you quite a while to re-layout the mid-size screen.

    I'll briefly tell you the problem I've checked. There is an error that the mobile version of the menu is maintained if you open the menu once in the mobile size and resize it to the desktop size. There are parts that are very slightly different in layout or size from the given design, but overall, it seems to have been completed well.

    In addition, it's amazing that you scripted the slide yourself! However, there are many parts that seem to have been scripted even if you don't have to use the script, so I'm worried that if there are more of these codes, there will be performance problems. It would be good to consider creating a class and toggle it.

    Thank you.

    Marked as helpful
    0
  • Yejin Han (한예진)•270
    @Yejin-HanPosted 4 months ago

    Your work is truly amazing! It's amazing that novice like me has used all of the CSS codes by component along with the various SASS grammar they couldn't use, increasing reuse and ease of maintenance. If it's a simple project, I often write code on a single file with ease, so I should emulate you. Also, I've never seen a way to implement a mobile menu without using javascript using an API called popover in HTML, so I think I'll be able to use it if it becomes more compatible in the future. I didn't give you any help, but on the contrary, it was a code that helped me a lot. Thank you

    0
  • Annalisa•200
    @Annalisa11Submitted 5 months ago
    What are you most proud of, and what would you do differently next time?

    I finished the project pretty quickly and although the hover effect for the image was a bit tricky I wasn't stuck for too long

    What challenges did you encounter, and how did you overcome them?

    I didn't have too many challenges except for getting used to write the css in a different way than usual :)

    What specific areas of your project would you like help with?

    I'd like to know if I implemented the BEM methodology right or if there is some aspect that is still not quite what it should be. Since it is my first time using BEM any feedback about it would be helpful

    BEM css NFT card component

    1
    Yejin Han (한예진)•270
    @Yejin-HanPosted 4 months ago

    Hello!

    First of all, I'm proud of you that you've tried hard to finish it! By using BEM, I can understand your code instantly. I think you've done well!

    But one thing, I learned that using 100vh in body or some area to make it full in screen is not suitable for mobile devices because of address bar, etc... How about considering it?

    Here are the links that you can refer to: https://medium.com/@alekswebnet/fix-mobile-100vh-bug-in-one-line-of-css-dynamic-viewport-units-in-action-102231e2ed56 https://dev.to/nirazanbasnet/dont-use-100vh-for-mobile-responsive-3o97

    Thank you.

    Marked as helpful
    0