Skip to content

Use key as main identity for st.color_picker#12569

Merged
lukasmasuch merged 3 commits intodevelopfrom
stable-key-identity-for-color-picker
Sep 22, 2025
Merged

Use key as main identity for st.color_picker#12569
lukasmasuch merged 3 commits intodevelopfrom
stable-key-identity-for-color-picker

Conversation

@lukasmasuch
Copy link
Copy Markdown
Collaborator

Describe your changes

If a custom key is passed to any of the st.color_picker widgets it will be used as the main identity. This allows for dynamically changing other parameters in between reruns without causing the widgets to be recreated in the frontend.

GitHub Issue Link (if applicable)

Testing Plan

  • Added unit and 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 Sep 18, 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 Sep 18, 2025

✅ PR preview is ready!

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

@lukasmasuch lukasmasuch added security-assessment-completed change:feature PR contains new feature or enhancement implementation impact:users PR changes affect end users labels Sep 18, 2025
Copy link
Copy Markdown
Collaborator

@sfc-gh-lwilby sfc-gh-lwilby left a comment

Choose a reason for hiding this comment

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

LGTM

@lukasmasuch lukasmasuch merged commit 1c1e79b into develop Sep 22, 2025
38 checks passed
@lukasmasuch lukasmasuch deleted the stable-key-identity-for-color-picker branch September 22, 2025 11:58
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.

2 participants