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 617747 - Initial implementation of "Finding and Reminding"
Initial implementation of "Finding and Reminding"
Status: RESOLVED OBSOLETE
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
: 640674 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2010-05-05 12:20 UTC by Owen Taylor
Modified: 2012-09-06 21:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Move D-Bus utilities to a separate helper file (7.92 KB, patch)
2010-05-05 12:21 UTC, Owen Taylor
reviewed Details | Review
Initial stabs at some tracker code (5.89 KB, patch)
2010-05-05 12:21 UTC, Owen Taylor
rejected Details | Review

Description Owen Taylor 2010-05-05 12:20:35 UTC
This bug covers an initial draft implementation of:

http://live.gnome.org/GnomeShell/Design/Whiteboards/FindingAndReminding

Some of the things we need to do:

 - Determine the set of items that belong in the "Desktop" based upon what's
   currently in Places and items tagged as "desktop"
 - Replace "Places" and "Recent Items" on the dash with "Desktop"
 - Add a more view that has the desktop items, a history of older items (fetched from tracker and other sources).

Filing this bug so I can attach some crude bits of code to interact with Tracker.
Comment 1 Owen Taylor 2010-05-05 12:21:11 UTC
Created attachment 160336 [details] [review]
Move D-Bus utilities to a separate helper file

To avoid duplicating code, move the D-Bus utilities from
telepathy.js to a separate file dbusUtils.js.
Comment 2 Owen Taylor 2010-05-05 12:21:13 UTC
Created attachment 160337 [details] [review]
Initial stabs at some tracker code

* Start putting some stuff in js/misc/tracker.js to interface
  with D-Bus
* Add a test case that gets some data from tracker
Comment 3 Colin Walters 2010-05-10 14:22:38 UTC
Review of attachment 160336 [details] [review]:

Can you add a comment on pathToName/nameToPath like:

// There is no default association between DBus interface names and
// object paths, but a number of services do name some of their
// object paths as the obvious transformation of an interface name.
// These functions are useful as a shorthand for those cases.
Comment 4 Colin Walters 2010-05-10 14:30:57 UTC
Review of attachment 160337 [details] [review]:

::: tests/interactive/tracker.js
@@ +22,3 @@
+/* -*- mode: js2; js2-basic-offset: 4; indent-tabs-mode: nil -*- */
+function parseDateTime(s) {
+    var match = /(-?\d{4,})-(\d\d)-(\d\d)T(\d\d):(\d\d):(\d\d)(\.\d+)?(([+-])(\d\d):(\d\d)|Z)?/.exec(s);

I'd prefer to see the regexp as a constant (and ideally with one or two sample strings it matches)
Comment 5 Giovanni Campagna 2011-01-26 20:38:34 UTC
*** Bug 640674 has been marked as a duplicate of this bug. ***
Comment 6 Jasper St. Pierre (not reading bugmail) 2011-09-28 18:07:37 UTC
Review of attachment 160337 [details] [review]:

Rejected to get it off the ACN patch list. It's not going in now.
Comment 7 Giovanni Campagna 2012-09-06 21:27:21 UTC
Finding and Reminding is in the domain of core apps now.