From 207fd5ea6624cbda900597127efcd5f7ee1464e0 Mon Sep 17 00:00:00 2001
From: Jessica Lawshe <5312836+lawshe@users.noreply.github.com>
Date: Wed, 26 Jun 2024 09:21:18 -0500
Subject: [PATCH] Merge pull request #19085 from
 overleaf/jel-light-touch-table-updates

[web] Update background color CSS for light touch redesign

GitOrigin-RevId: 574f813a9964a272742507817732c1b45bbdc2fe
---
 .../app/plans/plans-light-touch-redesign.less | 47 +------------------
 1 file changed, 1 insertion(+), 46 deletions(-)

diff --git a/services/web/frontend/stylesheets/app/plans/plans-light-touch-redesign.less b/services/web/frontend/stylesheets/app/plans/plans-light-touch-redesign.less
index c969aa2947..671ef3e4ea 100644
--- a/services/web/frontend/stylesheets/app/plans/plans-light-touch-redesign.less
+++ b/services/web/frontend/stylesheets/app/plans/plans-light-touch-redesign.less
@@ -449,27 +449,8 @@
 
   .plans-table-student {
     tr {
-      // "hide" the last column on desktop by making the background-color as the default background color
-      th:last-of-type > .plans-table-th {
-        background-color: @ol-blue-gray-0;
-      }
-
       &:last-child {
-        td:first-child {
-          background-color: @ol-blue-gray-0;
-
-          .plans-v2-table-feature-name {
-            border-bottom-left-radius: @plans-table-border-radius;
-
-            @media (max-width: @screen-xs-max) {
-              border-bottom-left-radius: 0;
-            }
-          }
-        }
-
         td:last-child {
-          background-color: @ol-blue-gray-0;
-
           .plans-table-cell {
             border-bottom-right-radius: @plans-table-border-radius;
 
@@ -515,21 +496,7 @@
   .plans-table-individual {
     tr {
       &:last-child {
-        td:first-child {
-          background-color: @ol-blue-gray-0;
-
-          .plans-v2-table-feature-name {
-            border-bottom-left-radius: @plans-table-border-radius;
-
-            @media (max-width: @screen-xs-max) {
-              border-bottom-left-radius: 0;
-            }
-          }
-        }
-
         td:last-child {
-          background-color: @ol-blue-gray-0;
-
           .plans-table-cell {
             border-bottom-right-radius: @plans-table-border-radius;
 
@@ -539,10 +506,6 @@
           }
         }
       }
-
-      &:not(.plans-v2-table-divider) td {
-        background-color: @white;
-      }
     }
 
     th {
@@ -611,7 +574,7 @@
   }
 
   .plans-table {
-    background-color: @ol-blue-gray-0;
+    background-color: white;
     margin-bottom: 15px;
     table-layout: fixed;
     width: 100%;
@@ -660,7 +623,6 @@
       vertical-align: top;
       padding: 0;
       position: relative;
-      background-color: @ol-blue-gray-0;
 
       &:first-child {
         border-left: 0;
@@ -691,8 +653,6 @@
       // css hack for table border-radius on the first feature name
       &:nth-child(2) {
         .plans-table-row-header:first-child {
-          background-color: @ol-blue-gray-0;
-
           .plans-v2-table-feature-name {
             border-top-left-radius: @plans-table-border-radius;
 
@@ -798,7 +758,6 @@
       // will have the full width of viewport,
       // and making the first td appear like the it is in its own row (although it's technically still on the same tr as the other tds)
       .plans-table-row-header {
-        background: @gray-lightest;
         width: 100vw;
 
         span.plans-v2-table-feature-name {
@@ -816,7 +775,6 @@
   // that doesn't contain divider or feature name
   // (i.e it contains either check mark icon or a dash)
   .plans-table-cell {
-    background-color: @white;
     height: 100%;
 
     .plans-table-cell-content {
@@ -836,7 +794,6 @@
 
   .plans-table-th {
     height: 100%;
-    background-color: @white;
 
     .plans-table-th-content {
       display: flex;
@@ -1006,7 +963,6 @@
   }
 
   .plans-v2-table-feature-name {
-    background-color: @white;
     display: flex;
     justify-content: space-between;
     align-items: center;
@@ -1284,7 +1240,6 @@
     margin-top: var(--spacing-09);
   }
 
-  .plans-table,
   .plans-table-column-header {
     background-color: transparent; // remove colors set by default variant
   }