Skip to content
OverviewSolutions
23
Comments
11
Marcin
@Frontmaniaac

All comments

  • Marcin•570
    @FrontmaniaacPosted over 3 years ago

    Hello! Everything looks great and the website is responsive. Three details that i have noticed:

    • Add cursor: pointer to your social icons
    • Add hover effects to all of the buttons
    • The "get started for free" button wraps text on small devices. You can change that by making the padding smaller

    Happy coding!

    Marked as helpful
    1
  • Marcin•570
    @FrontmaniaacPosted over 3 years ago

    Hello there! Only constructive critisizm here.

    1. On small devices the padding is too big making everything go into one worded columns on the "We're different" section.
    2. Around 830px width the font size is so small you can barely read it
    3. The footer and header font, is too small on all mobile devices.
    4. I would add transition for all the button hover effects. It's a small detail but makes a quite difference.
    5. There is no X button where opening the navbar. Despite from that, overall you did a good job! Keep up the good work and happy coding
    0
  • Sandro•345
    @SanidethSubmitted over 3 years ago

    Any feedback would be appreciated!

    P.S Input validation styling was very tough without js (this project required only css), so I tried with css only.

    Fylo dark theme with React and Sass

    2
    Marcin•570
    @FrontmaniaacPosted over 3 years ago

    Hello there :D Nice work on the project! You don't have to worry about the guidelines too much. You can always use Javascript or any framework you want. It's most important to acomplish the end effect of the website. Keep up the good work!

    1
  • Marcin•570
    @FrontmaniaacPosted over 3 years ago

    Nice work, I am having the same HTML issues. I wouldn't worry about that.

    1
  • Marcin•570
    @FrontmaniaacPosted almost 4 years ago

    Hey there :) It looks nice. Too bad it is not responsive for smaller devices starting at around 550px. Also the X doesn't go back to it's original form after closing. It shouldn't be to hard to fix! Happy coding :)

    2
  • Amol Shrivastava•80
    @Amol-ShrivastavaSubmitted almost 4 years ago

    Hey guys, I created this countdown app keeping in mind my need of aligning myself with my monthly targets. I used simple HTML, SCSS, JS to create it. If any one has any suggestion or improvement regarding this app, I'll be more than happy to know. Feel free to use my code as a reference.

    Responsive countdown for last day of every month using Html,SCSS,JS

    1
    Marcin•570
    @FrontmaniaacPosted almost 4 years ago

    Hey there :) Good job on the exercise, everything works as intended but sadly the responsivness does not work on any solution i tried. Apart from that nice work. Happy coding!

    1
  • Marcin•570
    @FrontmaniaacSubmitted almost 4 years ago

    I am really struggling with positioning those background images on different resolutions. I mean it's not hard to position them but the problem is them moving after resizing. Other than that the slider works good i think. Any tips on how I can improve?

    HTML + CSS + JS

    1
    Marcin•570
    @FrontmaniaacPosted almost 4 years ago

    Also don't look at the design comparision which doesn't update after changes in the code and updating the raport!

    0
  • Marcin•570
    @FrontmaniaacPosted almost 4 years ago

    When you link the path to your stylesheet remove the first "/" and it should work just fine. I had the same problem and it should fix that. So for example instead of

    <link rel="stylesheet" href="/public/styles/main.css"> write <link rel="stylesheet" href="public/styles/main.css">
    2
  • Marcin•570
    @FrontmaniaacPosted almost 4 years ago

    I would put the circle images under the card using z-index. Also it's not responsive at all ;(

    0
  • Marcin•570
    @FrontmaniaacSubmitted almost 4 years ago

    My solution on the pop up share menu is a bit different (on smaller devices) than the original but i am quite happy with it. What dou you think about it?

    Article preview component

    4
    Marcin•570
    @FrontmaniaacPosted almost 4 years ago

    Looks like the avatar is wider than in reality on the preview i don't know why.

    0
  • Marcin•570
    @FrontmaniaacPosted almost 4 years ago

    Hey! Good job on the project. I am wondering how you made that scrollbar popup after opening many answers? I've had trouble with that, making some of my answers impossible to see when i open too many of them.

    0