Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • P

    @AdamullaOsas

    Submitted

    What are you most proud of, and what would you do differently next time?

    First project using react and tailwindcss. I think it went good and smoothly. Im proud of my theme and background change provider that is wrapped over my app

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

    Switch toggle, finally ended up using one from the internet

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

    Any help or suggestions to further improvement would be nice

    P
    Tsukimi 260

    @hikawi

    Posted

    Great solution.

    I love how you had trouble with the switch toggle too T_T, making a decent toggle that doesn't do the weird flickering on refresh wasn't as simple as I thought. I also like that you used a button group instead of a radio inputs, I should probably have done the same since I was kinda bad with styling radio inputs, and had to concoct weird hotkeys 1 to 4 to choose options instead.

    A problem I can see is that component pages are big chunks of html, like having HTML, CSS, Accessibility and Javascript files being duplicates of each other, and only different in the strings part. And on 320px screens the text are too big and get mushed together a lot also.

    0
  • P
    Dan Marius 625

    @danmlarsen

    Submitted

    What are you most proud of, and what would you do differently next time?

    I wanted to challenge myself by making all of the algorithms and string manipulation functions myself, without copy pasting from somewhere else.

    But of course in a "real" app i would probably use a npm package for generating/evaluating passwords.

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

    I appreciate any helpful feedback.

    P
    Tsukimi 260

    @hikawi

    Posted

    This is crazy! Exact to the pixels recreation of the design, it's insane! Used great semantic HTMLs, commented and easily comprehensible stylesheets and scripts, you're a professional.

    Only problem I can find is the strength tab, "too weak" gets squished and overflew on screen 320px, even though that's only 0.1% of the users according to some data websites.

    Marked as helpful

    1
  • P
    Tsukimi 260

    @hikawi

    Posted

    Hello there

    • First thing I noticed is clicking the tips first when loading in yielded an error, even though both fields look like they are filled correctly, this is probably a problem with styling placeholder at full opacity, instead of kinda transparent to show that they're just placeholders.
    • The number of people field is validated, but forgot the case where the input can be floating points, and the results still calculate as if nothing is wrong.
    • There are also no error states and active states so I had to double back to see if I was in the correct input box. Inputting bad values don't show any effects visible and only seen after clicking a tip button. Labels are also not used for input boxes.
    • Custom tip button doesn't work and just output NaN.
    • The mobile design doesn't completely fill the screen and leaves a different padding, even though this is a small mistake.
    0
  • Quang 350

    @progressive-newbie263

    Submitted

    What are you most proud of, and what would you do differently next time?

    Hello! It's me, JackFrost again.

    Today, after a few projects using tailwind, i had a much better understanding of it in comparison to a few days ago.

    This project went much smoother than what i expected.

    Overall this was a really entertained challenge to do, i had learnt a lot during completing it. Next time i'll try to do it even faster

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

    As a fun experience, i tried doing the dark/light theme button.

    It does function, but i am still confused about which part of the grid card's color should be changed so ended up leaving it from now.

    Also, I was struggled a while with the color assignment in tailwind config, as the customized color didn't get applied on to the JSON data files, so i ended up assigned those color onto input.css file.

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

    Maybe a few tips for the dark/light theme toggle button

    P
    Tsukimi 260

    @hikawi

    Posted

    Overall very great solution, I like the self challenge. I had problems with the grid card's color change too and just ended up color picking.

    The "Daily" or "Weekly" thing doesn't show up white if it's selected, if it's on Daily it shows "Last Day" instead of "Yesterday" (this is fine but it's funny), and the three dots don't become white when you hover. That's it from me.

    0
  • P
    Tsukimi 260

    @hikawi

    Posted

    There are some not too important styling mismatches but overall it works well on large screen sizes.

    On mobile it is very different and is just scaled down desktop version instead of the full screen dialogs in the design.

    Nice work, needs more work on the mobile part.

    0
  • Boxer 130

    @morshed-murad

    Submitted

    What are you most proud of, and what would you do differently next time?

    I will developed it in any framework of js

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

    I need to develop in react to improve my skill

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

    all

    P
    Tsukimi 260

    @hikawi

    Posted

    Couldn't see your solution code so can't really give my input on this, tho interesting way to use absolute positioning for the share popup from what I can see on inspect.

    0
  • P
    Tsukimi 260

    @hikawi

    Posted

    Pretty well done! :D

    Few issues I can see are the footer image isn't visible, the hero image isn't there on smaller screen sizes and the footer not being center on smaller screen sizes.

    0
  • P

    @raswonders

    Submitted

    What are you most proud of, and what would you do differently next time?

    Happy about setting explicit positions for grid elements via col and row lines. Next time i'll try use grid areas.

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

    It went quite smooth only stacking order of quotation mark made stuck for a second.

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

    figuring Accessibility / Responsiveness mainly but anything will do

    P
    Tsukimi 260

    @hikawi

    Posted

    Great result, so close to the design.

    It seems like we did similar things except that you also extract the quotes text to a separate file, while i just passed as props. Amazing and elegant work!

    1
  • P
    Tsukimi 260

    @hikawi

    Posted

    Very close to the design on the desktop version, great job on that! Great use of flexboxes!

    The issue is on mobile, the cards overflow the screen, and you can only view the middle two cards, also the titles are too big and get clipped on mobile.

    Marked as helpful

    1
  • P
    Tsukimi 260

    @hikawi

    Posted

    Wow I don't even know what to say ToT you completely destroyed it. Very well done.

    I also see the unit for css i didn't know about which is ch, is there any reason you went with this instead of rem or em?

    1
  • P
    Max 70

    @korniykom

    Submitted

    What are you most proud of, and what would you do differently next time?

    I'm going to start doing responsive design from the beginning. This time, I made the mobile design first, then tablet and desktop versions. I think it would more productive if I start doing responsive design for the component for all screen sizes simultaneously.

    P
    Tsukimi 260

    @hikawi

    Posted

    Your solution is very close to the design!

    Funny tho the ingredients part you forgot to apply the outfit font. A lot of the classes seem to be hard-coded values instead of using tailwind provided classes, like mb-[8px] can be mb-2.

    0
  • P
    Tsukimi 260

    @hikawi

    Posted

    This is amazing

    I think it'd be better if the image has alt text since it's not a decorative image, hovering over the links also doesn't change the text to black and the white text was difficult to see when hover.

    1