Page MenuHomePhorge
Differential D332

Improve video player UI
Needs ReviewPublic
Actions

Authored by tusooa on Sat, Jul 18, 9:05 PM.

Details

Reviewers
None
Group Reviewers
O1: the Kazv Project
Summary

Add a little bit tweak to the UI, including: (1) focus ring to the play button, (2) making the play button focusable, (3) center the triangle in the circle, and (4) a "short-duration" animation for the play button.

Type: add

Test Plan

Verify you can see the above changes.

Diff Detail

Repository
rK kazv
Branch
tusooa/stacked-2
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 671
Build 1324: Invoke BuildbotBuildbot build #2161
Build 1323: arc lint + arc unit

Event Timeline

tusooa created this revision.Sat, Jul 18, 9:05 PM
Owners added a reviewer: O1: the Kazv Project.Sat, Jul 18, 9:05 PM
Herald added a reviewer: servant. · View Herald TranscriptSat, Jul 18, 9:05 PM

Please add a changelog type to the summary, using the following format:

Type: (add|remove|skip|security|fix)

Harbormaster completed remote builds in B671: Diff 1030.Sat, Jul 18, 9:33 PM
tusooa requested review of this revision.Sat, Jul 18, 9:33 PM
tusooa edited the summary of this revision. (Show Details)Sat, Jul 18, 9:41 PM
tusooa removed a reviewer: servant.

Revision Contents
Changeset List

PathSizePackages
src/
1 lineO1: the Kazv Project
contents/
ui/
event-types/
72 linesO1: the Kazv Project
media/
52 linesO1: the Kazv Project

Diff 1030

View Options

src/CMakeLists.txt

Loading...
View Options

src/contents/ui/event-types/Video.qml

Loading...
View Options

src/contents/ui/media/PlayButton.qml

Loading...