Ticket #9039 (closed: fixed)
Run list resize
Reported by: | Owen Arnold | Owned by: | Keith Brown |
---|---|---|---|
Priority: | critical | Milestone: | Release 3.2 |
Component: | Reflectometry | Keywords: | |
Cc: | Blocked By: | #9041, #9043 | |
Blocking: | #9170 | Tester: | Martyn Gigg |
Description
The following is a reasonable request by the users that we should get done
Somewhat minor, but a nuisance: the list of runs on the left does not scale to the maximum width of the titles, so it pops in an annoying scroll bar. Ideally we'd like to be able to resize the box once the list has been read in to the longest entry (sizehint), and be able to rescale it manually if required.
Change History
comment:2 Changed 7 years ago by Keith Brown
- Status changed from assigned to inprogress
Refs #9039 List is now resizable
The list now resizes itself to it's contents after searching.
There is also now a handle that can be used for manual resizing. Unfortunatly, it's not visible and you need to know where it is. It's the division between the list and the transfer button.
Changeset: e5acf49e478fedfacdb1c8a453c5544097240df1
comment:3 Changed 7 years ago by Keith Brown
- Blocked By 9041 added
(In #9041) As this ticket involved heavy changes to the UI file i would expect conflicts from other tickets if they were to attempt a merge, so until this has been tested and merged any other Reflectometry tickets can't be checkbuilt.
comment:4 Changed 7 years ago by Keith Brown
- Blocked By 9043 added
(In #9043) As this ticket involved heavy changes to the UI file i would expect conflicts from other tickets if they were to attempt a merge, so until this has been tested and merged any other Reflectometry tickets can't be checkbuilt.
comment:5 Changed 7 years ago by Keith Brown
- Blocking 9170 added
#9170 was had some big changes, so I need to be able to merge this in before i can put this up for testing
comment:6 Changed 7 years ago by Keith Brown
Refs #9039 merging in changes from previous tickets
Merge remote-tracking branch 'origin' into feature/9039_Refl_run_list_resize
Conflicts:
Code/Mantid/scripts/Interface/ui/reflectometer/refl_gui.py Code/Mantid/scripts/Interface/ui/reflectometer/refl_window.py Code/Mantid/scripts/Interface/ui/reflectometer/refl_window.ui
Changeset: fa81fa22637d3011cd456d074666face1c617a61
comment:7 Changed 7 years ago by Keith Brown
Refs #9039 Fix a small incomvienience with the table
Somehitng i thought i did in #9043 either wasn't done or was missed in the merge. Basically it's a little bit easier to edit a cell now as i've added another edit flag
Changeset: 52ab6d08d8ef7b7f1e7ea2cacf5fe1e61d7d25ab
comment:8 Changed 7 years ago by Keith Brown
To tester:
First make sure that all the functionality from #9043 and #9041 remains working.
The content of this ticket was basically allowing the listbox to be resized by means of a handle. It's not as visible as it'd like, at least on windows, but the handle is the division between the right edge of the listbox and the left edge of the transfer button
In the case of any style problems, note them, but still pass the ticket. #9170 will be doing a general clean-up of the code and already contains many improvements.
comment:9 Changed 7 years ago by Keith Brown
- Status changed from inprogress to verify
- Resolution set to fixed
comment:10 Changed 7 years ago by Martyn Gigg
- Status changed from verify to verifying
- Tester set to Martyn Gigg
comment:11 Changed 7 years ago by Martyn Gigg
- Status changed from verifying to closed
Resizing works (tested on Ubuntu 12.04), as does the other functionality.
comment:12 Changed 7 years ago by Martyn Gigg
Merge remote-tracking branch 'origin/feature/9039_Refl_run_list_resize'
Full changeset: 453e7f1dfb514e1e89af598d7245e20efab37a7a
comment:13 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 9882