Merge lp:~widelands-dev/widelands/bug1480928 into lp:widelands
Proposed by
Jens Beyer
Status: | Rejected | ||||
---|---|---|---|---|---|
Rejected by: | Jens Beyer | ||||
Proposed branch: | lp:~widelands-dev/widelands/bug1480928 | ||||
Merge into: | lp:widelands | ||||
Diff against target: |
64 lines (+5/-5) 5 files modified
tribes/atlanteans/woodcutter/conf (+1/-1) tribes/barbarians/lumberjack/conf (+1/-1) tribes/empire/lumberjack/conf (+1/-1) world/immovables/trees/aspen/init.lua (+1/-1) world/immovables/trees/oak/init.lua (+1/-1) |
||||
To merge this branch: | bzr merge lp:~widelands-dev/widelands/bug1480928 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jens Beyer | Needs Resubmitting | ||
Review via email: mp+270540@code.launchpad.net |
Description of the change
Animation seems to loop to the first picture again.
Easy fix - remove 1ms of the animation ;-)
I believe it would not be good to remove the looping from animation in general. Other way would probably be tinkering with the fps, but I don't know if it takes fraction values like 10.001 or something.
To post a comment you must log in.
I just tested it, and the oak is still getting up. Maybe we do need a switch in the animation after all to make sure that this is always clean (loop = true/false). I think we will need a second version of NumberGlob in graphics/ animation. cc for this.