minthing
@minthingAll comments
- @azzykesuma@minthing
Hello, Azzy! Nice Work! ✨
I'm not good at English too... but here is my suggestion!
when display size is around 1200px, the section part are not in center so i think it's better to give
width(like 200px)
.section
and givemargin: 0 auto
and.section > p{text-align:center}
and give
margin : 0 auto
to p under titleHappy Coding! :)
- @FabioAugustoDametto@minthing
Hello! Nice Work Fabio!
Here is my suggestion, if i do 2*0 = 0 and press any other number, then 0 doesn't dissapear like 02. butI think it's better to be 2, not 02.
- @gonzabaleta@minthing
The reason is that you only gave the border only when the button was hovering or it's display is inline-block. if you want to fix this with box-sizing, make sure that the button should be display:block; or it can't have height or width
nice work, Gonza ✨
- @Haldhardwivedi@minthing
Nice work! Haldhar ✨
Here's my opinion!
How about to use justify-content:center in wrapper class? (It makes set the element center easier)
And I think it’s better not give height:75% the 3 columns, it makes the box size change different
cheer! :)
- @Art-wdt@minthing
container is not centerd (if the display is bigger than 1552) how about use justify-content: center and remove margin:50px in wrapper class?
nice work! cheer! ✨
- @farildsen@minthing
if multiply 2 by 0 and then 0 still remains when i press another number key and i think it's better to add ellipsis and overflow:hidden when the number size gonna bigger than the result box
love your work, cheer!