Hexer• 3,660
@Phalcin
Posted
Great work.
Marked as helpful
1
Design comparison
Solution retrospective
Don't forget to tell me if something is wrong in my code or if i can improve my code and give your opinion/feedback.
Community feedback
Sönke Martens• 120
@Ribosom
Posted
Hi there, nice solution! And responsiveness is really nice!
Some advices:
- I think the boxes should have a white background
- The paragraph below the main heading, is too long. It is hard to read if lines are too long. You can see in the design, that it should be shorter. You can do something like
max-width: 55ch; - I think it would help, if you format your code html/css probably, so it is easier to read for yourself and others. (Using autoformatting of an IDE like vscode can help)
Marked as helpful
1
scarydeveloper• 130
@snake321
Posted
@Ribosom thank you so much for your reply next time I will not do this mistake
1
Account Deleted
Hi there 👋
Congratulate on finishing your project 🎉. You did a great job 💡
I give some suggestions to help you take your project design to the next level 📈😉
- It's not a big problem 🙂 but the Supervisor card should have a green border and Calculator blue
Happy coding ☕
Maqsud
Marked as helpful
1
scarydeveloper• 130
@snake321
Posted
@maqsudtolipov Thanks for your feedback
0
Please log in to post a comment
Log in with GitHub
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord