[Webkit-unassigned] [Bug 72013] Not able to navigate the Resource tab options properly with arrow keys after adding the sticky-notes.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Nov 15 04:50:49 PST 2011
https://bugs.webkit.org/show_bug.cgi?id=72013
--- Comment #7 from Rosen Dash <rosen.dash at motorola.com> 2011-11-15 04:50:49 PST ---
(In reply to comment #5)
> > This is certainly related. Even if we give focus by mouse to "NoteTest" and press down arrow key, it skips "WebKitStickyNotes" and moves to "LocalStorage".
> > Similarly while the focus is on "LocalStorage" we press up arrow key the focus doesn't move any where.
> >
> > To check this, keep abases -> Note Test -> WebKitStickyNotes expanded.
> Ok, I can see how is that related now. But I can't see how this fix makes this any better, it is still not working for me.
>
> Looks like onpopulate is called when the node is expanded and while traversing the tree. I think this code should me moved to onexpand, since repopulating the node on down key press is not obvious way for the user to refresh table names list and the only other visible result is this issue.
While traversing the tree the method traverseNextTreeElement has the option in third argument to choose whether we want to populate the list or not. I think the re-population is desired so that any update in children list will be reflected and ResourcePanel and view will be in sync.
>
> > Secondly, I thought DatabaseQueryView taking focus is a feature as function "moveBackIfOutside" is called in "wasShown" function of "DatabaseQueryView" which takes the caret to prompt.
> This is probably a feature but not a good one. We should remove that.
>
> Also, please don't use resourcesLastSelectedItem setting like that, the default node should only be revealed on resources panel load, while table names populating can happen later in application lifecycle.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list