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.

Sign up for GitHub

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

Jump to bottom

Checkbox, RadioButton, and spacing functionality/ux improvements #1346

Merged
merged 4 commits into from
May 15, 2017
Merged

Checkbox, RadioButton, and spacing functionality/ux improvements #1346

merged 4 commits into from
May 15, 2017

Conversation

maxnoy
Copy link
Contributor

@maxnoy maxnoy commented May 11, 2017

  • Checkbox description on the right
  • Checkbox indent
  • Improved spacing between description and widgets
  • Reduce spacing between radio buttons

@maxnoy maxnoy changed the title Checkbox descriptions are clickable and on the right, improved spacing for radios and descriptions Checkbox descriptions are clickable and on the right, improved spacing for radios and descriptions, radio widgets size is multiple of single widget, radio widget handling of zero options fixed May 12, 2017
@maxnoy maxnoy changed the title Checkbox descriptions are clickable and on the right, improved spacing for radios and descriptions, radio widgets size is multiple of single widget, radio widget handling of zero options fixed Checkbox, RadioButton, and spacing functionality/ux improvements May 12, 2017
@maxnoy
Copy link
Contributor Author

maxnoy commented May 12, 2017

Changes:

  • Checkbox descriptions are clickable and on the right
  • Reduced spacing between radiobutton options
  • RadioButton widget autosizes to a multiple of single-line widgets to still line up properly
  • Doubled margin between descriptions and widgets
  • Checkboxes and RadioButtons are now aligned properly vertically with other widget controls
  • RadioButton widget fix for when options are reset to empty list

@maxnoy
Copy link
Contributor Author

maxnoy commented May 12, 2017

Before
ipw1-before

After
ipw1-after

After -- showing horizontal alignment
ipw1-afteraligned

@jasongrout
Copy link
Member

Do you think this is ready to review/merge?

@jasongrout
Copy link
Member

Also, this fixes #237.

@jasongrout jasongrout mentioned this pull request May 12, 2017
Closed
@maxnoy
Copy link
Contributor Author

maxnoy commented May 12, 2017

@jasongrout, I believe so.

@jasongrout
Copy link
Member

Awesome! I won't be able to look at this again until Monday - @maartenbreddels or @SylvainCorlay, feel free to review and merge this before then if you want. Otherwise I'll take it back up Monday.

Thanks @maxnoy! It looks and functions way, way better.

@maxnoy
Copy link
Contributor Author

maxnoy commented May 12, 2017

Thank you again @jasongrout!!

@jasongrout
Copy link
Member

Tested this out, and it seems to work great. Thanks!

@jasongrout jasongrout merged commit 14bfd0b into jupyter-widgets:master May 15, 2017
@jasongrout jasongrout modified the milestone: 7.0 May 15, 2017
@maxnoy
Copy link
Contributor Author

maxnoy commented May 15, 2017

Thanks @jasongrout!!

@jasongrout jasongrout mentioned this pull request May 16, 2017
Closed
@jasongrout jasongrout mentioned this pull request Oct 4, 2017
Open
@github-actions github-actions bot added the resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label Feb 14, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion.
Projects
None yet
Milestone
7.0
Development

Successfully merging this pull request may close these issues.
2 participants
@maxnoy @jasongrout