[UI]: containers/modals should not show scrollbars

[L03TJ3]

Multiple containers and modals show a scrollbar even if there is nothing to scroll.

[L03TJ3]

@vldkhh you have screenshots of the places you noticed this?

@patpedrosa @decentralauren Priority?

[decentralauren]

@L03TJ3 can determine priority when @vldkhh adds screen shots (cannot tell without images whether it affects UX / ability to navigate significantly).

[vldkhh]

@decentralauren @L03TJ3 collective page:

stewards page:

donors page:

complete your donation modal :

Donation currency and frequency:

thank you modal:

[vldkhh]

and every single modal that could appears

[krisbitney]

Can you please upload the photos directly so I can see them, too? Which browser are you using?

Scrollbars should only appear if content doesn't fit. I see no scrollbars myself. It would help to have more information so I can reproduce the issue locally.

[decentralauren]

@vldkhh I'm also unable to see the images. Can you upload directly via the ticket in github (not zenhub) maybe that will solve it?

[vldkhh]

@decentralauren @krisbitney how about now?

collective page:

stewards page:

donors page:

complete your donation modal :

Donation currency and frequency:

thank you modal:

[krisbitney]

Yes, I see. Thanks. I'll look into it. Which browser are you using?

[vldkhh]

@krisbitney Chrome Version 121.0.6167.86 (Official Build) (64-bit), 1920x1080 resolution

[krisbitney]

I downloaded Chrome and tested it, but I can't replicate the issue locally.

[vldkhh]

@krisbitney Lewis also can't. I noticed that on small screens there is no such issue. Pat and on her MacBook looks fine, I believe Lauren also can't replicate it For sure it is something with my PC, but what exactly?

[krisbitney]

I'm not sure. It would be easier to find a solution if I could replicate the issue. I will do some googling soon and get back to you.

[L03TJ3]

@krisbitney overflow: auto

[krisbitney]

@L03TJ3 It's not supported in React Native. It might work on web, but you'll have to change it later for Android/iOS. Is that okay?

[L03TJ3]

@krisbitney thats fine, can just suppress with ts-ignore for now

(mvp won't have the android app release any way)

[krisbitney]

@vldkhh Can you please tell me if the problem persists in this deployment preview? https://good-collective-plvn065n3-gooddollarteam.vercel.app/

[vldkhh]

@krisbitney no, looks good

[L03TJ3]

Confirmed too so closing