Suggested fix for issue #158 #213

Closed
wants to merge 20 commits into
from

3 participants

@IQAndreas

The FlxObject following a path will continue to move even if they run
out of path. ( see Issue #158 )
#158

This fix adds a "StopWhenFinished" option to "FlxObject::followPath",
as well as some minor changes to avoid repetitious code.

and others added some commits Aug 20, 2011
@AdamAtomic added optional color map parameter to FlxTilemap.bitmapToCSV() 338f640
@AdamAtomic made flxg.log() able to split out arrays using flxu.formatarray 4ac1879
@AdamAtomic added readme file finally 85a99cd
@AdamAtomic minor readme edits 1d398b3
@AdamAtomic health was only initialized in sprite, should have been in object 289486b
@AdamAtomic added a function to trigger basic full-screen support 2c1e5dc
Gary Lake Fixed issue #193: FlxPreloader can now handle URLs with http or https 11c62d1
@IQAndreas IQAndreas Suggested fix for Issue #206
Using the following function names results in Migration Issue warning
from Flash Professional:
onKeyUp, onKeyPress, onKeyDown, onMouseDown, onMouseUp, onMouseMove
AdamAtomic#206

Replaced all occurances of those names with:
handleKeyUp, handleKeyPress, etc.
(I hope I got them all!)
2aeaa3e
Axcho (Axcho) Fix for issue #214
FlxU.formatArray() would include the first element twice:
AdamAtomic#214
6d09ea8
@IQAndreas IQAndreas Added `super.destroy()` to all overridden `destroy()` methods
Fixes GitHub issue #171
AdamAtomic#171

In some places, `super.destroy()` was not required. For instance, some
classes extended `FlxBasic` which does not have any content in the
`destroy()` method.

However, just to be safe (and in case that function gets populated at a
later date), calling super was added anyway.
3b85833
@FlixelCommunityBot FlixelCommunityBot referenced this pull request in FlixelCommunity/flixel Sep 13, 2012
Closed

Suggested fix for issue #158 #19

moly and others added some commits Aug 30, 2012
@moly moly fixed #11, Incorrect value for moves in FlxText, FlxTileblock and Flx…
…Tilemap
a20aca9
@moly moly Corrected documentation, FlxSound does not have a moves property. 93d3f6b
@moly moly Fix issue #191 - Fixed FlxU.round() giving incorrect results for nega…
…tive numbers

Fixed FlxU.round() giving incorrect results for negative numbers.
AdamAtomic#191
96bfd5f
@IQAndreas IQAndreas Merge pull request #79 from IQAndreas/fix_issue_191
Fix issue #191 - Fixed FlxU.round() giving incorrect results for negative numbers
adfb381
@moly moly Merge pull request #70 from moly/fix_#220
fixed #12, AdamAtomic/flixel#220 Incorrect value for moves in FlxText, FlxTileblock and FlxTilemap
2886406
@moly moly Merge pull request #71 from IQAndreas/fix_issue_193
Fix issue #38, AdamAtomic/flixel#193 - FlxPreloader.myURL should take a proper URL
dde536b
@IQAndreas IQAndreas Merge pull request #76 from IQAndreas/fix_issue_206
Fix issue #206 - Migration issues if published by Flash IDE
96087a6
@IQAndreas IQAndreas Merge pull request #73 from IQAndreas/fix_issue_171
Fix issue #171 - Added `super.destroy()` to all overridden `destroy()` methods
574ae5b
@IQAndreas IQAndreas Merge remote-tracking branch 'IQAndreas/fix_issue_214' into dev 2be8e01
@IQAndreas IQAndreas FlxObject now stops when it reaches the end of its path
Also:
 - Avoided repetitious code (and made it clearer)
 - Fixed minor typo in ASDoc
15dad8b
@IQAndreas IQAndreas closed this Nov 17, 2013
@IQAndreas IQAndreas deleted the IQAndreas-testprojects:fix_issue_158 branch Nov 17, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment