@Andrusik1
Latest comments
- @ZglobiulViu
You should use box-shadow instead of border for the items hover state. When you set that border of 3px your page will grow in dimension when you hover over a specific item. Also I wouldn't use innerHtml to read numeric values from the html elements instead i would use variables to set for example an incrementor like this: let numOfProducts=0; then create a function to either increment or decrement based on the clicked sign and then display numOfProducts on the page or just let that in two separate functions like you did there.Also instead of using parseFloat() method use Number("num") to convert from string to num or shorter put a + sign in front of the string representation of a number like this : +exOfstringNumber;
Marked as helpful - @Igwe0001@ZglobiulViu
in your body you don't need to set a specific width of 100% or height of 100%, instead use just your min height of 100vh, also you do not need to set the flex-direction to column.There would be more things to say, you abuse too much the position absolute and relative for unnecessary things.
- @kozlowskimarek@ZglobiulViu
I would get rid of display flex from your .card because it doesn't do anything there, also put the .attribution inside of a <footer> tag and place it outside the .container.
- P@ShanePinderDev@ZglobiulViu
You can check for my solution if you want. I made it with 106 lines of css. Also I saw in your code that you used !important. Try to not abuse on it and if you can don't use it at all. Learn more about specificity.
Marked as helpful - @purplejragon@ZglobiulViu
Try to use BEM naming convention.
- @brythnl@ZglobiulViu
your media query doesn't apply because you miss typed and you wrote max_width instead of max-width.
Marked as helpful
