Skip to content
OverviewSolutions
4
Comments
65
Aleksandr Aksenov
@axevldk

All comments

  • Aleksandr Aksenov•750
    @axevldkPosted almost 4 years ago

    Hi, Raunak Jain ~ I have studied your work, and here are some of my opinions.

    1. You can update background image to match design exactly.

    2. For the social meta data on the card footer, I hope you to use flex rather than table. Flex and grid can replace table in these cases.

    3. Card top image can be set to background image ~

    Here is my solution to this challenge, hope this will help you even a bit, though it's not perfect.

    https://www.frontendmentor.io/solutions/profile-card-component-2n53OyvZb

    Happy coding ~ ✨

    1
  • Aleksandr Aksenov•750
    @axevldkPosted almost 4 years ago

    Hi, Moncles Junior Herisse ~ I have studied your work and here are some of my opinions.

    1. You used image tags for background, but I think that's not good. You can use background image style or pseudo elements in this case.

    2. You can add some padding to your card footer.

    3. Also card top image can be set to background-image.

    Here is link to my solution. Hope this will help you even a bit, though it's not perfect ~

    https://www.frontendmentor.io/solutions/profile-card-component-2n53OyvZb

    Happy coding ~ ✨

    0
  • Aleksandr Aksenov•750
    @axevldkPosted almost 4 years ago

    hi, @apocc, I have studied your work, and it's really great ~ Nothing to say, but 'Beautiful!' ~ Happy coding ~ 👍

    0
  • Augusto•130
    @Augusto248Submitted almost 4 years ago

    Hello everyone, I would like to know if something is right or something failed, any comment is very helpful, thank you!

    Social proof section

    2
    Aleksandr Aksenov•750
    @axevldkPosted almost 4 years ago

    Hi, Augusto ~ I have studied your work and good work so far. Here are some of my opinions.

    1. There is no background on the whole page.

    2. Star rating section is different from design. You can give margin-left to individual sections, or can use grid in this case. Also border radius and background-color is missing.

    3. And I think cards to go mobile mode too early. Hope it to maintain desktop vision until 769px or 993px.

    Hope this will help you even a bit. Happy coding ~ ✨

    1
  • Aleksandr Aksenov•750
    @axevldkPosted almost 4 years ago

    Hi, @FahrulZul ~ I have studied your work and it looks really great ~ 👍 Here are some of my opinions.

    1. I hope you to try to use BEM naming convention in this situation. It will help you naming classes.

    2. You can add max-width: 100% to your card component.

    Hope this will help you even a bit. Happy coding ~ ✨

    1
  • dan-o•10
    @dpress13Submitted almost 4 years ago

    This is my first successful dev practice project after teaching myself to code off and on since this past March :) Would love feedback on overall project in addition to responsive functionality between mobile and desktop. Also, any tips for Gulp 4 JS with regard to BrowserSync and the plugins in my gulpfile.js would be awesome. Appreciate any feedback in advance!

    Node / Gulp 4 SASS, PostCSS, Autoprefixer / Bootstrap 4 / BrowserSync

    2
    Aleksandr Aksenov•750
    @axevldkPosted almost 4 years ago

    Hi, @dpress13 ! I have studied your work, and it looks really great. Especially I like the way star review sections move. Other guys used to make it full width on smaller resolution, but you did it in other way. And it looks awesome.

    I am really surprised that this is your first one after teaching yourself. 👍

    Happy coding ~ ✨

    0
  • Aleksandr Aksenov•750
    @axevldkPosted almost 4 years ago

    Hi, @Nikola ~ I have studied your work, it really looks great ~ Good usage of BEM naming and also great on using grid.

    Keep it up ~ 👍✨😃🎉👏

    1
  • Aleksandr Aksenov•750
    @axevldkPosted almost 4 years ago

    Hi, @Stephen Dueñas ~ I have just studied your work, and it really looks great ~ 👍 Here is tiny opinion on your work.

    Cards container is not responsive from 1200px width resolution. I think that's because of fixed width of 275px from grid-template-columns: 1fr repeat(4, [col-start] 275px [col-end]) 1fr;. I am not sure how to change this style on your case, but it should be solved in my thought.

    Hope this will help you even a bit ~ Happy coding ~ ✨

    0
  • Elliot Case•295
    @ElliotCaseSubmitted almost 4 years ago

    Hi guys here is my attempt at the Todo app project,

    Any feedback is welcome!

    This project definitely took me longer than first expected especially setting up draggable functionality, but after a post on StackOverflow and this super helpful Youtube Video I managed to solve my issues! Let me know anything that I can improve on and ill make sure to apply it to my next projects

    Happy Coding!

    Todo app with theme switcher. HTML CSS Vanilla Javascript

    4
    Aleksandr Aksenov•750
    @axevldkPosted almost 4 years ago

    Hi, @Elliot Case ~ I have studied you work, and I think it's really great ~ 👍 Here are some of my opinions on your work.

    1. I hope you to try to use BEM naming convention for the list objects.

    2. There are some HTML and Accessibility errors as you can see on the report. As you said, they have to be get eliminated ~

    3. I think there was hover effect on list items - circle border gradient. That is also tricky part in this challenge. I hope you to solve that one.

    Anyway, overall looks great ~ Happy coding ~ ✨

    1
  • Aleksandr Aksenov•750
    @axevldkPosted almost 4 years ago

    Hi, @Omar Ahmed ~ I have studied your work, and it's really great ~ 👍

    Here are some of my tips.

    1. You can use ul instead of div for .advs elements. I think that would be more appropriate for semantic requirement.

    2. I think if you use BEM naming convention here, it would become greater.

    Hope this will help you even a bit ~ Happy coding ~ ✨

    0
  • Aleksandr Aksenov•750
    @axevldkPosted almost 4 years ago

    Hi, @wenadev ~ I have studied your work, and it's really great! 👍

    One thing I want to comment is to add email validation to your great form. That will make your form perfect.

    Anyway, overall looks wonderful! ✨ Happy coding ~

    0
  • Dave•5,295
    @dwhensonSubmitted almost 4 years ago

    I wanted to retry this challenge as it was the first one I attempted since really going back to basics. I think I did OK, I did it in less than half the time of my last attempt, and I think the code is DRYer and more organised. I chose this one as I want to watch how Kevin Powell does it on YouTube now...

    Any comments or suggestions are most welcome.

    Social proof - Second attempt

    2
    Aleksandr Aksenov•750
    @axevldkPosted almost 4 years ago

    Hi, @dave ! I have studied your work, and it's really great ~ 👍 Clean in code and perfect in using grid.

    I have learnt a lot from your code. Happy coding ~ ✨

    1