Skip to content

Commit

Permalink
feat(accordion): remove hover state (#163)
Browse files Browse the repository at this point in the history
* feat(accordion): remove hover state

removed the hover state from accordion, as per designs

HEYUI-267

* update snapshots

* updated font weigth

* update snapshot
  • Loading branch information
nathanjamal committed Mar 15, 2023
1 parent 9a601a7 commit df62524
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 12 deletions.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
19 changes: 7 additions & 12 deletions packages/accordion/src/styles/default.module.css
Original file line number Diff line number Diff line change
Expand Up @@ -13,18 +13,18 @@
color: var(--color-neutral-700);

&.accordionOpen {
padding-bottom: var(--gap-5xl);
padding-bottom: var(--spacing-10);
}
}

.header {
@mixin typography-body-3;
@mixin typography-body-1;
text-align: left;
flex-grow: 1;
width: 100%;
cursor: pointer;
transition: margin 150ms cubic-bezier(0.4, 0, 0.2, 1) 0ms;
padding: var(--gap-m) 0;
padding: var(--spacing-4) 0;

-webkit-box-align: center;
-webkit-tap-highlight-color: transparent;
Expand All @@ -48,7 +48,7 @@
display: flex;

&.headerOpen {
margin-bottom: var(--gap-xs);
margin-bottom: var(--spacing-2);

& .headerArrow {
transform: rotate(180deg);
Expand All @@ -72,19 +72,14 @@
display: flex;
align-items: center;
justify-content: center;
width: var(--gap-xl);
height: var(--gap-xl);
width: var(--spacing-6);
height: var(--spacing-6);

margin-left: var(--gap-xl);
margin-left: var(--spacing-6);

transition: transform 150ms cubic-bezier(0.4, 0, 0.2, 1) 0ms;
transform: rotate(0deg);
}

&:hover,
&:focus-visible {
@mixin typography-body-1;
}
}

.body {
Expand Down

1 comment on commit df62524

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 97.7% 1188/1216
🟢 Branches 85.53% 266/311
🟢 Functions 94.79% 91/96
🟢 Lines 98.27% 1082/1101

Test suite run success

273 tests passing in 41 suites.

Report generated by 🧪jest coverage report action from df62524

Please sign in to comment.