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

Fix Table Cells rule based styling by applying the first valid rule f… #1036

Merged
merged 2 commits into from
Feb 18, 2025

Conversation

bastienhubert
Copy link
Collaborator

@bastienhubert bastienhubert commented Feb 11, 2025

  • Applying the first valid rule for a table cell, according to the rule definition order

  • Fixing global evaluateSingleRuleOnDict() function behavior, in order to avoid always returning the first evaluation result

@bastienhubert bastienhubert added bug Something isn't working 2.4.10 For this release labels Feb 11, 2025
@bastienhubert bastienhubert linked an issue Feb 11, 2025 that may be closed by this pull request
@bastienhubert bastienhubert force-pushed the fix/995-cells-rule-based-styling branch 2 times, most recently from afdab11 to c399571 Compare February 11, 2025 16:07
@bastienhubert bastienhubert force-pushed the fix/995-cells-rule-based-styling branch from c399571 to ff4f12f Compare February 11, 2025 16:23
@bastienhubert bastienhubert added the charts To use when speaking about visualizations (EX: Line chart, Bar chart, etc...) label Feb 14, 2025
@alfredorubin96 alfredorubin96 merged commit ff4f12f into develop Feb 18, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.4.10 For this release bug Something isn't working charts To use when speaking about visualizations (EX: Line chart, Bar chart, etc...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rule-based styling for table cells is broken in 2.4.9
2 participants