fix: unable to change button color on table widget (#8521)

* fix: unable to change button color on table widget * Adds test for Table button color validation * removes commented out code
appsmithorg · Oct 14, 2021 · 0faf333 · 0faf333
1 parent de9a056
commit 0faf333
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/...ntegration/Smoke_TestSuite/ClientSideTests/DisplayWidgets/Table_Widget_Add_button_spec.js b/...ntegration/Smoke_TestSuite/ClientSideTests/DisplayWidgets/Table_Widget_Add_button_spec.js
@@ -45,6 +45,23 @@ describe("Table Widget property pane feature validation", function() {
         expect(someText).to.equal("Successful tobias.funke@reqres.in");
       });
   });
+
+  it("Table Button color validation", function() {
+    cy.openPropertyPane("tablewidget");
+    // Open column details of "id".
+    cy.editColumn("id");
+    // Changing column data type to "Button"
+    cy.changeColumnType("Button");
+    const color = "rgb(255, 0, 0)";
+    cy.get(widgetsPage.buttonColor)
+      .click({ force: true })
+      .clear()
+      .type(color);
+    // Close Property pane
+    cy.get(commonlocators.editPropCrossButton).click({ force: true });
+    cy.get(widgetsPage.tableBtn).should("have.css", "background-color", color);
+  });
+
   it("Table widget triggeredRow property should be accessible", function() {
     cy.get(commonlocators.TextInside).should("have.text", "Tobias Funke");
   });

diff --git a/app/client/src/widgets/TableWidget/widget/getTableColumns.tsx b/app/client/src/widgets/TableWidget/widget/getTableColumns.tsx
@@ -47,6 +47,7 @@ export const getCellProperties = (
         columnProperties.cellBackground,
         rowIndex,
       ),
+      buttonColor: getPropertyValue(columnProperties.buttonColor, rowIndex),
       buttonLabelColor: getPropertyValue(
         columnProperties.buttonLabelColor,
         rowIndex,