How i can improve HTML structure? How i can simplify CSS?
Ashraful Ferdous
@FarivAll comments
- @mrsev17Submitted about 1 year ago@FarivPosted about 1 year ago
Hi there,
I just saw your solution and the outcome is really good π
You want to know to simplify your CSS.
At first glance, I saw the /* Reset CSS */ section. And my instinct is just telling me, why not use
* { margin: 0; padding: 0; border: 0; font-size: 100%; font: inherit; vertical-align: baseline; box-sizing: border-box; }
So, I think, first and foremost, you may use above for resetting the CSS for all elements.
Also, you wish to know about your html structure.
I think, your html structure is ok, an expert person could give you more insights, I hope.
Regards, Ashraful
Marked as helpful0 - @ferassmohanaSubmitted about 1 year ago
If anyone has any comment about the code, the structure, specially JS code, don't hesitate telling me!
@FarivPosted about 1 year agoHi there,
I just saw your code and you did great so far! π
I just want to point out that you could give two more
else if
checks on month and day after year after line 105If you do that, 29/8/2023 will not give negative year.
Additionally, you may try to get more accurate width and height of the card if you use any image editor tools to get exact pixels of that.
Regards, Ashraful
Marked as helpful0 - @Amens11Submitted about 1 year ago
I am looking to get a feedback on my solution, please tell me how I can improve it.
@FarivPosted about 1 year agoHi there,
I just saw your code and preview.
You did great so far.
But There's surely room for improvement.
I suggest you add the image to the repository, so that image can be seen in Live Site.
Also you got to give a few margin to both left and right for mobile view.
Otherwise, all good so far!
Happy coding π
Regards, Ashraful
Marked as helpful0