Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Log in with GitHub
OverviewSolutions5Comments26

All comments

  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi , Great work implemented things well I just want to add something "Use cursor:pointer property " for links and buttons.

    Marked as helpful

    1
  • Nayana Bhat M• 90

    @nayanabhatm

    Submitted

    Hi there!,

    1. If I don’t specify the height in px, top: X% with relative position doesn’t work. Is there any way I can specify height in % instead of hardcoding the value?
    2. I have specified overflow: hidden but can still see the scrollbars when I resize the window.

    Profile Card using Flexbox

    3

    SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi @Nayana great work implemented things well there is no issue in website

    Keep Coding

    0
  • Juan Zapien• 95

    @1Zapien

    Submitted

    Any advice on how to approach the email error handling would be appreciated. Any other suggestions on what to improve would be greatly appreciated.

    Responsive landing page using flex

    2

    SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi nice work but there are a few problems

    1. There should be some margin between links in header as they are designed for mobile devices so they are not looking good at laptop screens.

    2. In testimonial section remove the overflow:scroll .

    Marked as helpful

    1
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi Brother Nice Work implemented things well But there are few problems

    1. Remove accessibility issues as well Html issues
    2. Your web is responsive great work
    3. Add transition effect on mobile nav as it appears just after clicking button
    4. Also add hover effect on mobile nav Hrefs.
    1
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi nice work

    1. Use overflow-x: hidden property to remove horizental scrollbar.

    Marked as helpful

    0
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi nice work There are few problems

    1. First of all use property overflow-x:hidden to avoid from horizentally scrolling
    2. Padding for your header links is also too much decrease it so that it may look pretier
    3. Add transition effect on hover
    0
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi nice work but there are some problems

    1. Firstly you have to check for responsiveness check after 580px(i am talking about mobile devices)
    2. Add transition effect on hover. Keep Coding
    0
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi nice work implemented things well Keep Coding And yes add transition property to hover

    0
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi Nice work implemented every thing well. Keep Coding

    0
  • Divyank Dubey• 60

    @dubeydivyank

    Submitted

    Hello Everyone! This is the first time I wrote html and css. It took a while to finish it. Please do let me know how it can be improved. What are some obvious things that I missed which could have helped improve the final outcome?? Eagerly waiting for responses! :) Thank You!

    Order-smry-card using flexbox media querry

    2

    SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi nice work implemented things well Keep Coding

    0
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi nice work there is only one problem and that is of main heading

    1. You can use div for main heading and can align it in center

    Marked as helpful

    1
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi nice work

    1. First of all remove accessibility issues
    2. Make anchor tag effective you can use hover effect 3.There are also responsiveness issues this webpage is only responsive for mobile screens , so try to make it responsive
    3. Also work on social icons in footer Keep Coding
    1
  • Heather Smith• 220

    @sorengrey

    Submitted

    This was one of those challenges where if I fixed something on desktop, it broke something on mobile, but if I fixed something on mobile, it broke something on desktop. But I think I got it straightened out in the end!

    Loved the color scheme on this one. If you can see any ways to improve this, please let me know. Thanks!

    Also, did anyone else have trouble with Chrome Devtools during this challenge? I could not get the zoom to cooperate. It would cut parts of the page off, and would only show cropped versions of the site. I ended up using What Is My Screen Resolution to get a better look at my whole page layout.

    Huddle Landing Page - HTML, CSS, Google Fonts, FontAwesome, Flexbox

    2

    SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi your webpage is not responsive it is just responsive for mobile screens Also

    1. use cursor:pointer property for button
    2. use anchor tag for social links
    0
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi, nice work implemented things well Your fonts are not working bcz you have not added link from google fonts library

    1. Just go to googlefonts.com
    2. Choose the fonts of your choice
    3. copy the link and paste it into header
    4. Finally you can use your fonts
    0
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi nice work implemented things well but there are few problems

    1. You have to set width and height of card as it seems stretched
    2. You have to make Cancel Button an anchor or you can simply use cursor : pointer

    Marked as helpful

    0
  • Topi Eskelinen• 25

    @topieskelinen

    Submitted

    Feedback appreciated! First challenge submission. Really just trying to figure out how to submit a solution, so some things might be a little all over the place.

    In this particular challenge I didn't feel the need to recreate the design 1-to-1 as I felt like it was more about nailing the visual elements than centering the content on the page.

    Responsive 3-column preview card component

    2

    SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    This comment was deleted

    0
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Good work implemented things well You have to set button to look it well as it's looking stretched For this either you can using padding or you can use height and width properties

    0
  • Traz Davids• 50

    @Royal-tea

    Submitted

    Hi guys, this is my very first challenge and I'm completely self taught. I'm looking for a way to make it more responsive. I was also wondering if any one has any pointers/feedback for my CSS and how to make it more concise

    Royal tea - Order summary component challenge

    3

    SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Great work implemented things well. You have to make it responsive for mobile devices also , it's responsive till 450px you have to work on it

    1
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Good Work implemented things well but you have to adjust the button by either you are using padding for that or you can use height an width also 2nd thing is you have to adjust the height and width of card as it's not looking well. Keep Coding

    Marked as helpful

    0
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Good work implemented everything well,

    Marked as helpful

    0
  • yamen• 190

    @YamenAlzeiter

    Submitted

    hello guys I just finish this challenge but I have some issues that I don't know how to fix

    1. in Firefox, the grid is broken, or it gives more width than it should be.
    2. when minimize the screen I can't see the first box, even when I set the display on the <body> to grid, but it work when I set it to block or inline. I really don't know what I am doing, It look easy but when I started I straggled.

    thank you guys for your help <3.

    solution using grids

    1

    SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hi Yamen implemented things well but you have to use a media qurie for screen of max width 900px you have to display grid-template-columns: 80% or like this

    1
  • SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Great work implemented things well There should be margin between annual plan and img

    Marked as helpful

    2
  • Marlon Passos• 940

    @MarlonPassos-git

    Submitted

    I particularly don't know how to improve this project even more, my biggest doubt is regarding the git-hub, there I use a gif for the best experience, but I don't know where I should put the .gif file Should I host the file? But I couldn't find a good free place to host big gifs without losing quality I see that some sizes I can't reproduce like the photo, there is a tool for those of us who don't have the pro to know the exact spacing size of each element

    Order Summary Component Main with HTML and CSS

    3

    SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    Hey, implemented everything well. But I noticed a problem in "Change" inside plan box when we hover it looks strange. Also instead of .card .plan__button { text-decoration: underline; } you can just use an anchor tag for that by which you don't have to use text-decoration property.

    Marked as helpful

    1
  • Alice• 245

    @AliceMenzie

    Submitted

    • card moves around on background, best way to make it fixed?

    FlexBox

    2

    SHAHAB MALIK• 345

    @Shahab-Malikk

    Posted

    @Alice implemented everything well. You have not added hover effects to buttons(Proceed Payment) also add hover effect for anchor tag(Change). And the last thing I cannot understand your question so Did you mean that when we change screen size card move?

    0