search

yt-project / widgyts

Widgets for yt
https://widgyts.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
10 stars 10 forks source link

Update widget controls for use with GridBox #25

Closed munkm closed 5 years ago

munkm commented 5 years ago

I've modified the widget controls to use GridBox instead of VBox. Things look pretty similar to my old hacky way, but I had to hard set the nav controls box to be wider than the default (otherwise the controls got squished and the zoom slider was unreadable).

This is the old nav controls pane: image

this is the new one: image

This also fixes the @data-exp-lab/yt-tools versioning issue in package.json

munkm commented 5 years ago

@MatthewTurk I think maybe we should come up with a way of checking that the controls render nicely on different types of screens. Right now I'm looking at a notebook on my laptop, but I'm not sure on a different system how this might differ and it'd be nice to know that our widgets are controllable regardless of device.

matthewturk commented 5 years ago

I think using gridbox is a good way to make sure they render correctly, I think. So this is an improvement!