Skip to content
OverviewSolutions
7
Comments
4
Olalekan Bello
@OlaOluwalekan

All comments

  • Sankalpa Sarkarā€¢520
    @IllusiveCoder1101Submitted almost 2 years ago

    Hello everyone , I completed yet another project , It did Take me a while. Things I used to built this Project:- 1.React (Reason:- To handle state Management and also to organise my code.) 2.Sass (Reason:- To seperate out my css into different files.) 3.Vite (Reason:- Much faster than NPM) Things I will fix Later :-

    1. Accessibility :- There is an accessibility issue that you guys might encounter which is related to the Headings, I will fix that as soon as possible.
    2. First page issue:- There is currently an issue on the first page of the website where once u add the details , and move to the next page and comeback , the details will be gone, I probably have to add a "value" to my input fields, which I will do as soon as possible.

    If you guys find any other bugs/problems/code issues please feel free to inform me , your feedbacks are always welcome .

    Multi-Step-Form (React, Sass, Vite)

    1
    Olalekan Belloā€¢180
    @OlaOluwalekanPosted almost 2 years ago

    Hi!

    Viewing your first step, your app design was cool, but I can't view it further. I can't also move to subsequent steps. Also, I noticed the border of your steps is oval instead of a circle. Though I haven't used vite before, I would like to know how to use it. Overall nice try

    0
  • Blessing Oluwapelumiā€¢20
    @blessingOluwapelumiSubmitted over 2 years ago

    What did you find difficult while building the project? I was not able to use the arrow icon beside the main menu. I couldn't get how to make it change to the arrow down on hover.

    Which areas of your code are you unsure of? The media queries

    Do you have any questions about best practices? Please, how do I make use of the arrow down and arrow up icon and also make it interchange on HOVER?

    RESPONSIVE LANDING PAGE WITH DROP DOWN MENU USING HTML,CSS,AND JAVASCR

    2
    Olalekan Belloā€¢180
    @OlaOluwalekanPosted over 2 years ago

    Hi Blessing!

    I've gone through your work and it's such a nice one. I love your transitions - they are smoothly done. However, when I opened your live site, it takes time to load the main image. I don't know why. Also, on your GitHub link to the project, you don't have readme.md to hint visitors on what your work is about, the images you used, etc.

    But overall, its a nice one šŸ‘. Keep it up

    0
  • Olalekan Belloā€¢180
    @OlaOluwalekanPosted over 2 years ago

    Hi! Great work you have here. it's even better than mine

    One glitch though is that you didn't stick to the colours provided in the style-guide.md. It's always good to stick to clients' designs or colours though you can also make recommendations to them. And also, your image isn't properly closed.

    But overall, you're good to go. It's a very good job šŸ‘

    Marked as helpful
    0
  • Olalekan Belloā€¢180
    @OlaOluwalekanPosted over 2 years ago

    Nice try!

    However, you added to much padding into your qr-code-container class gene, there's little space for it to occupy. Also the border radius you added is somehow awkward. You could have used something like 10px for the qr-code-container and 5px for the image.

    Also, you didn't stick strictly to the styles in the style-guide.md

    Marked as helpful
    1