[Bugs] #4200 Browse NORM: OSK Backspace does not work on the OLPC home page when not focusing the entry explicitly

Sugar Labs Bugs bugtracker-noreply at sugarlabs.org
Wed Nov 14 02:21:14 EST 2012


#4200: OSK Backspace does not work on the OLPC home page when not focusing the
entry explicitly
------------------------+---------------------------------------------------
    Reporter:  humitos  |          Owner:  humitos 
        Type:  defect   |         Status:  accepted
    Priority:  Normal   |      Milestone:  0.98    
   Component:  Browse   |        Version:  0.97.x  
    Severity:  Major    |       Keywords:          
Distribution:  OLPC     |   Status_field:  Assigned
------------------------+---------------------------------------------------
Changes (by erikos):

  * status_field:  Unconfirmed => Assigned


Comment:

 Ok, I have a test case that gets things more clear:

 - in laptop mode

 - open a new Browse instance

 - load google.com

 - close Browse instance

 - resume it

 - type with your hw keyboard

 ---> the google entry does not receive any characters, visually it looks
 unfocused

 The issue we have is that the webview is not focused. Here is a simple
 patch that does fix the above test case.

 {{{
 diff --git a/webactivity.py b/webactivity.py
 index d140128..e9847d4 100644
 --- a/webactivity.py
 +++ b/webactivity.py
 @@ -371,6 +371,7 @@ class WebActivity(activity.Activity):
              for number, tab in enumerate(self.model.data['currents']):
                  tab_page = self._tabbed_view.get_nth_page(number)
                  tab_page.browser.set_history_index(tab['history_index'])
 +                tab_page.browser.grab_focus()

 self._tabbed_view.set_current_page(self.model.data['current_tab'])
          elif self.metadata['mime_type'] == 'text/uri-list':
 }}}

 This does make the same est case work as well in ebook mode actually.

 Humitos, can you prepare us a patch based on the above that handles all
 the cases correctly? a) new Browse instance b) resume a Browse instance
 with several tabs, the active tab should get the focus here [...]

-- 
Ticket URL: <http://bugs.sugarlabs.org/ticket/4200#comment:10>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system


More information about the Bugs mailing list