Skip to content
OverviewSolutions
11
Comments
11

AgilitiesZ

@phichayut-pak310 points

👋 Hi! I'm Pak. ‎‍💻 I'm interested in website development and planning to go on mobile development.‎‍ 💞️ I’m looking to collaborate on any website related work. 📫 Feel free to contact me via phichayut.somjai01@gmail.com

I’m currently learning...

NextJS, Python, Java, TypeScript, and Backend Development

Latest solutions

View more solutions

Latest comments

  • AgilitiesZ•310
    @phichayut-pak

    Hi! Congrats on your first project submission! 🎉🎉 Here are some suggestions from me:

    1. Check your image if it works when your website is deployed. From what I've seen in your code, you assigned the wrong path in <img> tag. How you can fix is to change it to <img src="./image-qr-code.png" alt="Qr Code">

    2. Sizing doesn't matter that much in here. But if you really want to make it challenging, it's easier for you to buy premium since you'll get an access to figma design which is more accurate than jpeg/png image.

    3. From your second question, I would recommend you implement JavaScript to make it interactive as you want user to hover, the qr code will scale up and background will get blurred.

    4. Try to make it to the center by making use of flexbox, it's one of the important features in css.

    5. Even though you're trying to make the qr code scales up on hover, but it might be a bit big. You might want to scale that down. Moreover, when user quits hovering the qr code, it immediately quits the scaling. You should try using transition on this.

    Happy Coding 👨‍💻👨‍💻

    Marked as helpful
  • AgilitiesZ•310
    @phichayut-pak

    Hey 👋 For your question, I think you're using margin-block-start and margin-block-end on the body making it moves and prevents it from having normal page. I would suggest you to use flex in div instead of body since it'll easily to see and it might fix this problem.

    Happy Coding!

    Marked as helpful
  • AgilitiesZ•310
    @phichayut-pak

    Congratulations on your fixed submission 🎉🎉 What I would suggest here is :

    1. Make the eye bigger ( the one that appears when you hover the image )
    2. change the font and font size of the title Equilibrium #3429
    3. Optional Do some transitions to make the website more alive

    Happy Coding!

  • AgilitiesZ•310
    @phichayut-pak

    Hi! I would say it's a great start for you! But first of all, congratulations on your project! 🎉🎉 Here are the things I would suggest :

    1. For your question, I fixed it by playing with opacity on hover. You might wanna try that out, or another way is to use group and group-hover from TailwindCSS ( it's a css framework that pretty much makes our styling quicker and easier )

    2. You might be using padding without noticing that the overflow on x axis appears. Take a deep look on this, it'll affect a lot when you're working with different designs.

    3. Font size and Font weight matter! You can choose whether you want to fix on this one, it's just for more detail.

    Keep studying and practicing! 🔥🔥

    Marked as helpful
  • AgilitiesZ•310
    @phichayut-pak

    Congratulations on your project 🎉🎉 What I would suggest is to take a deeper look at the font size on 0.041 ETH and 3 days left. This is the only one I can suggest since you've done a pretty good job on this. Keep going on!! 🔥🔥

  • AgilitiesZ•310
    @phichayut-pak

    Hey! First of all, Congratulations on your project 🎉🎉 I've done reading your code and I saw that you're reloading the website everytime users click on the dice button. So here are few suggestions from me:

    1. Don't reload website, since it might affect other things if you want to add something more.
    2. When users click the button, fetch the data then it'll automatically change your value. You don't need to reload it anymore!

    You can check mine if you want, Advice Project

    Good luck on your next project! 🔥🔥🔥

View more comments