Skip to content
OverviewSolutions
33
Comments
33
P
AdamullaOsas
@AdamullaOsas

All comments

  • AdamullaOsas•800
    @AdamullaOsasPosted about 1 month ago

    Wow, your page looks very good, honestly nothing more to add

    1
  • Peter Godspower•640
    @TheBeyonder616Submitted about 2 months ago
    What are you most proud of, and what would you do differently next time?

    This is my first time working with API

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

    Coming up with the logic for the country border

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

    Any advice would do

    Interactive Country App: Built with JavaScript, Focused on Accessibili

    1
    AdamullaOsas•800
    @AdamullaOsasPosted about 1 month ago

    You're crazy writing this in html file hahahahah, I really like this little features (like this button moving you to the top or rainbow borders). Although you need to work a little bit on spacing elements and styling, but rest looks just fine

    Marked as helpful
    0
  • AdamullaOsas•800
    @AdamullaOsasPosted about 1 month ago

    you definitely should improve your styling and add your own validation, becaulse I can't put values like 5.25 inside interest rate. Also on mobile devices you shouldn't have these strips on the sides and it should take full screen

    0
  • AdamullaOsas•800
    @AdamullaOsasPosted about 2 months ago

    bro xd

    0
  • AdamullaOsas•800
    @AdamullaOsasPosted about 2 months ago

    Your page looks very similar but there are a few changes to make:

    • you're missing hover effects
    • you haven't added responsiveness to mobile devices
    • also you should center this overall score on left side
    Marked as helpful
    0
  • AdamullaOsas•800
    @AdamullaOsasPosted about 2 months ago

    good job, your page looks great

    0
  • AdamullaOsas•800
    @AdamullaOsasPosted about 2 months ago

    Your page looks good, but not perfect, the first problem I saw is your layout in home page, images are not perfectly fit, as they should. You can look up for mansory layout in google and get some help from there, or just assign pictures to their columns by some id or something. Also you dont have much hover effects and progress bar. I took my time to design mine, but the best way is to calculate value and assign it to variable and then use gradient to a div with position absolute right above footer.

    0
  • AdamullaOsas•800
    @AdamullaOsasPosted about 2 months ago

    You should mostly work on your sizing, page should take full width and height of page without white strips on the sides. Try applying background image on full size window, not the same with max-w-[xxx], becuse this is the reason of these white lines.

    Marked as helpful
    1
  • AdamullaOsas•800
    @AdamullaOsasPosted about 2 months ago

    bruh

    good job anyways

    0
  • AdamullaOsas•800
    @AdamullaOsasPosted 2 months ago

    good job, I really like your image slider, but you dont have a hover effect on 'Shop now" button. The rest looks just perfect ;)

    0
  • AdamullaOsas•800
    @AdamullaOsasPosted 2 months ago

    your page looks great, but I when I click on a hamburger icon on mobile view nothing happens. Also I've seen your code, and maybe you should try to avoid DRY code (don't repeat yourself) because e.g. you have 5 links with the same classes and it just doesn't look great

    0
  • AdamullaOsas•800
    @AdamullaOsasPosted 2 months ago

    Your coloring and styling looks pretty well, but there are some misplaced elements that could be fixed with one line of code align-items:center try using this in flexbox elements to align items horizontally (for example this avatar picture and text would be at the same height)

    0