Community
Participate
Working Groups
Forked from Bug 379237, comment 10: git-status2 use explorers to show staged and unstaged changes. Compare widgets render their actions in the parent actions div now. It works, but: - commands are visible all the time, even if the compare widget is collapsed - I use commandSpanId and set it to the parent actions div id. When I traverse the page using tab and enter the unstaged(staged) list and click tab again, the selection is changed to the first compare widget action instead of leaving the section. See the screenshot. - actions are not right aligned
I'll start with this as far as action alignment and helping with the hide/show actions goes. The traversal issue is a slightly bigger issue. Libing...what we need is that the compare widget can be passed the nav grid holder array so that when it renders commands, those commands get added to the traversal. What I don't know is the timing/life cycle of all this. The commands get rendered long after the row that will contain them get rendered. We'll probably need to talk about this.
(In reply to comment #0) > Forked from Bug 379237, comment 10: > > git-status2 use explorers to show staged and unstaged changes. Compare widgets > render their actions in the parent actions div now. It works, but: > - commands are visible all the time, even if the compare widget is collapsed now fixed. I had to add an innerspan for the compare widget actions and hide/show this span when the expand state changes. I couldn't simply hide/show the entire actions column because that affected the drawing of the line separator (shortened it when the actions column disappeared.) > - I use commandSpanId and set it to the parent actions div id. When I traverse > the page using tab and enter the unstaged(staged) list and click tab again, the > selection is changed to the first compare widget action instead of leaving the > section. See the screenshot. > - actions are not right aligned fixed. The explorer should style the actions column to be right justified.
Libing and I swapped some bugs this morning, he's going to look at the remaining traversal issue.
Fixed with http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=48cc193ae8245ebf634d1bc0ffe9df7aeaafa20e. You can now use key board on all compare commands.