Focus of url entry stays when closing a tab

[erikos]

Build: os11

Steps to reproduce:

• go into ebook mode

• open Browse

• open a new tab using the add-tab button

• close the new tab

• the url entry of the old tab is focused (you will know because the OSK is up)

[humitos]

Well this bug report is quite tricky. I'm not sure that the problem is related to the focus.

The Home Page (OLPC Library) has another bug (related with Javascript) that puts the focus in the Google input but it is not a "clean" focus. I say "clean" because the cursor blinking is not shown and "Backspace" does not work (#4200).

So, let's check with another page instead of the Home Page:

Test 1:

1. Open Browse
2. Go to ​http://www.google.com
3. Tap outside the search input
4. Open a new tab using add-tab button
5. Close the new tab
6. The OSK does not appears

Test 2:

1. Open Browse
2. Go to ​http://www.google.com
3. Open a new tab using add-tab button
4. Close the new tab
5. The OSK does appear

Test 3:

1. Open Browse
2. Go to ​http://www.google.com
3. Tap on the URL entry
4. Open a new tab using add-tab button
5. Close the new tab
6. The OSK does not appears

Are these behaviours right?

It's like the focus is always in the WebKit.WebView widget (on tab switching) and OSK only if this widget has the focus inside an input field.

[erikos]

Like discussed on irc, focus the canvas on tab close (see ff734de4e8829c9873fb84f0a9cc0166860cdd96 in sugar).

[humitos]

I'm attaching a patch that every time a tab is closed the focus is grabbed by WebKit.WebView of the tab that will be shown.

With this patch you won't be placed on the URL entry after closing a tab and you will be taken to the input field inside the WebView if you were on it at the moment of switching to another tab.

The only test that I couldn't fix is this one:

1. Open Browse
2. Go to Google
3. Focus goes to the search input inside the WebKit.WebView
4. Tap on the URL entry
5. Add a new tab with add-tab button
6. Focus goes to the URL entry
7. Close the new tab
8. Focus goes to the search input inside the WebKit.WebView

I would like that in this case the focus were in the WebKit.WebView but not in the search input because I've already quit from that input when I tap on the URL entry (step 4)

NOTE: this patch doesn't fix the issue related with the startup page (that case is out of the question) #4200

[manuq]

Working great, only fails when the prevvious tab has a PDF, the entry is left focused and there is an error in the log:

Traceback (most recent call last):
  File "/home/olpc/Activities/Browse.activity/webtoolbar.py", line 351, in __switch_page_cb
    self._connect_to_browser(tabbed_view.props.current_browser)
  File "/home/olpc/Activities/Browse.activity/webtoolbar.py", line 360, in _connect_to_browser
    address = self._browser.props.uri or self._browser.loading_uri
AttributeError: 'DummyBrowser' object has no attribute 'loading_uri'

[humitos]

Replying to manuq:

Working great, only fails when the prevvious tab has a PDF, the entry is left focused and there is an error in the log:

Great finding! This error has nothing to do with this patch. It's another issue. I filled a new ticket for this one here: #4231

[manuq]

Great, pushed as 0f656938 .

[greenfeld]

We restore focus to the previous tab as appropriate and do not leave the on-screen keyboard displaying attempting to type text in a field which no longer exists in Browse-148/OLPC 13.1.0 os20.

[dnarvaez]

Milestone 0.98 deleted