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

Support customizing the color of the flamegraph #756

Closed
connor4312 opened this issue Sep 14, 2020 · 6 comments
Closed

Support customizing the color of the flamegraph #756

connor4312 opened this issue Sep 14, 2020 · 6 comments
Assignees
Labels
feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded

Comments

@connor4312
Copy link
Member

We got some feedback that users wanted to better match the flamegraph coloring to their theme. I tried some scheme to automatically pick an existing color from the theme, but didn't have good results. I think in part because we don't have a singular "flame"-like color, for example I tried using the border color of input validation errors, but on some themes this didn't work very well.

Kai suggested adding a new color for this that themes should customize. I'm not sure what it should be named, maybe just a "vibrant" color?

@connor4312 connor4312 added the feature-request Request for new features or functionality label Sep 14, 2020
@connor4312 connor4312 added this to the September 2020 milestone Sep 14, 2020
@miguelsolorio
Copy link
Contributor

Since we are introducing webviews in custom views, I'd imagine we'll be getting a lot more data visualization extensions so having a set of chart accent colors would work in both scenarios. Thinking of something like chart.accentOne or chart.accentBlue like the terminal colors?

@connor4312
Copy link
Member Author

Yea, that'd be good -- I can use that for the sidebar visualizer, where I'll have multiple lots, as well #755

@RandomFractals
Copy link

RandomFractals commented Sep 20, 2020

not sure what you are proposing here: that those settings become standard vscode theme styles?

Probably would not cover or be useful for a vast number of data viz extensions. maybe some simple use cases only.

There is much more than a primary color for a graph that goes into it. I don't know if chart prefix is a good idea either.

Do try some Vega viewer examples & see that if you want to go there, you probably want to add more than just chart.accentOne. What about ticks, grid lines, etc?

@RandomFractals
Copy link

you probably want to look at this repo for a comprehensive list of styles you'll need to make charts themeable: https://github.com/vega/vega-themes

@miguelsolorio
Copy link
Contributor

@RandomFractals thanks for the heads up about how themes work in Vega. Our goal here is to start to create a baseline of chart colors that extensions can reference in webviews. We can't possibly create a token for every color that exists but the thought is we can start with primary colors and then if needed we can shift the hue of these colors if we need additional colors. Thanks to your feedback, we're also considering foreground and grid lines as well.

@connor4312
Copy link
Member Author

connor4312 commented Sep 24, 2020

@connor4312 connor4312 added the verification-needed Verification of issue is requested label Sep 28, 2020
@roblourens roblourens added the verified Verification succeeded label Sep 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants