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

add: add color field in data bar cf in typescript #2655

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

krhambaliyaavesta
Copy link

Summary

While working with the Databar feature in Conditional formating (cf), I encountered an issue related to the color property in TypeScript typings.

Purpose of Change

I identified a deficiency in the existing TypeScript typings for the Databar feature in conditional formatting. The current Databar interface lacks a 'color' type, which is essential for specifying colors within the Databar. This gap in the typings prevents developers from accurately representing and utilizing color settings in Databar conditions.

Changes Made

I have made a targeted change to the TypeScript type file associated with the Databar interface. Specifically, I introduced the 'color' type, which aligns with the functionality of assigning colors in Databar conditions.

Details

  • Issue Faced: The absence of the 'color' type in the Databar interface impairs the ability to set colors for Databar conditions in conditional formatting.

  • Solution Proposed: Addition of the 'color' type to the Databar interface in the TypeScript type file.

Testing

I have conducted thorough testing to ensure that the introduced 'color' type integrates seamlessly with the Databar feature and does not introduce any regressions.

Related to Source Code (for typings update)

I made a change in the TypeScript type file, specifically in the Databar interface, to address the color property's absence, enabling developers to utilize color settings in Databar conditions effectively.

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