Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use QValidators instead of popup boxes where possible for Muon Analysis user input #9693

Closed
PeterParker opened this issue Feb 3, 2014 · 1 comment
Closed

Use QValidators instead of popup boxes where possible for Muon Analysis user input #9693

PeterParker opened this issue Feb 3, 2014 · 1 comment
Labels
Low Priority Things that you don't ever want to be done. Muon Issues and pull requests related to muons
Milestone
Release 3.2

Comments

@PeterParker
Copy link
Contributor

For example, in the settings tab for Muon Analysis we allow the user to enter alphabetic characters in a lot of the fields, when only numeric characters make sense. The user is presented with a popup box in this situation.

This is by no means the end of the world, but elsewhere within Mantid (including the Home tab of the Muon Analysis interface itself) we use QValidators so that the user is forced to only use numeric characters.

This is a better way of doing it and it would be nice to keep a uniform approach as much as possible.
@PeterParker PeterParker added Low Priority Things that you don't ever want to be done. Muon Issues and pull requests related to muons labels Jun 3, 2015
@PeterParker PeterParker added this to the Release 3.2 milestone Jun 3, 2015
@PeterParker
Copy link
Contributor Author

This issue was originally trac ticket 8849

raquelalvarezbanos added a commit that referenced this issue Jun 5, 2015
@raquelalvarezbanos
Re #9693 Apply naming conventions
3060935
raquelalvarezbanos added a commit that referenced this issue Jun 5, 2015
@raquelalvarezbanos
Re #9693 Adding unit test
e25c6e6
raquelalvarezbanos added a commit that referenced this issue Jun 5, 2015
@raquelalvarezbanos
Re #9693 Throw exception when blocksize is negative or zero
e74e35a
raquelalvarezbanos added a commit that referenced this issue Jun 5, 2015
@raquelalvarezbanos
Re #9693 Use Algorithm manager in unit test
f144b05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Low Priority Things that you don't ever want to be done. Muon Issues and pull requests related to muons
Projects
None yet
Milestone
Release 3.2
Development

No branches or pull requests
1 participant
@PeterParker