Skip to content

Peer programming code review on my-portfolio project #5

Open
@LUCASFRANKINC

Description

@LUCASFRANKINC

Hello Faiz 🥇 ,

First of all, bravo 😮 🎆 on the design of the site. We can really acknowledge that way more than perfection has been applied on this project. We cannot assume the essence of good UI, perfect design and the hardwork displayed to bring the site to life.

Secondly, as a team, we have concluded that the design is working perfectly, though we would like to add on some few touches to better it 💯.

Having reviewed the content of your project and the UI, we have come up with strategic ideas that could in a way better the design and also the semantics of your projects. These are listed below ⬇️

  • Kindly consider implementing a div wrapper to hold the body of your site. This will help in aligning your content to implement flex and grid for easier positioning 💪🏽 .
  • Please try to use the semantic html tags such as section's, article's which will in turn reduce the number of div's that you apply in a certain block. This will ensure that your code is neat and self-explanatory ♻️ .
  • Please note that once you implement the required attribute on a form element, you really do not require to use a pattern on it. This is a good practice to let the html handle the validation unless you have a better way to validate 😀 .
  • Kindly apply the use of flex and grids over the position to ensure that the flow does not break down in different viewports. Please avoid hard-coded pixels and also if you have to implement positioning, please use percentages which will ensure that all the displays are visible as expected 🥂 .

Cheers and happy coding 😀 . From team!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions