Merge pull request #436 from dcos-labs/brandonc/sort_arrow_direction

fix: arrow direction / asc & desc
d2iq · Nov 22, 2019 · bf5fbb3 · bf5fbb3
2 parents 477eacf + 26d923b
commit bf5fbb3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/packages/table/style.ts b/packages/table/style.ts
@@ -139,13 +139,13 @@ export const styleArrowDirection = displaySortDirection => {
     case "DESC":
       return css`
         &:after {
-          ${pseudoElTriangle("bottom", pointerSize, "currentColor")};
+          ${pseudoElTriangle("top", pointerSize, "currentColor")};
         }
       `;
     case "ASC":
       return css`
         &:after {
-          ${pseudoElTriangle("top", pointerSize, "currentColor")};
+          ${pseudoElTriangle("bottom", pointerSize, "currentColor")};
         }
       `;
     default:

diff --git a/packages/table/tests/__snapshots__/SortableHeaderCell.test.tsx.snap b/packages/table/tests/__snapshots__/SortableHeaderCell.test.tsx.snap
@@ -23,10 +23,10 @@ exports[`SortableHeaderCell renders default 1`] = `
   content: "";
   height: 0;
   width: 0;
-  border-bottom: 4px solid currentColor;
+  border-top: 4px solid currentColor;
   border-left: 4px solid transparent;
   border-right: 4px solid transparent;
-  border-top: none;
+  border-bottom: none;
 }
 
 <Clickable