Bug 580925 - Better search bar for word searches
Better search bar for word searches
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.26.x (obsolete)
Other Linux
: Normal enhancement
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
:
Depends on:
Blocks:
  Show dependency tree
 
Reported: 2009-04-30 18:31 UTC by Matthew Barnes
Modified: 2009-05-20 23:18 UTC (History)
1 user (show)

See Also:
GNOME target: ---
GNOME version: ---


Attachments
Message Window Search Bar (105.05 KB, image/png)
2009-04-30 18:32 UTC, Matthew Barnes
  Details
New icon for wrapped searches (680 bytes, image/png)
2009-04-30 18:33 UTC, Matthew Barnes
  Details
Proposed patch (59.05 KB, patch)
2009-04-30 18:34 UTC, Matthew Barnes
committed Details | Diff | Review

Description Matthew Barnes 2009-04-30 18:31:59 UTC
Evolution's current UI for searching within a message sucks.

  - To the average user, Edit -> Find in Message does nothing.  They don't
    notice the cursor being placed in the search entry, which is the only
    visible feedback.

  - The preview pane in the main window and the message viewing window
    have different search interfaces.

  - The search entry is far away from the other search controls in the
    preview pane.  If you want to clear the search you have to reach all
    the way up to the top of the window.

  - The "Current Message" search scope is still shown when the preview pane
    is turned off.  Granted, it's disabled.  But it still feels awkward and
    I take that as a sign that it shouldn't be there.

I rewrote the "Find in Message" search bar to more closely resemble Firefox, and to be consistent between the preview pane and the message window.  I also broke it out of EMFormatHTMLDisplay as a standalone (possibly reusable) widget.
Comment 1 Matthew Barnes 2009-04-30 18:32:49 UTC
Created attachment 133677 [details]
Message Window Search Bar

Looks the same in the preview pane of the main window.
Comment 2 Matthew Barnes 2009-04-30 18:33:52 UTC
Created attachment 133678 [details]
New icon for wrapped searches

I stole this from Firefox.  Put it under data/icons.
Comment 3 Matthew Barnes 2009-04-30 18:34:48 UTC
Created attachment 133679 [details] [review]
Proposed patch
Comment 4 Matthew Barnes 2009-04-30 19:04:26 UTC
There's more I'd like to do with this once we start rendering mails as a vbox of HTML widgets instead of one big HTML widget with GtkHTMLEmbedded sections.

Search-as-you-type (like in Firefox) would be really cool, but it's impractical right now because we'd have to redraw the entire message with every key press.  Eventually we could just redraw the "text/*" parts, which should be fairly fast.

In addition, I'd like to handle the "no matches found" case better, which is currently broken due to some kind of bug in ESearchingTokenizer.  I'm not going to deal with that here.  Ultimately I'd like for this case to work like Firefox too, where the entry background changes color and you get a "Phrase not found" message in the bar as you type.
Comment 5 Srinivasa Ragavan 2009-05-04 03:27:02 UTC
Matt, on Embedded/VBox. I render Anjal with Webkit, and optionally with MozEmbed. Surely we can add GtkHTML to the list in a matter of few more lines. It does it by vbox. Needs good time for me to test & bugfix. 
Comment 6 Matthew Barnes 2009-05-04 12:15:58 UTC
Great!  I'm looking forward to that landing in Evolution.

Note You need to log in before you can comment on or make changes to this bug.