Skip to content
OverviewSolutions
5
Comments
4
Venky4c
@venky4c

All comments

  • Venky4c•195
    @venky4cPosted over 4 years ago

    Great Job! Few things are missing though: The 'copy' functionality, The center alignment in the mobile version for the bottom part of contents. The display of contents upon click of breadcrumbs button is novel and looks awesome.

    1
  • Venky4c•195
    @venky4cPosted over 4 years ago

    The desktop version is good. In mobile version the header image is missing. Also when we try to filter, the input filter gets cut off. I have tested it in Firefox, I am not sure if it is a browser issue. Functionality works fine. Great job!

    0
  • Venky4c•195
    @venky4cPosted over 4 years ago

    The solution looks neat - The UI is very close to the original. I could not go through code, as I am just about to begin the project. I could see that you have not implemented the functionality of being able to filter/click on the LHS like 'Front End Developer'. It was shown as a requirement under the 'Active' tabs. Not sure if you missed it, or left it by choice.

    1
  • Venky4c•195
    @venky4cPosted over 4 years ago

    Good job Ken! The solution is very close to the challenge. I have seen that you have made an improvisation by adding the github link also. However, I could not make out where the animations were used in the link.

    0