-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
feat: Add Series Colour property for default charts #18229
feat: Add Series Colour property for default charts #18229
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Unable to find test scripts. Please add necessary tests to the PR. |
Default color set as primary theme color
Deployment failed with the following error:
|
- Some bug fixes to the series colour feature
This PR has not seen activitiy for a while. It will be closed in 7 days unless further activity is detected. |
New property placeholderText for ColorPickerComponentV2 Disable series color feature for Pie Chart Code clean up
Unable to find test scripts. Please add necessary tests to the PR. |
1 similar comment
Unable to find test scripts. Please add necessary tests to the PR. |
…5522-series-color-for-default-charts
Observation: @souma-ghosh cc: @dilippitchika
|
This is happening because Fusion chart interprets hex colours differently when the hex is less than 6 digits. This is unpredictable as far as I tested. So we can expect this to work predictably when the series colour hex is 6 characters. Also fusion chart does not support text colours like "red", "green" etc. so the option has been limited to hex code. |
Ok this is a limitation, please create a new bug for it. Also @souma-ghosh can we support JS in the field? |
@dilippitchika ATM we can't. Since we have implemented the property controls for Chart Series in a bit hacky way. So we will need a refactor there before we can make the Series Color field JS convertible. |
Understood, can you please create a new issue for it? |
Here is the issue |
/ok-to-test sha=a199af5 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3638524348. |
/ok-to-test sha=a199af5 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3646012301. |
Test Plan for Series Colour property of Chart series |
Description
In default charts, only series title and series data could be provided by developer.
We have added the Series Color property as well in this PR to control the color of all the default charts (Except Pie Chart).
Fixes #15522
Type of change
How Has This Been Tested?
Test Plan
Issues raised during DP testing
Checklist:
Dev activity
QA activity: