-
Notifications
You must be signed in to change notification settings - Fork 30
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
@@ -0,0 +1,55 @@ | |||
.rc-table.ads-v2-table-wrapper { |
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.
Why not keep all styles in one place? You already have classes for all these overrides. I think it's better to move.
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.
So the reason for doing this here is, RC adds some divs on the fly and the classes we add is not applied on these. That's why I had to keep a seperate CSS file for this.
@albinAppsmith
|
Description
This PR adds table component to ADS v2. Table component will be using ec-table internally.
Fixes appsmithorg/appsmith#27771
Type of change
How Has This Been Tested?
Test Plan
Issues raised during DP testing
Checklist:
Dev activity
QA activity: