Skip to content
OverviewSolutions
9
Comments
7
Franklyn Okenwa
@franklynwisdom

All comments

  • Joshua Caleb Bolito•50
    @JCBolitoSubmitted about 2 years ago
    • Were the semantic tags used properly?
    • Is the component properly responsive to various device sizes?
    • What would you recommend in order to improve my code / output?

    Your feedback would be highly appreciated!

    Stats Preview Card Component

    4
    Franklyn Okenwa•260
    @franklynwisdomPosted about 2 years ago

    @JCBolito I just confirmed that you have made the necessary adjustments well done!

    0
  • Franklyn Okenwa•260
    @franklynwisdomPosted about 2 years ago

    Hi Maria,

    Kudos on completing the challenge, you really did a great job.

    The app is responsive on all screen sizes, however, it is not fully adapted to the mobile design when viewed from a mobile device.

    You can take a look at the mobile design file and try to make the app appear exactly as the mobile design.

    Well done so far.

    1
  • Joshua Caleb Bolito•50
    @JCBolitoSubmitted about 2 years ago
    • Were the semantic tags used properly?
    • Is the component properly responsive to various device sizes?
    • What would you recommend in order to improve my code / output?

    Your feedback would be highly appreciated!

    Stats Preview Card Component

    4
    Franklyn Okenwa•260
    @franklynwisdomPosted about 2 years ago

    Hi Joshua,

    Kudos to you on completing the project, however, from my observation: -You made good use of HTML semantics -The layout looks good on all screen sizes, however, the mobile design is been displayed on a laptop of size 1280px and above, kindly make the proper adjustment on that. -Yes, the code is well structured and readable.

    Well done!

    Marked as helpful
    1
  • Rasil•240
    @mzn-rasilSubmitted over 3 years ago

    This is my first frontend mentor challenge. Any kind of feedback will be helpful for me.

    Simple HTML and CSS

    2
    Franklyn Okenwa•260
    @franklynwisdomPosted over 3 years ago

    Well done Rasil, you did a very great job.

    Your solution looks great, it is responsive and I like the hover effects on the buttons.

    Keep it up.

    Marked as helpful
    0
  • Franklyn Okenwa•260
    @franklynwisdomPosted almost 4 years ago

    This is a very nice solution. Keep it Simon. The site is very responsive.

    1
  • Rahul Gupta•80
    @rahulxyzSubmitted about 4 years ago

    1. For active states of social media icons, I created 2 SVGs with different colors and switched them on hover. (a) Is there any way we can do it with CSS properties like "color" or anything? (b) Or any other better approach to it?

    2. In the articles section, for some size of the screen ( > 1024px i.e. for tablet view), one of the cards would increase in size as compared to others due to content length. For solving it, I have fixed the height for content and change it for different screen sizes. Is there any better approach to it?

    HTML, CSS(+SASS), JS

    3
    Franklyn Okenwa•260
    @franklynwisdomPosted about 4 years ago

    Hi Gupta, well done on your challenge.

    Yes, you can actually change the color of an svg by using this "filter: invert(22%) sepia(96%) saturate(5090%) hue-rotate(112 deg) brightness(96%) contrast(105%);" style.

    You can change the hex code of any color to the filter style by using this tool on codepen https://codepen.io/sosuke/pen/Pjoqqp

    0
  • Franklyn Okenwa•260
    @franklynwisdomPosted about 4 years ago

    I have made the site responsive for mobile. I targeted a device width of 375px. You can help to check it out. Thanks

    0