After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 783648 - Find Matches String Across Separate List Items' Content
Find Matches String Across Separate List Items' Content
Status: RESOLVED NOTGNOME
Product: epiphany
Classification: Core
Component: Interface
3.24.x (obsolete)
Other Linux
: Normal minor
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
https://en.wikipedia.org/wiki/GNOME_Web
Depends on:
Blocks:
 
 
Reported: 2017-06-11 00:32 UTC by spm2011
Modified: 2017-06-11 15:04 UTC
See Also:
GNOME target: ---
GNOME version: 3.23/3.24


Description spm2011 2017-06-11 00:32:53 UTC 
To reproduce:
https://en.wikipedia.org/wiki/GNOME_Web
Find "browsersGNOME"

Expected: No matches.

Actual: Match on text across two separate <li> elements "Free web browsers" and "GNOME Core Applications".

I'm open to discussion but IMO this case should not result in a match.

FWIW it seems this is also an issue in Chromium (https://bugs.chromium.org/p/chromium/issues/detail?id=703930) but not in FF or Safari according to the Chromium testers (so I assume not a WebKit issue).
Comment 1 spm2011 2017-06-11 03:52:47 UTC 
> FWIW it seems this is also an issue in Chromium
> (https://bugs.chromium.org/p/chromium/issues/detail?id=703930) but not in FF
> or Safari according to the Chromium testers (so I assume not a WebKit issue).

Upon testing, Safari actually has the same behaviour when searching with the Contains operator, and Midori also exhibits this behaviour.

So this is probably an upstream issue in WebKit somewhere.
Comment 2 Michael Catanzaro 2017-06-11 04:21:44 UTC 
Wow, good... find! ;)

It's a problem with WebKitFindController. I've looked briefly at the code and everything looks fine down to the level of Editor::countMatchesForText, where I stopped looking. Do you mind reporting this on WebKit Bugzilla?

Thanks for taking the time to report this.
This issue is most likely a bug in WebKit. Please report the bug to https://bugs.webkit.org/ including a link to this bug report and noting the version of WebKitGTK+ that you have installed. When reporting the WebKit bug, be sure to select the WebCore Misc component and CC bugs-noreply@webkitgtk.org.
Comment 3 spm2011 2017-06-11 13:17:31 UTC 
(In reply to Michael Catanzaro from comment #2)
> Wow, good... find! ;)
> 
> It's a problem with WebKitFindController. I've looked briefly at the code
> and everything looks fine down to the level of Editor::countMatchesForText,
> where I stopped looking. Do you mind reporting this on WebKit Bugzilla?
> 
> Thanks for taking the time to report this.
> This issue is most likely a bug in WebKit. Please report the bug to
> https://bugs.webkit.org/ including a link to this bug report and noting the
> version of WebKitGTK+ that you have installed. When reporting the WebKit
> bug, be sure to select the WebCore Misc component and CC
> bugs-noreply@webkitgtk.org.

Will do, thanks