Ticket #6906 (closed: fixed)
Remove the mantid logo from QTI tables
Reported by: | Anders Markvardsen | Owned by: | Nick Draper |
---|---|---|---|
Priority: | critical | Milestone: | Release 2.6 |
Component: | MantidPlot | Keywords: | |
Cc: | james.lord@… | Blocked By: | |
Blocking: | Tester: | Russell Taylor |
Description (last modified by Nick Draper) (diff)
- Open a QTI table
- Insert some numbers
- Execute Table -> 'Convert to TableWorkspace'
you are now left which nearly identical tables, where it is very hard to tell which one is which.
Here the suggestion is to remove the Mantid logo from the QTI table to give a user a better change to know which one is which.
Attachments
Change History
comment:3 Changed 7 years ago by Nick Draper
- Status changed from new to assigned
- Owner set to Roman Tolchenov
comment:5 Changed 7 years ago by Roman Tolchenov
Set table icon to qtiplot table. Re #6906.
Changeset: 0572a9215327bda381e769ccd037d446444526b8
comment:6 Changed 7 years ago by Roman Tolchenov
Set table icon to qtiplot table. Re #6906.
Changeset: 0572a9215327bda381e769ccd037d446444526b8
comment:7 Changed 7 years ago by Roman Tolchenov
- Status changed from accepted to verify
- Resolution set to fixed
comment:8 Changed 7 years ago by Russell Taylor
Re #6906. Remove unused variable (gcc compiler warning).
Changeset: 64b9476dd9c01849df2be0ec39970249df1325ac
comment:9 Changed 7 years ago by Russell Taylor
Re #6906. Remove unused variable (gcc compiler warning).
Changeset: 64b9476dd9c01849df2be0ec39970249df1325ac
comment:10 Changed 7 years ago by Gesner Passos
- Status changed from verify to verifying
- Tester set to Gesner Passos
comment:11 Changed 7 years ago by Gesner Passos
- Status changed from verifying to reopened
- Resolution fixed deleted
this also changes the icon for new->table which it should not. See the attached image.
comment:13 Changed 7 years ago by Roman Tolchenov
Forgot to change back my testing code. Re #6906.
Changeset: 1266054f0b70f5d7101146385d49c71cee6f38d3
comment:14 Changed 7 years ago by Roman Tolchenov
Getting back missing pixmap. Re #6906.
Changeset: d7fde2aba5b961896b1fa42054b5193167dab93f
comment:15 Changed 7 years ago by Roman Tolchenov
Getting back missing pixmap. Re #6906.
Changeset: d7fde2aba5b961896b1fa42054b5193167dab93f
comment:16 Changed 7 years ago by Roman Tolchenov
- Status changed from accepted to verify
- Resolution set to fixed
comment:17 Changed 7 years ago by Martyn Gigg
- Status changed from verify to verifying
- Tester changed from Gesner Passos to Martyn Gigg
comment:18 Changed 7 years ago by Martyn Gigg
- Status changed from verifying to reopened
- Resolution fixed deleted
It looks like the change in comment 13, https://github.com/mantidproject/mantid/commit/1266054f0b70f5d7101146385d49c71cee6f38d3/, has gone to develop and not to the branch.
This will require a cherry-pick of the commit from develop and NOT a merge: git checkout feature/6906_remove_mantid_logo git cherry-pick 1266054f0b70f5d7101146385d49c71cee6f38d3 git check-build
then the ticket can be reclosed.
comment:19 Changed 7 years ago by Martyn Gigg
Merge branch 'feature/7000_function_property_type_string' into develop into 6856_ConvertToDiffractionMDWS_v2
comment:20 Changed 7 years ago by Russell Taylor
Merge branch 'feature/6906_remove_mantid_logo' into develop into 6856_ConvertToDiffractionMDWS_v2
comment:21 Changed 7 years ago by Roman Tolchenov
Merge branch 'feature/6906_remove_mantid_logo' into develop into 6856_ConvertToDiffractionMDWS_v2
comment:22 Changed 7 years ago by Roman Tolchenov
Merge branch 'feature/6906_remove_mantid_logo' into develop into 6856_ConvertToDiffractionMDWS_v2
comment:23 Changed 7 years ago by Martyn Gigg
Merge branch 'feature/7000_function_property_type_string' into develop into 6856_ConvertToDiffractionMDWS_v2
comment:24 Changed 7 years ago by Russell Taylor
Merge branch 'feature/6906_remove_mantid_logo' into develop into 6856_ConvertToDiffractionMDWS_v2
comment:25 Changed 7 years ago by Roman Tolchenov
Merge branch 'feature/6906_remove_mantid_logo' into develop into 6856_ConvertToDiffractionMDWS_v2
comment:26 Changed 7 years ago by Roman Tolchenov
Merge branch 'feature/6906_remove_mantid_logo' into develop into 6856_ConvertToDiffractionMDWS_v2
comment:27 Changed 7 years ago by Roman Tolchenov
Set table icon to qtiplot table. Re #6906.
Changeset: 104d1f44d0009c154af619bc095fe6fa6bf60b90
comment:28 Changed 7 years ago by Roman Tolchenov
Forgot to change back my testing code. Re #6906.
Changeset: f340e458c7093ee37dc40dc1c391ec6364a40c03
comment:30 Changed 7 years ago by Roman Tolchenov
- Status changed from accepted to verify
- Resolution set to fixed
comment:31 Changed 7 years ago by Stuart Campbell
- Status changed from verify to verifying
- Tester changed from Martyn Gigg to Stuart Campbell
comment:32 Changed 7 years ago by Stuart Campbell
One issue is that on the Mac there is no icon in the corner so both tables look identical.
comment:33 Changed 7 years ago by Nick Draper
- Status changed from verifying to closed
- Description modified (diff)
This has sat too long on Stuart's list.
Returned to the testing pool. If this is an improvement to most environments and does not harm others it should be passed.
comment:34 Changed 7 years ago by Nick Draper
- Status changed from closed to reopened
- Resolution fixed deleted
comment:35 Changed 7 years ago by Nick Draper
- Status changed from reopened to verify
- Owner changed from Roman Tolchenov to Nick Draper
- Resolution set to fixed
Closed by mistake, I meant to return it to the testing pool, I'll mark it as resolved again to get it into the right state.
comment:36 Changed 7 years ago by Russell Taylor
- Status changed from verify to verifying
- Tester changed from Stuart Campbell to Russell Taylor
comment:37 Changed 7 years ago by Russell Taylor
- Status changed from verifying to closed
Note that the icons appear next to the title on the Mac.
comment:38 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 7752