Merge lp://staging/~jeremywootten/pantheon-files/fix-context-menu-when-admin into lp://staging/~elementary-apps/pantheon-files/trunk
Proposed by
Jeremy Wootten
| Status: | Merged |
|---|---|
| Approved by: | Danielle Foré |
| Approved revision: | 2211 |
| Merged at revision: | 2579 |
| Proposed branch: | lp://staging/~jeremywootten/pantheon-files/fix-context-menu-when-admin |
| Merge into: | lp://staging/~elementary-apps/pantheon-files/trunk |
| Diff against target: |
233 lines (+56/-39) 5 files modified
libcore/PluginManager.vala (+15/-12) libwidgets/Chrome/BreadcrumbIconList.vala (+7/-7) src/Application.vala (+1/-1) src/View/AbstractDirectoryView.vala (+26/-18) src/View/Sidebar.vala (+7/-1) |
| To merge this branch: | bzr merge lp://staging/~jeremywootten/pantheon-files/fix-context-menu-when-admin |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Adam Bieńkowski (community) | code | Approve | |
|
Review via email:
|
|||
Commit message
Do not show certain unusable options when running as admin and cures some terminal warnings
Description of the change
This branch does not show certain unusable options when running as admin and cures some terminal warnings.
TO TEST:
After installing and killing existing Files processes, run "New Window as Administrator" from plank or "pantheon-
To post a comment you must log in.
Revision history for this message
| Jeremy Wootten (jeremywootten) wrote | # |
Revision history for this message
| Adam Bieńkowski (donadigo) wrote | # |
review:
Needs Fixing
(code / testing)
Revision history for this message
| Jeremy Wootten (jeremywootten) wrote | # |
Revision history for this message
| Jeremy Wootten (jeremywootten) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Whether or not the program is run with pkexec or (gk)sudo, the network options in the sidebar do not work in trunk and are suppressed in this branch.