Skip to content

Conversation

@cha2hyun
Copy link
Collaborator

overflow-y-scroll' applies the same CSS properties as 'overflow-y-auto

overflow-y-scroll' applies the same CSS properties as 'overflow-y-auto
@onesine
Copy link
Owner

onesine commented Feb 24, 2023

Thanks @cha2hyun for this MR too. But if this applies the same CSS properties then why use it.

@cha2hyun
Copy link
Collaborator Author

cha2hyun commented Feb 25, 2023

Thanks @cha2hyun for this MR too. But if this applies the same CSS properties then why use it.

Hi there, The comment i wrote was a warning message from lint
image

Duplicate with the overflow-y option makes CssConflict

overflow-y-auto makes overflow-y: auto;

Use overflow-y-auto to allow vertical scrolling if needed.

overflow-y-scroll makes overflow-y: scroll;

Use overflow-y-scroll to allow vertical scrolling and always show scrollbars unless always-visible scrollbars are disabled by the operating system.

for my case overflow-y-auto works well 👍

@onesine
Copy link
Owner

onesine commented Feb 25, 2023

Oh yes, it's a duplicate. I didn't realize that. I thought there was overflow-y-scroll and this PR just replaces it with overflow-y-auto.

Thanks for this PR

@onesine onesine merged commit 47e3174 into onesine:master Feb 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants