Merge pull request #133 from NIAEFEUP/fix/table-styling

Fix table styling in classes page
NIAEFEUP · Apr 11, 2024 · ac0d538 · ac0d538
2 parents 0055ca1 + 93f9ef4
commit ac0d538
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 14 deletions.
diff --git a/content-scripts/src/modules/initialize.js b/content-scripts/src/modules/initialize.js
@@ -272,7 +272,7 @@ export const addSortTableActions = () => {
         return firstClass.startsWith("i") || firstClass.startsWith("p") || firstClass.startsWith("d");
       })  
 
-      console.log("rows", rowsToSort)
+      // console.log("rows", rowsToSort)
 
       index += aditionalColspan-1;
 

diff --git a/content-scripts/src/pages/class_page.js b/content-scripts/src/pages/class_page.js
@@ -10,10 +10,10 @@ const removeExtras = () => {
 const titleClick = (table, title) => {
     if (title.dataset.expand == "true") {
         title.dataset.expand = "false";
-        table.style.gridTemplateRows = "0fr";
+        table.style.height = "0px";
     } else {
         title.dataset.expand = "true";
-        table.style.gridTemplateRows = "1fr";
+        table.style.height = "100%";
     }
 }
 

diff --git a/css/classPage.css b/css/classPage.css
@@ -3,15 +3,8 @@
 }
 
 .tableWrapper {
-    overflow: hidden;
-    display: grid;
-    transition: grid-template-rows 0.5s ease-in-out;
-    grid-template-rows: 1fr;
-}
-
-.tableWrapper > table {
-    display: grid;
-    overflow: hidden;
+    overflow-y: hidden;
+    overflow-x: scroll;
 }
 
 .classWrapper {
@@ -74,13 +67,17 @@ dialog[id^="photosDialog"]::backdrop {
 }
 
 .titleWrapper[data-expand="true"] .rightChevron {
-    transform: rotate(0deg);
+    transform: rotate(-180deg);
 }
 
 .titleWrapper[data-expand="false"] .rightChevron {
-    transform: rotate(-180deg);
+    transform: rotate(0deg);
 }
 
 .tabela tr.k, .tabela td.k {
     background-image: none;
+}
+
+.tabela tr:hover  {
+    border: none !important;
 }