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 737628 - Implement searches
Implement searches
Status: RESOLVED FIXED
Product: gnome-calendar
Classification: Applications
Component: General
unspecified
Other Linux
: Normal blocker
: 3.26
Assigned To: GNOME Calendar maintainers
GNOME Calendar maintainers
: 703289 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2014-09-29 23:03 UTC by Georges Basile Stavracas Neto
Modified: 2017-04-17 18:20 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
First design proposal (112.75 KB, image/png)
2014-09-29 23:04 UTC, Georges Basile Stavracas Neto
Details
Second design proposal (85.97 KB, image/png)
2014-09-29 23:05 UTC, Georges Basile Stavracas Neto
Details
Design proposal (355.60 KB, image/png)
2014-10-23 10:27 UTC, Georges Basile Stavracas Neto
Details
Search prototype (89.84 KB, image/png)
2014-10-23 20:58 UTC, Erick Perez Castellanos
Details

Description Georges Basile Stavracas Neto 2014-09-29 23:03:10 UTC
GNOME Calendar currently has only a stub interface for searches. It should be implemented, as it is a necessary feature for a complete calendar application.
Comment 1 Georges Basile Stavracas Neto 2014-09-29 23:04:30 UTC
Created attachment 287405 [details]
First design proposal

I've currently designed two trials for the search view.
This is the first one.
Comment 2 Georges Basile Stavracas Neto 2014-09-29 23:05:17 UTC
Created attachment 287406 [details]
Second design proposal

The second design trial.
Comment 3 Bastien Nocera 2014-09-30 10:08:30 UTC
I think that the date/time is probably the most important part of the search, not the event comment. This means that the date/time should feature more prominently on the left/top of the search results.

Google calendar does something like:
[Date]  [Time]    [Entry name]
                  [Extra metadata]
Comment 4 Erick Perez Castellanos 2014-09-30 13:01:14 UTC
(In reply to comment #3)
> I think that the date/time is probably the most important part of the search,
> not the event comment. This means that the date/time should feature more
> prominently on the left/top of the search results.
> 
> Google calendar does something like:
> [Date]  [Time]    [Entry name]
>                   [Extra metadata]

Agree. I like the second one better. Also you should take into consideration [1]
Also, I think maybe the rows doesn't need to take the entire windows, and a border around the result list would be better, like [2]

[1]: https://github.com/gnome-design-team/gnome-mockups/tree/master/calendar/v.1/search
[2]: https://developer.gnome.org/hig/stable/figures/patterns/list-styles.svg
Comment 5 Erick Perez Castellanos 2014-10-01 13:38:52 UTC
*** Bug 703289 has been marked as a duplicate of this bug. ***
Comment 6 Georges Basile Stavracas Neto 2014-10-20 10:51:53 UTC
We definitely need an ultimate design approved by the design team. My trials are just this: trials. We shouldn't rely on them to implement the search. I think Reda's designs are very good, and even if we don't implement them as they are, we should base our efforts on them.
Comment 7 Georges Basile Stavracas Neto 2014-10-23 10:27:12 UTC
Created attachment 289196 [details]
Design proposal

This is a much improved design. While working on it, I thought so many things:

 - For the location section, we could use libchamplain.
 - The three buttons actually mean: [ Information | Participants | Attachments ]
 - If the user wants to see detailed info about the event, how could that be achieved?
 - Search could be done not only against the event title, but against: event's source, date and/or time range, other fields (notes, participants, attachments). This is pretty advanced stuff, I think we should support only the minimum functionality first.
 - Anything else I might forgot?
Comment 8 Erick Perez Castellanos 2014-10-23 20:51:39 UTC
Let me answer this in order, it will make sense in the end I guess.

>  - For the location section, we could use libchamplain.
Yes, that's the natural choice since that's what Maps use. Now, I don't want to place too much information in the search view.

> - Search could be done not only against the event title, but against: event's source, date and/or time range, other fields (notes, participants, attachments). This is pretty advanced stuff, I think we should support only the minimum functionality first.
Yes, we should cover most of the string fields, title, notes, email of guests

> - If the user wants to see detailed info about the event, how could that be achieved?
This is the more important question I guess. I think we should only show some overview of the event here and add a button (like the one in the screenshot) for showing "more details" which will open the edit-dialog. Otherwise we will end-up with two different ways of showing information to the user. Now, this mean that the edit-dialog will have to grow all-mighty, Yes it have to.
Comment 9 Erick Perez Castellanos 2014-10-23 20:58:44 UTC
Created attachment 289237 [details]
Search prototype

This is already coded. I have some bits missing but once we agree on the UI there wont be that much left.
Comment 10 Georges Basile Stavracas Neto 2014-10-24 02:38:51 UTC
I propose some adjustments:
- Make each result one-lined (color, date, name, description, [go to day], [more details])
- Highlight the matching text section (maybe with bold + yellow background)
- Remove spacing between rows (but don't leave a 2px border between them)
- Keep a minimum left & right margin of 96px
Comment 11 Georges Basile Stavracas Neto 2014-10-24 02:40:57 UTC
Also, search shouldn't be a constant, fixed view by itself, but a dynamic view that is destroyed when the user leaves it. I liked the old titlebar behavior, where it shows a simple title.
Comment 12 Marcus Lundblad 2014-10-27 13:59:47 UTC
I guess the search interface should be exposed as a gnome-shell search provider as well.
Maybe there should be a separate ticket about that?
Comment 13 Georges Basile Stavracas Neto 2014-10-27 14:01:45 UTC
It's an awsome idea, and I didn't even think of that.
It should be filled as another bug.
Comment 14 Georges Basile Stavracas Neto 2015-01-08 22:24:42 UTC
Search is implemented. It can be tracked on 'search-improvements' branch, and soon on master.

Further features should be filled as individual bugs.

Closing this bug as it reached it's target.