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

Final design changes for 6.0 #1130

Merged
merged 10 commits into from
Feb 20, 2017
Merged

Conversation

ellisonbg
Copy link
Member

This does the final design changes for 6.0. Fixes #1110

Based on work by @cameronoelsen but modified.

@ellisonbg ellisonbg added this to the 6.0 milestone Feb 19, 2017
@ellisonbg
Copy link
Member Author

Here are screenshots:

Default slider state:

screen shot 2017-02-18 at 5 33 02 pm

Hover/active slider state:

screen shot 2017-02-18 at 5 33 10 pm

Default range slider state:

screen shot 2017-02-18 at 5 33 20 pm

Hover/active slider state:

screen shot 2017-02-18 at 5 33 28 pm

Progress bar:

screen shot 2017-02-18 at 5 33 36 pm

Active text area:

screen shot 2017-02-18 at 5 33 44 pm

Default button state:

screen shot 2017-02-18 at 5 33 52 pm

Hover button state:

screen shot 2017-02-18 at 5 33 58 pm

Press/active button state:

screen shot 2017-02-18 at 5 34 03 pm

Dropdown:

screen shot 2017-02-18 at 5 34 12 pm

Copy link
Member

@SylvainCorlay SylvainCorlay left a comment

Choose a reason for hiding this comment

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

This is truly wonderful! Some first remarks

  • The change of default value for the Button description should also be done in the js defaults.
  • The fact that this label is added by default breaks some of the examples in the Styling notebook, such as the carousel example.
  • The capitalization by default is very nice but makes the caption take more space, which will require the styling notebook to be adjusted.

Edit: I also read through the slider styling logic change and like it very much!

@ellisonbg
Copy link
Member Author

Button default fixed in ts. file, working on other updates now...

* Fixed bug in interactive that was auto-displaying the output.
  interact should auto-display, but interactive can't or its entire
  purpose doesn't work.
* Update examples to Matplotlib 2.0.
* Got all examples working again. Some API changes in skimage, etc.
* Using viridis in Lorentz example
* Shorten the button text in used in interact_manual.
* Updated VariableInspector to use new table styles.
* Other minor changes
@ellisonbg
Copy link
Member Author

  • Fixed bug in interactive that was auto-displaying the output.
    interact should auto-display, but interactive can't or its entire
    purpose doesn't work.
  • Update examples to Matplotlib 2.0.
  • Got all examples working again. Some API changes in skimage, etc.
  • Using viridis in Lorentz example
  • Shorten the button text in used in interact_manual.
  • Fixed button labels in Styling notebook
  • Fixed Variable inspector to use default table styling (not boostrap)

Should be ready to merge!

@SylvainCorlay
Copy link
Member

Thanks. I have minor changes to push on the top of this, but will do in a fixup PR.

@SylvainCorlay SylvainCorlay merged commit ffd3c15 into jupyter-widgets:master Feb 20, 2017
@SylvainCorlay
Copy link
Member

@ellisonbg I have merged, saved all the notebooks with the widget state and made another pre-release of jupyter-js-widgets to test on nbviewer and readthdocs.

I added a comment inline on a change that does not seem to be parsed. A consequence is that dropdowns don't render correctly on Readthedocs.

http://ipywidgets.readthedocs.io/en/latest/examples/Widget%20List.html

@@ -93,6 +95,7 @@
/* General Button Styling */

.jupyter-button {
outline: none !important;
Copy link
Member

Choose a reason for hiding this comment

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

This makes it impossible to see when the button is focused, which is a huge accessibility problem. Can we add some styling change for when the keyboard has focused the button?

}

.widget-select select {
outline: none !important;
Copy link
Member

Choose a reason for hiding this comment

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

Again, this is a huge accessibility problem. We need to have some way to signify the slider handle has focus for keyboard input.

Copy link
Member

Choose a reason for hiding this comment

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

(the problem here, I think, is that border is none below, so setting the border color in the focus doesn't help - we also need to set the border)

@ellisonbg
Copy link
Member Author

ellisonbg commented Feb 20, 2017 via email

@jasongrout
Copy link
Member

I think a lot of these styling changes look good, thanks!

I'm puzzled by the toggle button hover/active state, though - the button appears flat until I hover over it (so it's a bit undiscoverable as a button until I've happened to hover over it, which requires a lot of waving the mouse around and is impossible to do on mobile), and then when it is active, it appears to jump off the page even more. I usually think of buttons as being pressed down, so having an active (i.e., 'pressed') button floating above the page looks odd to me.

Can we (a) make the button look pressable (maybe with a shadow) by default, and (b) make the pressed state look lower than the unpressed state? I can try some possibilities out too.

@ellisonbg
Copy link
Member Author

ellisonbg commented Feb 20, 2017 via email

@ellisonbg
Copy link
Member Author

ellisonbg commented Feb 20, 2017 via email

@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 21, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 21, 2021
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.

UI review
3 participants