Code review comment for lp://staging/~artmello/webbrowser-app/webbrowser-app-bookmark_folders

Revision history for this message
Bill Filler (bfiller) wrote :

Good fixes, still seeing 2 problems:
1) pressing the Save button in the dialog doesn't close it on the device, it only dismisses the keyboard but doesn't close the dialog. You have to press it twice to make it go away

2) after pressing it a second time, the popover is shown but you see the selection in the list switching to the new folder name and it takes a while. I would expect the new folder name to already be selected in the list before the popover becomes visible

review: Needs Fixing

« Back to merge proposal