Bassel || باسل
@BasselfathyAll comments
- @razzasidSubmitted 2 days ago@BasselfathyPosted 1 day ago
Good job. I see that you faced some issues for the table part, here is how you can fix it.
- first, you don't need table container, just use the
<table>
element directly and let it takes the full width. - let
<th>
and<td>
have 50% of the table width instead of the fixed padding.
Marked as helpful0 - first, you don't need table container, just use the
- @Pawan-arch589Submitted 9 days ago@BasselfathyPosted 1 day ago
Good job! you can use these hints to improve your CSS code_
-
When styling elements, it’s best to define reusable values, like colors, spacing, and font sizes, as CSS variables. This way, you only need to change a value once if you ever want to update it across the site!
-
Hardcoding sizes like px can make your design less adaptable to different screens. Instead, try using responsive units like
rem
orem
for things like font sizes, padding, and margins.
0 -
- @mohamed-fathy3010Submitted 13 days agoWhat are you most proud of, and what would you do differently next time?
Enhancing my understanding about BEM methodology and gaining more knowledge about best practices.
What challenges did you encounter, and how did you overcome them?Actually what seemed challenging to me was not the code itself but my lack of knowledge about Figma and how to easily access some of the needed info about the used styles specially with transitions.
What specific areas of your project would you like help with?Any feedback is appreciated.
- @fly0utWestSubmitted about 1 year ago@BasselfathyPosted 11 days ago
Links to solution and live page are broken...
Comparing to the original design, these parts need to be improved:
- h1 color
- text block sides padding
- card shadow
0