Z slider bugfixes #303

will-moore · 2018-07-06T12:35:32Z

See https://trello.com/c/zXvu3QjV/202-bug-z-slider-bugs

To test:

Clicking Z / T increment buttons should update Z/T labels and the slider position.
Selecting the Z/T slider handle, then using the arrow keys to increment the slider should not nudge the selected panels in the figure.
Check other Z/T slider & label functionality is unchanged: Test images with different Z/T sizes and values.

jburel · 2018-07-25T14:26:59Z

src/js/views/right_panel_view.js

+            $("#vp_t_slider").slider({'value': theT + 1});
+            $("#vp_t_value").text(t_label + "/" + (sizeT || '-'));
+
+            console.log('deltaT', deltaT);


to be removed

jburel · 2018-07-25T14:31:47Z

Tested the changes
Everything works as expected.
Display of time using unit will need to be reviewed.
e.g. an image with 6 timepoints show 0 secs all the time.
This is outside the scope of this PR

will-moore added 2 commits July 6, 2018 11:31

Silence keyboard events from jQuery-UI $.slider() handles

aa6202e

Click Z/T increment buttons updates labels and sliders

959651c

jburel added this to the 4.0.1 milestone Jul 25, 2018

jburel reviewed Jul 25, 2018

View reviewed changes

Remove console

7958014

jburel merged commit 51be8dd into ome:master Jul 25, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Z slider bugfixes #303

Z slider bugfixes #303

will-moore commented Jul 6, 2018 •

edited

Loading

jburel Jul 25, 2018

jburel commented Jul 25, 2018

Z slider bugfixes #303

Z slider bugfixes #303

Conversation

will-moore commented Jul 6, 2018 • edited Loading

jburel Jul 25, 2018

Choose a reason for hiding this comment

jburel commented Jul 25, 2018

will-moore commented Jul 6, 2018 •

edited

Loading