Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

FlxSound pitch is missing #264

Open
T1mL3arn opened this issue Feb 18, 2023 · 3 comments
Open

FlxSound pitch is missing #264

T1mL3arn opened this issue Feb 18, 2023 · 3 comments

Comments

@T1mL3arn
Copy link
Contributor

  1. go to https://api.haxeflixel.com/flixel/system/FlxSound.html
  2. try to find pitch
  3. see no results

Pitch prop was added with HaxeFlixel/flixel#2564 .
@Geokureli
Copy link
Member

I believe there's a specific compiler flag we should use to expose fields to the api page, I forgot what it is so we'll have to find it

@T1mL3arn
Copy link
Contributor Author

T1mL3arn commented Mar 9, 2023

The flag is FLX_PITCH and it is set by flixel on targets except flash. But the doc is built for flash target, so the pitch property is not generated.

Possible solutions:

  • build doc for non flash target, e.g. neko
  • wrap pitch prop in #if doc_gen

@ninjamuffin99
Copy link
Member

after some investigation this sorta thing seems to affect other "non-flash target" related defines
FlxG.android, FlxG.accelerometer. Neither have documentation generated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Geokureli @T1mL3arn @ninjamuffin99