Bug 307031 - Cannot resize and reorder result columns in MAT using keyboard
Summary: Cannot resize and reorder result columns in MAT using keyboard
Status: RESOLVED WONTFIX
Alias: None
Product: MAT
Classification: Tools
Component: GUI (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Krum Tsvetkov CLA
QA Contact:
URL:
Whiteboard:
Keywords: accessibility
Depends on:
Blocks: 300655
  Show dependency tree
 
Reported: 2010-03-25 05:57 EDT by Krum Tsvetkov CLA
Modified: 2014-01-31 07:56 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Krum Tsvetkov CLA 2010-03-25 05:57:37 EDT
It is not possible to reorder and resize the columns of table and tree results in MAT using just the keyboard.
Comment 1 Krum Tsvetkov CLA 2010-03-25 06:07:50 EDT
The described ConfigureColumns class in Bug 38662 can help.
Comment 2 Krum Tsvetkov CLA 2010-03-25 06:16:48 EDT
I submitted some changes using the CompareColumns.forTree() and forTable() methods. Now the query context menu contains also a "Configure Columns..." entry next to the "Edit Filter" entry.
Comment 3 Andrew Johnson CLA 2010-03-31 03:48:28 EDT
This fix uses org.eclipse.jface.util.ConfigureColumns forTable() and forTree()
but ConfigureColumns was added in Eclipse 3.5.

In http://wiki.eclipse.org/index.php?title=MemoryAnalyzer/Contributor_Reference we say that Eclipse 3.4 is suitable, and some consumers of MAT such as IBM Support Assistant are Eclipse 3.4 based.
Comment 4 Krum Tsvetkov CLA 2010-03-31 04:02:49 EDT
Thanks for the hint. I also found this and was wondering if this restriction is OK.
If the IBM Support Assistent is 3.4. based, then I think the change is not OK. I have reopened the bug. We should probably search for an alternative solution then.
Any suggestions? Having in mind that we have to add some further context menu (see bug 307396) I suggest that we add
"Columns ..." at the bottom of the context menu, and then the necessary sub-nodes - "Edit filter", "Sort by ...", and "Configure columns...".
Back to configuring the size - shall we implement something on our own, or rather leave this out for the moment?
Comment 5 Krum Tsvetkov CLA 2010-04-07 06:15:27 EDT
I have for now reverted my changes which use the ConfigureColumns class. I left parts of the coding and marked the commented code with:
/* temporarily removed because of dependency on 3.5, see comments 3-5 in bug 307031 */
Comment 6 Andrew Johnson CLA 2010-09-16 09:22:37 EDT
Bug 322508 makes this unnecessary.