Merge lp:~shockone89/audience/rewinding into lp:~audience-members/audience/trunk
Proposed by
Volodymyr Shatsky
Status: | Rejected | ||||
---|---|---|---|---|---|
Rejected by: | Cody Garver | ||||
Proposed branch: | lp:~shockone89/audience/rewinding | ||||
Merge into: | lp:~audience-members/audience/trunk | ||||
Diff against target: |
79 lines (+34/-7) 3 files modified
src/Audience.vala (+15/-5) src/Consts.vala (+2/-1) src/Widgets/VideoPlayer.vala (+17/-1) |
||||
To merge this branch: | bzr merge lp:~shockone89/audience/rewinding | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Cody Garver | Disapprove | ||
Robert Roth (community) | Needs Fixing | ||
Review via email: mp+218217@code.launchpad.net |
To post a comment you must log in.
Unmerged revisions
- 337. By Volodymyr Shatsky
-
Short and long rewinding.
The changes look fine, some comments though
* with these changes, the jump intervals are hardcoded to 5 seconds and 60 seconds, instead of 5% of the total length. Is this something we want? In a 3-hour movie, 60 seconds jumps are rather short. Maybe short jump could work with jump 5-10 seconds, long jump could jump 5-10% of the total length.