search

olawalecoder / Conference-Page

This website is for 5-days music festival. Interested participants can have an overview of what the music fest will be like and see a list of performing acts and artists.
https://olawalecoder.github.io/Conference-Page/
5 stars 0 forks source link

Morning session: peer-to-peer code reviews #2

Open VSeray opened 2 years ago

VSeray commented 2 years ago

Great job coming this far and finishing the project 👍 Here are some of the points that I believe would be good if you look at them.

gtekle commented 2 years ago

Well done! The layout of your code is easy to follow. But here are some points that I believe will help improve your project.

  1. In the ReadMe file, it is good if you could update the snapshot to reflect the current status and also if you could include both desktop version and mobile version snapshots. Besides, it would be good if you could add your name in the Author section of the same file.

  2. I think, it would be good if you could include the social-media and languages bar above the main nav bar. And on the nav bar it would good, if you could center align the menu items vertically as in the design. top-bar-design

  3. It would be good if you can include the black-white image that is behind the thumbnail image of the speakers. speaker-section-desktop speaker-section-mobile

  4. Finally, I believe the css file name can be replaced by something more descriptive.