Merge lp://staging/~jeremywootten/pantheon-files/fix-1577399-cannot-search-trash into lp://staging/~elementary-apps/pantheon-files/trunk

Proposed by Jeremy Wootten
Status: Merged
Approved by: Danielle Foré
Approved revision: 2140
Merged at revision: 2148
Proposed branch: lp://staging/~jeremywootten/pantheon-files/fix-1577399-cannot-search-trash
Merge into: lp://staging/~elementary-apps/pantheon-files/trunk
Diff against target: 190 lines (+35/-17)
5 files modified
libcore/FileUtils.vala (+13/-6)
libcore/marlin-icons.h (+2/-2)
libwidgets/View/LocationBar.vala (+5/-1)
libwidgets/View/SearchResults.vala (+14/-7)
src/View/Sidebar.vala (+1/-1)
To merge this branch: bzr merge lp://staging/~jeremywootten/pantheon-files/fix-1577399-cannot-search-trash
Reviewer Review Type Date Requested Status
elementary Apps team Pending
Review via email: mp+293919@code.staging.launchpad.net

Commit message

Enable search in trash://, recent:// and network:// views

Description of the change

This branch enables search to work in trash:// recent:// and network:// by matching on FileAttribute.STANDARD_DISPLAY_NAME instead of FileAttribute.STANDARD_NAME.

It also fixes manual navigation to these locations by correcting the constant URL strings for the root locations, making them consistent with other similar constant URL strings. Another side-effect of this is to fix the bug where these views cannot be restored.

To post a comment you must log in.
Revision history for this message
Zisu Andrei (matzipan) wrote :

Nice one! I was going to file this bug myself :P

2139. By Jeremy Wootten

Fix regression searching root filesystem

Revision history for this message
Jeremy Wootten (jeremywootten) wrote :

Fixed regression in searching root filesystem caused by r2130. An empty scheme string is equivalent to the root filesystem and should be browsable/searchable.

Revision history for this message
Danielle Foré (danrabbit) wrote :

I can confirm that results now show in the dropdown, but selecting only works for toplevel items. Pressing enter on one of these results does not show it if it's in a subfolder.

2140. By Jeremy Wootten

Fix activating certain search results

Revision history for this message
Jeremy Wootten (jeremywootten) wrote :

Fixed bug in sanitizing trash uris causing failure to open results in trashed subfolders.

Also fixed launching results of searching in network:// and other cases where search result is a link - the target location is now activated.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: