Unity

Merge lp://staging/~azzar1/unity/fix-1053014 into lp://staging/unity

  1. fix-1053014
  2. Merge into trunk
Proposed by Andrea Azzarone
Status: Merged
Approved by: Brandon Schaefer
Approved revision: no longer in the source branch.
Merge reported by: Neil J. Patel
Merged at revision: not available
Proposed branch: lp://staging/~azzar1/unity/fix-1053014
Merge into: lp://staging/unity
Diff against target: 48 lines (+21/-2)
2 files modified
dash/previews/ActionButton.cpp (+2/-2)
manual-tests/Preview.txt (+19/-0)
To merge this branch: bzr merge lp://staging/~azzar1/unity/fix-1053014
Reviewer Review Type Date Requested Status
Brandon Schaefer (community) Approve
Review via email: mp+125292@code.staging.launchpad.net

Commit message

Implement "focus on hover" behavior for preview action buttons.

Description of the change

== Problem ==
It is possible to have two highlighted preview actions buttons.

== Fix ==
Use "focus on hover" behavior.

== Test ==
Manual test added.

To post a comment you must log in.
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

Confirmed works, and code looks good.

review: Approve

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.