Ticket #9039 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

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:1 Changed 7 years ago by Nick Draper

  • Status changed from new to assigned

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

Note: See TracTickets for help on using tickets.