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

refactor(flamegraph)!: Color function now takes a framebox #55

Merged
merged 6 commits into from
Apr 14, 2022

Conversation

bric3
Copy link
Owner

@bric3 bric3 commented Apr 13, 2022

In order to be more flexible in regard of the frame coloring, this PR change the signature of the function callback to get the color from a frame.

- Function<T, Color> frameColorFunction
+ Function<FrameBox<T>, Color> frameColorFunction

@github-actions
Copy link

❌ Java CI with Gradle failed: https://gradle.com/s/7cdnjvd5gxei2

BREAKING CHANGE: Signature change on the color function
@bric3 bric3 force-pushed the break-color-mapping branch from 294ba8d to 69ac60c Compare April 13, 2022 13:54
bric3 added 4 commits April 13, 2022 17:45
This component is embedded in a JLayer, however it's LayerUI doesn't paint background.
In order to enable consumer to tweak the background, this change introduce a dumb JPanel
wrapper that will forward the color to contributing component.
@github-actions
Copy link

❌ Java CI with Gradle failed: https://gradle.com/s/janyo43kpgmma

Introduce another method, and keep the old one.
@bric3 bric3 force-pushed the break-color-mapping branch from 446391f to 8b06b01 Compare April 14, 2022 19:11
@bric3 bric3 merged commit 7f8aa07 into master Apr 14, 2022
@bric3 bric3 deleted the break-color-mapping branch April 14, 2022 19:13
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.

1 participant