labscript-suite-oldfinal1 / blacs

BLACS, part of the labscript suite, provides an interface to hardware used to control a buffered experiment. It manages a queue of shots to be run as well as providing manual control over devices between shots.
Other
1 stars 1 forks source link

Queue does not have horizontal scrollbar #11

Closed philipstarkey closed 9 years ago

philipstarkey commented 9 years ago

Original report (archived issue) by Shaun Johnstone (Bitbucket: shjohnst, GitHub: shjohnst).


The queue does not have a horizontal scrollbar, meaning that the size of the box has to be adjusted to see the end of long file names. This means that the queue ends up taking up much more space on the screen than it needs to, since typically the most important part of the file name (i.e. shot number within a sequence or repeat number) is at the end.

philipstarkey commented 9 years ago

Original comment by Philip Starkey (Bitbucket: pstarkey, GitHub: philipstarkey).


I don't think this is a bug. The visibility of the horizontal scrollbar should depend on whether anything in the treeview extends beyond the right border. If you resize the header of the treeview, I think the horizontal scrollbar will appear.

This is perhaps not great behaviour, and I think there may be some UI issues with it adjusting the horizontal scroll position automatically. If you can get the horizontal scroll bar to appear, could you post a new feature request issue for UI improvements for the Queue once you've identified annoying behaviour? (eg, save/load scrollbar position, header widths on restart. etc)

philipstarkey commented 9 years ago

Original comment by Chris Billington (Bitbucket: cbillington, GitHub: chrisjbillington).


There's a setting to say whether the last header should extend to the end, I think if we turn this on we should have desirable behaviour for all.

philipstarkey commented 9 years ago

Original comment by Chris Billington (Bitbucket: cbillington, GitHub: chrisjbillington).


Fixed in krb fork, pull request made.

philipstarkey commented 9 years ago

Original comment by Shaun Johnstone (Bitbucket: shjohnst, GitHub: shjohnst).


Fixes issue #11 by ensuring the queue treeview column expands to the size of the filenames in it.

→ \<\<cset 512da0c2f560ee7e8bd4451cdc9f43573a8c1c4d>>