Skip to content
OverviewSolutions
12
Comments
12
P
Ivan Orsolic
@ivanorsolic95

All comments

  • Ivan Orsolic•240
    @ivanorsolic95Posted 4 months ago

    Good solution.

    1
  • ikitamalarose•400
    @ikitamalaroseSubmitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    ...

    What challenges did you encounter, and how did you overcome them?

    Fetch the data and fill the html with JS

    What specific areas of your project would you like help with?

    On firefox the content is in the center but on chrome the block is up to the top. I would like to know how to solve this problem.

    Time tracking dashboard

    1
    Ivan Orsolic•240
    @ivanorsolic95Posted 4 months ago

    Nice solution!

    0
  • Ivan Orsolic•240
    @ivanorsolic95Posted 4 months ago

    Your solution doesnt match with the design file provided from Frontend Mentor.

    Here are coule things to notice: -error message should appear right above the input field, not under the field. Try to fix this using css property translate or any other property responsible for changing the position of HTML elements. -error message should appear only on bigger screens, not on the smaller screens such as mobile phone

    • when success message on smaller devices is being displayed, I noticed that gap between dismiss message button and rest of content is too big.
    Marked as helpful
    0
  • marcobaass•70
    @marcobaassSubmitted 5 months ago
    What are you most proud of, and what would you do differently next time?

    I build it completely with Flexbox. Looking back, a combination of Flexbox and Grid might have been easier. Other than that I had a lot of fun building it.

    What challenges did you encounter, and how did you overcome them?

    Making it responsive. Again, combining with grid might have helped here.

    What specific areas of your project would you like help with?

    Happy about any feedback.

    HTML, CSS, Flexbox, Javascript

    1
    Ivan Orsolic•240
    @ivanorsolic95Posted 5 months ago

    There are a couple issues with your solution. First of all, on a smaller screens, when you toggle and the social media icons are being displayed, I cannot see share button.

    When comes to bigger screens such as desktop, you have to much space under the div with class "profile-container".

    Also when I'm zomming in your component on desktop, it is not responsive, the content does not stay in it's position.

    Marked as helpful
    1
  • Hexerse•460
    @HexerseSubmitted 5 months ago
    What are you most proud of, and what would you do differently next time?

    I am not proud of this. I learned I really need to understand how to optimize my layout for reactive display more.

    What challenges did you encounter, and how did you overcome them?

    I wasn't able to understand many things like how to do a linear gradient for the colored photo.

    What specific areas of your project would you like help with?

    Layout and code optimization. I realised how bad my layout was and stopped doing the project and plan to tackle it again in the future. I did mistakes such as not using flex:1 etc. and other interactive features.

    I need some advice on how to control my layout easier.

    Meet landing page

    1
    Ivan Orsolic•240
    @ivanorsolic95Posted 5 months ago

    I like your solution.

    1
  • Ivan Orsolic•240
    @ivanorsolic95Posted 5 months ago

    I like your solution, it follows the best practices when comes to HTML and CSS

    1
  • Ivan Orsolic•240
    @ivanorsolic95Posted 5 months ago

    I like your solution with well-written code, also responsive on every screen size.

    0
  • Ivan Orsolic•240
    @ivanorsolic95Posted 5 months ago

    Your solution uses best practices when comes to Frontend Dvelopment. Nicely done!

    0
  • Ivan Orsolic•240
    @ivanorsolic95Posted 5 months ago

    I like your solution

    0
  • Ivan Orsolic•240
    @ivanorsolic95Posted 5 months ago

    I would suggest to remove color variables into separate CSS file, and then import this file to the main CSS file.

    ul { list-style: none; } Instead of using type selector, give this <ul> a class, then use this class to style the list.

    Marked as helpful
    0
  • Ivan Orsolic•240
    @ivanorsolic95Posted 5 months ago

    There is a grammatic error in the category name of blog ("Learining"). I like the responsiveness of your solution, but I would suggest to improve the names of the HTML classes, they should be more concise. Also write your own README, it is much better when the github repo has clear, well-written README file.

    0
  • Ivan Orsolic•240
    @ivanorsolic95Posted 5 months ago

    I noticed that your README file is missing, I would suggest to delete template README file, and write your own README, so the other developers can read about your solution, visit your solution by themselves, or see what technologies did you use to make this project.

    Marked as helpful
    1