Merge lp://staging/~jeremywootten/pantheon-files/fix-debian-build into lp://staging/~elementary-apps/pantheon-files/trunk

Proposed by Jeremy Wootten
Status: Merged
Merged at revision: 2257
Proposed branch: lp://staging/~jeremywootten/pantheon-files/fix-debian-build
Merge into: lp://staging/~elementary-apps/pantheon-files/trunk
Diff against target: 207 lines (+58/-28)
4 files modified
libwidgets/Chrome/BasicBreadcrumbsEntry.vala (+3/-1)
libwidgets/Chrome/BreadcrumbElement.vala (+6/-3)
src/CMakeLists.txt (+42/-19)
src/View/Sidebar.vala (+7/-5)
To merge this branch: bzr merge lp://staging/~jeremywootten/pantheon-files/fix-debian-build
Reviewer Review Type Date Requested Status
elementary Apps team Pending
Review via email: mp+300616@code.staging.launchpad.net

Description of the change

This branch fixes some mistakes in ./src/CMakeLists.txt that prevented the package compiling if Unity and Plank were absent (e.g. under Debian Sid).

It also fixes an issue with the drawing of breadcrumbs under different themes causing the outline to be drawn inaccurately.

To post a comment you must log in.
Revision history for this message
Cody Garver (codygarver) wrote :

Looks good but I made some diff comments about indention.

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: