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 697804 - Text highlight case sensitive
Text highlight case sensitive
Status: RESOLVED DUPLICATE of bug 696673
Product: evolution
Classification: Applications
Component: Mailer
3.8.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2013-04-11 14:59 UTC by Milan Crha
Modified: 2013-04-12 14:23 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Fixes case sensitive matches (499 bytes, patch)
2013-04-12 02:18 UTC, Matthew Barnes 		reviewed Details | Review

Description Milan Crha 2013-04-11 14:59:21 UTC 
When searching "Body contains" for Evolution (like in the email from bugzilla), then only "evolution" is highlighted, not the Evolution. (It seems like the search text is lower-cased, and then searched case sensitively within HTML.)
Comment 1 Matthew Barnes 2013-04-12 02:18:20 UTC 
Created attachment 241319 [details] [review]
Fixes case sensitive matches

The case sensitive substring matching is easy enough to fix, but now I find it's replacing the matched text with what I typed verbatim.

So if I search for "matthew", it's replacing "Matthew" with a purple "matthew" in the message.
Comment 2 Milan Crha 2013-04-12 07:38:28 UTC 
Review of attachment 241319 [details] [review]:

I suppose this will be obsolete with Tomas' work, related to bug #696673, we decided to drop this two-way text highlighting, and only internal webkit text highlight will be used now (thus searching for Body contains will mark the text in the same was as searching inside preview panel).
Comment 3 Matthew Barnes 2013-04-12 12:40:37 UTC 
Yes!  I'd much prefer that.  I think we're even already using WebKit's builtin text highlighting for Edit -> Find in Message.
Comment 4 Matthew Barnes 2013-04-12 12:44:52 UTC 
(In reply to comment #3)
> I think we're even already using WebKit's builtin text highlighting for
> Edit -> Find in Message.

Oh heh, you said that already.  /me needs coffee. :)
Comment 5 Milan Crha 2013-04-12 14:23:43 UTC 
Because this is covered by bug #696673, I'm closing this as a duplicate of it.

*** This bug has been marked as a duplicate of bug 696673 ***