Ticket #1807 (closed: fixed)
Minor ICat Matters
| Reported by: | Sofia Antony | Owned by: | Jay Rainey |
|---|---|---|---|
| Priority: | minor | Milestone: | Release 3.0 |
| Component: | Framework | Keywords: | ICAT |
| Cc: | Blocked By: | #6457, #8126 | |
| Blocking: | Tester: | Arturs Bekasovs |
Description (last modified by Jay Rainey) (diff)
Suggestions by Russell Taylor for ICat interface improvements.
1) Maximising the data window leads it to take up the whole Mantid window and you can't get back to Mantid! It also pops up half off the screen for me. It probably needs a different parent window/widget set.
2) Progress reporting is missing from some algorithms. It might also be useful to have a 'pending' message in the applicable window while a search is going on.
3) It would be nice to be able to download and load in one shot (i.e. click).
4) The Load button on 'Investigation Data' should be greyed out if no data files have been downloaded.
Change History
comment:2 Changed 10 years ago by Nick Draper
- Milestone changed from Iteration 26 to Iteration 27
Bulk move of tickets to iteration 27, if your ticket is essential for Iteration 26 then move it back.
comment:3 Changed 10 years ago by Nick Draper
- Milestone changed from Iteration 27 to Iteration 28
Bulk move of tickets at the end of iteration 27
comment:4 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 28 to Iteration 29
Bulk move of tickets at the end of iteration 28
comment:5 Changed 9 years ago by Nick Draper
- Owner Sofia Antony deleted
Taken from Sofia Antony's list after she left
comment:6 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 29 to Iteration 30
"New" tickets moved at the code freeze of iteration 29
comment:7 Changed 9 years ago by Nick Draper
- Status changed from new to assigned
- Owner set to Anyone
comment:8 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 30 to Iteration 31
Bulk move of tickets to iteration 31 at the iteration 30 code freeze
comment:9 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 32 to Iteration 33
Moved to iteration 33 at iteration 32 code freeze
comment:10 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.1 to Release 2.2
Moved at end of release 2.1
comment:11 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.2 to Release 2.3
Moved at the end of release 2.2
comment:12 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.3 to Release 2.4
Moved to milestone 2.4
comment:13 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.4 to Release 2.5
Moved at the code freeze for release 2.4
comment:14 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.5 to Release 2.6
comment:18 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.6 to Backlog
Moved to backlog at the code freeze for R2.6
comment:19 Changed 7 years ago by Nick Draper
- Owner changed from Anyone to Jay Rainey
- Milestone changed from Backlog to Release 3.0
comment:20 Changed 7 years ago by Jay Rainey
- Description modified (diff)
When developing the new ICAT (#7637) interface I took these comments into account.
1) Maximising the data window leads it to take up the whole Mantid window and you can't get back to Mantid! It also pops up half off the screen for me. It probably needs a different parent window/widget set.
The new GUI appears in its own window on top of Mantid, and is a fixed size (which prevents full-screen mode, and improves UX).
2) Progress reporting is missing from some algorithms. It might also be useful to have a 'pending' message in the applicable window while a search is going on.
There is now a label on the GUI that informs the user that an "search is in progress". However, during maintenance I will improve the algorithms (functionality & error handling). I will also improve upon the progress reporting.
3) It would be nice to be able to download and load in one shot (i.e. click).
This has been implemented in the new GUI. If the user has access to archives the archives then when the user clicks "load" it will load it from the archives. If they do not have access and they press "load" then the file is downloaded and loaded into a workspace.
4) The Load button on 'Investigation Data' should be greyed out if no data files have been downloaded.
If no data files exist the data files frame is not shown. By default the download/load buttons are disabled (grayed out) in the data files frame until a user selects a data file(s) to download.
comment:22 Changed 7 years ago by Jay Rainey
- Status changed from new to verify
- Resolution set to fixed
- Blocked By 6457 added
comment:23 Changed 7 years ago by Jay Rainey
To verify that my changes have addressed the above issues open the ICAT GUI, and check that what I said above is valid.
comment:24 Changed 7 years ago by Jay Rainey
- Blocked By 8126 added
This should not be tested until blocking tickets have been fixed.
comment:30 Changed 7 years ago by Arturs Bekasovs
1) ICat is it's own window now, which is good. Ability to make it full-screen does no harm, not to mention it can be a huge benefit for small screen owners. It does look okay in full-screen mode, apart from result tables. They take a fixed == small amount of space, even if some of the column contents do not fit.
2) Haven't noticed any "freezed" states, when the user couldn't realize what is happening. Progress report could be improved for additional eye-candy, but it looks informative enough for me.
3) Load button is here, but is not functioning correctly now, throwing errors
4) Load button is grayed out when no files are selected. Gets re-enabled when user selects any. No surprises.
There are some 'Blocked By' tickets, which presumably should be the reason for 3) not working. When they are fixed, this one should be re-tested before it can be closed.
comment:31 Changed 7 years ago by Arturs Bekasovs
- Status changed from verify to verifying
- Tester set to Arturs Bekasovs
comment:32 Changed 7 years ago by Arturs Bekasovs
The problem with a load button wasn't solved. When you select a RAW file and try to load it, it throws an error. Looking at Debug log, I notice that the Load algorithm tries to load a wrong file (.txt file from the same investigation), so a cause of that is probably in archive searching algorithm.
This seems to be connected to the issue in #8209, so leaving this one till the other one is closed.
comment:33 Changed 7 years ago by Arturs Bekasovs
- Status changed from verifying to closed
Load functionality has been fixed and nothing else seems to be broken, hence closing this ticket.
comment:34 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 2654
