Code review comment for lp:~uriboni/webbrowser-app/new-tab-wide-format

Revision history for this message
Olivier Tilloy (osomon) wrote :

The 'inBookmarksView' property should be marked readonly.

Ctrl+Tab works to switch between sections, Tab doesn’t, even though there seems to be code for handling this (is it bug #1481233 ?).

If I cancel a drag (by dropping it somewhere it’s not accepted), it seems the bookmarks list’s interactivity is not restored (I cannot click and drag to move up/down the list).

review: Needs Fixing

« Back to merge proposal