Merge lp://qastaging/~njpatel/unity/lens-ordering into lp://qastaging/unity
Proposed by
Neil J. Patel
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Mirco Müller | ||||
| Approved revision: | no longer in the source branch. | ||||
| Merged at revision: | 1516 | ||||
| Proposed branch: | lp://qastaging/~njpatel/unity/lens-ordering | ||||
| Merge into: | lp://qastaging/unity | ||||
| Diff against target: |
40 lines (+21/-1) 1 file modified
UnityCore/FilesystemLenses.cpp (+21/-1) |
||||
| To merge this branch: | bzr merge lp://qastaging/~njpatel/unity/lens-ordering | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Mirco Müller (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
This makes sure that the apps lens is always first, and adds basic ordering to the lenses instead of them always coming in randomly. It means that we send of the lens_added signal a little later, but that shouldn't make a difference to consumers.
This is done in a way to preserve ABI/API of UnityCore and I've added FIXMEs for the couple of things that should be changed when we branch for P.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Gee... still got get used to all that new C++ stuff, but approved.