Sajal
@SJGit15All comments
- @AZZAM-K@SJGit15
Semantic HTML. Font-colors could use changing. Layout looks good on mobile. Looks similar to the design. Great.
- @GehadazzamWhat are you most proud of, and what would you do differently next time?
Handling grid and responsive
What specific areas of your project would you like help with?Text size
@SJGit15The layout looks good on mobile. The background-color for kira div has to be a different color. Semantic HTML is included.
Marked as helpful - @tchananet@SJGit15
The layout looks similar to the original. It looks good on mobile and desktop. H2 font size is a little too big.
- @DearNatthaphongWhat are you most proud of, and what would you do differently next time?
...
What challenges did you encounter, and how did you overcome them?...
What specific areas of your project would you like help with?...
@SJGit15This looks good on desktop and on mobile(375px). Some changes could be made such as the top padding and the font-weight of "add to cart".
Marked as helpful - @apostolovac@SJGit15
The layout looks pretty good. It includes semantic HTML. The solution does not look much different from the design. Good job.
- @NANDHINIMARIYAPPANWhat are you most proud of, and what would you do differently next time?
HTML CSS Form mobile-first workflow 📲📲 did my best to make it accessible 🦼
What challenges did you encounter, and how did you overcome them?Mobile view is a little difficult to set up.
What specific areas of your project would you like help with?if you notice anything or have suggestions, you know what to do
@SJGit15The layout looks good on desktop. Some changes could be made to this project such as the font-size and font-weight of the name, location, and links. Also, the size of the buttons.
Marked as helpful - @abdulbusybrainWhat are you most proud of, and what would you do differently next time?
Using display flex to center an item.
What challenges did you encounter, and how did you overcome them?Turning the project to mobile took me more than an hour to tweak less than 10 lines of css code
What specific areas of your project would you like help with?naming of classess
@SJGit15This layout looks good on desktop. Some changes could be made to this project such as the font size of "learning" and the font weight of "greg hooper"
Marked as helpful - @dynamic109What are you most proud of, and what would you do differently next time?
I was able to use git and github since i don't use it for most of my projects, and for sure i will be using it again
What challenges did you encounter, and how did you overcome them?I did not encounter any challenges
What specific areas of your project would you like help with?None for this project
@SJGit15This looks pretty good. One thing I would change would be the border-radius.
Marked as helpful