
Akash Sharma
@aakash19090All comments
- @RocTanweer@aakash19090
Hi, @RocTanweer!
First of all brilliant work done with the subtle animations!😍
I appreciate your creativity still I would suggest few tweaks from my side:
-
the dashed border can be ignored on checkboxes and theme toggle button. Just an opinion!
-
You can transition the header image in same duration same way as the dark theme color toggles so that all transitions stay uniform.
Thanks it from my side. Overall well done!😀
-
- @RocTanweer@aakash19090
Hi, @RocTanweer!
First of all brilliant work done with the subtle animations!😍
I appreciate your creativity still I would suggest few tweaks from my side:
-
the dashed border can be ignored on checkboxes and theme toggle button. Just an opinion!
-
You can transition the header image in same duration same way as the dark theme color toggles so that all transitions stay uniform.
Thats it from my side. Overall well done!😀
-
- @Heitoluis@aakash19090
Hi @Heitoluis
Overall solutions is good. Layout is well built with flex box. Just make sure of always giving
alt
attribute in<img>
tags whenever you put an image. When the image doesn't get loaded the text inside the alt attribute will get displayed.