Skip to content

[AdvancedLayouts] Adds a width parameter to st.toggle and st.checkbox#11496

Merged
sfc-gh-lwilby merged 3 commits intodevelopfrom
feature/add-width-checkbox-toggle
Jun 5, 2025
Merged

[AdvancedLayouts] Adds a width parameter to st.toggle and st.checkbox#11496
sfc-gh-lwilby merged 3 commits intodevelopfrom
feature/add-width-checkbox-toggle

Conversation

@sfc-gh-lwilby
Copy link
Copy Markdown
Collaborator

Describe your changes

This PR is part of the [AdvancedLayouts] project and introduces a new width parameter to toggle and checkbox widgets.

Screenshot 2025-06-02 at 4 53 16 PM

Screenshot 2025-06-02 at 4 52 54 PM

GitHub Issue Link (if applicable)

#8197

Testing Plan

Unit Tests (JS and/or Python) ✅
E2E Tests ✅


Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

@snyk-io
Copy link
Copy Markdown
Contributor

snyk-io bot commented Jun 2, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 2, 2025

✅ PR preview is ready!

Name Link
📦 Wheel file https://core-previews.s3-us-west-2.amazonaws.com/pr-11496/streamlit-1.45.1-py3-none-any.whl
🕹️ Preview app pr-11496.streamlit.app (☁️ Deploy here if not accessible)

@sfc-gh-lwilby sfc-gh-lwilby added security-assessment-completed change:feature PR contains new feature or enhancement implementation impact:users PR changes affect end users labels Jun 2, 2025
@sfc-gh-lwilby sfc-gh-lwilby changed the title [WIP] [AdvancedLayouts] Adds a width parameter to st.toggle and st.checkbox [AdvancedLayouts] Adds a width parameter to st.toggle and st.checkbox Jun 3, 2025
@sfc-gh-lwilby sfc-gh-lwilby marked this pull request as ready for review June 3, 2025 13:04
Comment on lines +142 to +144
width : Width
The width of the widget. Can be an integer for pixel width,
or one of "content" or "stretch". Defaults to "content".
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: The docstring seems a bit different from the other ones.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Debbie will be making the docstrings better prior to release, these are just placeholders.

Copy link
Copy Markdown
Collaborator

@lukasmasuch lukasmasuch left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@sfc-gh-lwilby sfc-gh-lwilby merged commit 7cd8d0d into develop Jun 5, 2025
37 checks passed
@sfc-gh-lwilby sfc-gh-lwilby deleted the feature/add-width-checkbox-toggle branch June 5, 2025 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:feature PR contains new feature or enhancement implementation impact:users PR changes affect end users

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants