Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1719 support more styles to toggle buttons #1834

Conversation

Doodledun
Copy link

No description provided.


button_color: str or None
The color of each button. This should be a valid CSS color unit,
e.g. 'yellow', '#FFFF00', '255,255,0'.
Copy link
Member

Choose a reason for hiding this comment

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

The rgb color should be of the form rgb(255,255,0), right? https://developer.mozilla.org/en-US/docs/Web/CSS/color

Copy link
Author

Choose a reason for hiding this comment

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

Yes, you are totally right. Thanks for the catch! I will make updates accordingly.

@jasongrout
Copy link
Member

Thanks!

@jasongrout jasongrout added this to the 7.x milestone Nov 21, 2017
@jasongrout
Copy link
Member

For future reference, this is fixing #1719.

@jasongrout
Copy link
Member

Testing the examples given reveals that the color trait won't take something like 'rgb(255,255,0)', so I deleted that from the examples.

@jasongrout jasongrout merged commit 95b91f3 into jupyter-widgets:master Nov 22, 2017
@jasongrout
Copy link
Member

Thanks again!

@ankostis
Copy link

ankostis commented Mar 12, 2019

Is this supposed to be merged in ipywidgets-7.4.2 (jupyterlab-v0.35.4)?

>>> display(widgets.Button().style.keys)
'_model_module',
 '_model_module_version',
 '_model_name',
 '_view_count',
 '_view_module',
 '_view_module_version',
 '_view_name',
 'button_color',
 'font_weight']

>>> display(widgets.ToggleButton().style.keys)
['_model_module',
 '_model_module_version',
 '_model_name',
 '_view_count',
 '_view_module',
 '_view_module_version',
 '_view_name',
 'description_width']

@lock lock bot added the resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label May 21, 2020
@lock lock bot locked as resolved and limited conversation to collaborators May 21, 2020
@jasongrout
Copy link
Member

Is this supposed to be merged in ipywidgets-7.4.2 (jupyterlab-v0.35.4)?

This is answered in #3268 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants