sugarlabs / browse-activity

Sugar activity to browse the internet; WebKit on GTK on Sugar Toolkit
GNU General Public License v2.0
8 stars 44 forks source link

Paste of URL with a leading space does search Google #78

Closed quozl closed 6 years ago

quozl commented 6 years ago

Expected result: same site is visited.

Observed result: Google Search is given the URL

rhl-bthr commented 6 years ago

Unable to reproduce on Sugar 0.112, Ubuntu 16.04 Method used for testing:

Observed Result: the same site is visited

While trying to reproduce this, a strange issue observed was:

Expected Result:

Observed Result

quozl commented 6 years ago

I've no explanation for the observed difference in behaviour.

I've tested again, Browse 201.4 or d9eecc3e3cddbb08eecff7e0839bd43bc8bfd647, Sugar 0.112 on Ubuntu 16.04, in KVM;

screenshot_ubuntu-xenial_2018-03-26_11 56 58

Can you confirm you have latest version of WebKit2? dpkg-query -W | grep webkit gives me gir1.2-webkit2-4.0 version 2.18.6-0ubuntu0.16.04.1

On the additional problem you noted, I do observe that. Astounding.

@geekrypter, you added the search feature to www.sugarlabs.org, how did you break Browse in this way?

rhl-bthr commented 6 years ago

Able to reproduce. Sorry. my mistake was that I was adding a trailing space.

rhl-bthr commented 6 years ago

Fixed in 230a278