Merge lp://qastaging/~michihenning/unity-scopes-api/doc-fix into lp://qastaging/unity-scopes-api/devel

Proposed by Michi Henning
Status: Merged
Approved by: Paweł Stołowski
Approved revision: 654
Merged at revision: 652
Proposed branch: lp://qastaging/~michihenning/unity-scopes-api/doc-fix
Merge into: lp://qastaging/unity-scopes-api/devel
Diff against target: 49 lines (+9/-5)
2 files modified
src/scopes/CategoryRenderer.cpp (+6/-2)
src/scopes/PreviewWidget.cpp (+3/-3)
To merge this branch: bzr merge lp://qastaging/~michihenning/unity-scopes-api/doc-fix
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Paweł Stołowski (community) Approve
Review via email: mp+283001@code.qastaging.launchpad.net

Commit message

Doc updates to clarify semantics of fallback images.

Description of the change

Doc updates to clarify semantics of fallback images.

To post a comment you must log in.
Revision history for this message
Albert Astals Cid (aacid) wrote :

"If a result does not specify a fallback image and the actual image is empty or cannot be loaded, Unity substitutes a generic fallback image."

We don't do that at the moment, i.e. we just show nothing, but one could argue that the "no image" is our "generic fallback image" for the moment :D

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
654. By Michi Henning

Consistency fix.

Revision history for this message
Paweł Stołowski (stolowski) wrote :

Looks good, +1

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

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: