Skip to content

Embedded Scrollbar support for Scrollable#2269

Merged
hecrj merged 2 commits intoiced-rs:masterfrom
dtzxporter:scrollbar-disabled
Jul 11, 2024
Merged

Embedded Scrollbar support for Scrollable#2269
hecrj merged 2 commits intoiced-rs:masterfrom
dtzxporter:scrollbar-disabled

Conversation

@dtzxporter
Copy link
Contributor

Disabled styles are used when a scrollbar is shown, but no content is overflowing.

@dtzxporter
Copy link
Contributor Author

CI Issues are unrelated to the PR.

Co-authored-by: dtzxporter <dtzxporter@users.noreply.github.com>
@hecrj hecrj force-pushed the scrollbar-disabled branch from cc804b0 to 8ae4e09 Compare July 11, 2024 06:01
Copy link
Member

@hecrj hecrj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

I ended up reimplementing the feature on top of master and made the scrollbars actually affect content layout when always shown.

@hecrj hecrj enabled auto-merge July 11, 2024 06:02
@hecrj hecrj added feature New feature or request widget layout addition labels Jul 11, 2024
@hecrj hecrj changed the title Add a configuration to scrollable to always show the scrollbar. Adds a disabled style for scrollbar. Embedded Scrollbar support for Scrollable Jul 11, 2024
@hecrj hecrj merged commit c63a81f into iced-rs:master Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments