Merge lp:~jeremywootten/pantheon-files/fix-1626240-breadcrumb-name-with-hash into lp:~elementary-apps/pantheon-files/trunk
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Danielle Foré | ||||
Approved revision: | 2394 | ||||
Merged at revision: | 2571 | ||||
Proposed branch: | lp:~jeremywootten/pantheon-files/fix-1626240-breadcrumb-name-with-hash | ||||
Merge into: | lp:~elementary-apps/pantheon-files/trunk | ||||
Diff against target: |
229 lines (+61/-28) 6 files modified
libcore/FileUtils.vala (+3/-1) libcore/gof-directory-async.vala (+8/-2) libwidgets/Chrome/BasicBreadcrumbsEntry.vala (+2/-1) src/Application.vala (+9/-3) src/View/Sidebar.vala (+2/-2) src/View/Window.vala (+37/-19) |
||||
To merge this branch: | bzr merge lp:~jeremywootten/pantheon-files/fix-1626240-breadcrumb-name-with-hash | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Dieter Debast (community) | Approve | ||
elementary Apps team | Pending | ||
Review via email: mp+312413@code.launchpad.net |
Commit message
enable paths containing folders starting with a '#'
Description of the change
The changes made enable paths containing folders starting with a '#' character to be handled correctly both on local and remote (smb) file systems when creating and restoring views.
Also, the breadcrumb tooltip now displays the unescaped text.
Views are now only created via the Window.
(This branch is the basis for a more extensive reworking to make this kind of bug less likely and easier to fix - see lp:~jeremywootten/pantheon-files/refactor-path-change-requests)
TO TEST: See bug report.
When launching files from the command line with an URI containing a #, it won't navigate to the correct location but has the same behaviour like a bookmark (before the fix of course).
Example: localhost/ share/# test/second/ " localhost/ share/"
pantheon-files "smb://
Will navigate to "smb://
Otherwise seems to work as intended.