-
Notifications
You must be signed in to change notification settings - Fork 39
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
186182704 formula column background #1529
Conversation
Disables value editing when attribute has a formula
Splits case card test to 2 context so mouseSensor does not interfere with attribute menu clicks
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1529 +/- ##
=======================================
Coverage 85.10% 85.10%
=======================================
Files 563 563
Lines 28149 28155 +6
Branches 7728 7731 +3
=======================================
+ Hits 23955 23961 +6
Misses 3888 3888
Partials 306 306
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
codap-v3 Run #4517
Run Properties:
|
Project |
codap-v3
|
Branch Review |
main
|
Run status |
Passed #4517
|
Run duration | 08m 46s |
Commit |
e9dfa69634: 186182704 formula column background (#1529)
|
Committer | Evangeline Ireland |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
30
|
Skipped |
0
|
Passing |
227
|
View all changes introduced in this branch ↗︎ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good -- one suggestion to use clsx
instead of concatenating class strings ourselves.
* Adds formula column background color * Adds formula background styling to case card value cell. Disables value editing when attribute has a formula * Adds Cypress test to verify background color when attribute is a formula Splits case card test to 2 context so mouseSensor does not interfere with attribute menu clicks * PR fix
Adds background color to columns with formula.
Adds background color to value cell in case card if attribute has a formula
Disables cell editing in case card when attribute has a formula.
Adds cypress test to verify background color for cells with formula values.
Splits case-card cypress test so that two different URLs are used to load CODAP. The
mouseSensor
query param was interfering with test that required the attribute menu.