Ticket #7815 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Change Label in SANS Data Analys Fit Trans

Reported by: Gesner Passos Owned by: Gesner Passos
Priority: trivial Milestone: Release 3.0
Component: SANS Keywords:
Cc: Blocked By:
Blocking: Tester: Peter Peterson

Description

The user tested #7733 and said:

Could I ask for a change to the drop-down box that currently says ‘Both’ or ‘Sample’, please.

‘Both’ is obvious, but ‘Sample’ – which brings up the separate sample and can transmission fit options – is I think less obvious. Users might think they will only be fitting the sample run.

‘Separate’ might be better than ‘Sample’

Change History

comment:1 Changed 7 years ago by Gesner Passos

  • Status changed from new to inprogress

Change label in SANS Interface

the user said that the previous label could cause confusion. Users might think they will only be fitting the sample run.

Hence, he suggested using Separate label.

re #7815

Changeset: ec5ecdf6191486ddc2a785cf9e9ea73f7fd48cc0

comment:2 Changed 7 years ago by Gesner Passos

  • Status changed from inprogress to verify
  • Resolution set to fixed

Tester:

it is a trivial change. The only side effect would be if trans_selector_opt used the text, but it is using only the index (currentIndex). So, it is sufficient to open SANS interface and check that instead of Sample, the label is Separate. (SANS interface -> Reduction Settings )

Click on combobox with 'Both' label. It will open two lines: Both, and Separate.

comment:3 Changed 7 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:4 Changed 7 years ago by Peter Peterson

  • Status changed from verifying to closed

comment:5 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8660

Note: See TracTickets for help on using tickets.