Anna Dominguez
@anna-dominguezAll comments
- @Jioh19Submitted over 1 year ago@anna-dominguezPosted over 1 year ago
Hi ! This is a good first project with HTML/CSS only π
Some differences that jump out at me from the two reference images:
- You must have forgotten to use the font given in the guide, as in the case of font colours
- I'm sure you could improve the margin and padding for the title and description
Have a good day!
0 - @Jioh19Submitted over 1 year ago@anna-dominguezPosted over 1 year ago
Hey ! I had never seen the picture tag used before, it helps a lot for media queries. Good job with only HTML and CSS ππ»
Things you could improve:
- Hover over the
add to cart
button - You should check out the
align...
to center your button content vertically - Try to delete the spaces for your
<h5>Perfume</h5>
and style it out with CSS withletter-spacing
Let me know if you have any doubt π
1 - Hover over the
- @Philip-DroubiSubmitted over 1 year ago
Hey everyone πββοΈ, this is my solution for
calculator-app-main
challenge. I have tried to take the challenge to another levelπ.What did I try to do?
- I have added some effects π.
- Making the calculator interactive with keyBoard, keypad, mouse, and touch devices β¨οΈ.
- Working on PWA (prograssive web application) to make it downloadable on Win/Mac/Linux/Android/IOS...etc, so you can use it offline as standAlone application π.
- Adding a button sound π.
- Copy to clipboard button π.
- Dinamic calculator screen π.
- A very good logicβοΈ.
I've tried to make the site as pixel perfect and responsive as possible without using paid designs π― and for doing this i used this google chrom extension : PerfectPixel
Please take a look and give me your feedback. Thanks :)
@anna-dominguezPosted over 1 year agoGood job! And a good idea for the copy button, it's a plus ππ» I've seen your GitHub repository for this challenge, you have done all the logic with the base of the project in HTML, why don't you try challenging yourself with frameworks? I've seen a lot of
cases
and I'm sure you could find a way with states in order to change your classesMarked as helpful1