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.

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

Minor adjustments to documentation and minor adjust to SpectrogramFlatView #86

Merged
merged 8 commits into from
Apr 20, 2024

Conversation

mahal
Copy link
Contributor

@mahal mahal commented Apr 10, 2024

No changes to functionality, just documentation. Created a .docc file so the manual can be built with Xcode and exported. Which I already did, see PR AudioKit/audiokit.io#5

Furthermore some minor tweak in SpectrogramFlatView. I wanted to to two PRs but this all ended up in this one. It's a bugfix so that full amplitude shows loudest instead of half-loud.

///
/// Example: `let color = [.green, .yellow, .red].intermediate(0.7)`
///
/// inspired by [stackoverflow 15032562 answer Nikaaner](https://stackoverflow.com/questions/15032562/ios-find-color-at-point-between-two-colors/59996029#59996029)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line Length Violation: Line should be 120 characters or less: currently 163 characters (line_length)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would have no clue how to break this line into two without breaking the link .

Should I add? // swiftlint:disable line_length

@mahal mahal changed the title Minor adjustments to documentation Minor adjustments to documentation and minor adjust to SpectrogramFlatView Apr 12, 2024
@aure aure merged commit 83bacdb into AudioKit:main Apr 20, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants