-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
css bug in travel-list #44
Comments
Can you share your code using https://codesandbox.io/? I'll be able to help out once I see the code. |
|
You probably missed using |
@dowdy I just made the CSS class .list ul overflow: hidden, be sure that your className in App.js is "app" all lowercase |
There is a similar issue on the usepopcorn as well
[image: image.png]
…On Fri, 8 Mar 2024 at 15:45, arp690 ***@***.***> wrote:
I just made the CSS class .list ul overflow: hidden, be sure that your
className in App.js is "app" all lowercase
—
Reply to this email directly, view it on GitHub
<#44 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BAUQGL262V4IE6M4V3FRHGTYXHFJLAVCNFSM6AAAAABBJKYJD6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBVHAYTSNBTHA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***
com>
|
That is a scrollbar, I changed the container CSS style |
For me, I needed to add the app class to the App component as stated above but also the overflow: hidden in the .list ul... |
Hi, I'm following the tutorial on the Udemy. The css file gives me a large white space down there. Everything above the white space is html element. So I suspect
height: 100vh
doesn't work properly. But I'm not a css expert and cannot figure out how to fix it. I tried on Firefox but having the same issue.The text was updated successfully, but these errors were encountered: