Skip to content

Commit

Permalink
fix: pr feedbacks pt.3
Browse files Browse the repository at this point in the history
  • Loading branch information
TomMenga committed Dec 3, 2024
1 parent 714be9c commit b7ed26c
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 19 deletions.
7 changes: 7 additions & 0 deletions src/elements/core/styles/core.scss
Original file line number Diff line number Diff line change
Expand Up @@ -158,6 +158,13 @@ input[data-sbb-time-input] {
// Apply the brightness effect on mouse hover
// TODO: Move back to the teaser components when the global css refactoring happens
:is(sbb-teaser, sbb-teaser-hero, sbb-teaser-product) {
--sbb-teaser-image-brightness-hover: var(--sbb-hover-image-brightness);
--sbb-teaser-image-animation-duration: var(
--sbb-disable-animation-zero-duration,
var(--sbb-animation-duration-4x)
);
--sbb-teaser-image-animation-easing: var(--sbb-animation-easing);

& [slot='image']:is(sbb-image, img),
& [slot='image'] :is(sbb-image, img) {
will-change: filter;
Expand Down
7 changes: 0 additions & 7 deletions src/elements/teaser-hero/teaser-hero.scss
Original file line number Diff line number Diff line change
Expand Up @@ -10,13 +10,6 @@
// which appears in normalize css of several frameworks.
outline: none !important;

--sbb-teaser-image-brightness-hover: var(--sbb-hover-image-brightness);
--sbb-teaser-image-animation-duration: var(
--sbb-disable-animation-zero-duration,
var(--sbb-animation-duration-4x)
);
--sbb-teaser-image-animation-easing: var(--sbb-animation-easing);

@include sbb.panel-variables;
}

Expand Down
Original file line number Diff line number Diff line change
@@ -1,12 +1,6 @@
@use '../../core/styles' as sbb;

:host {
--sbb-teaser-image-brightness-hover: var(--sbb-hover-image-brightness);
--sbb-teaser-image-animation-duration: var(
--sbb-disable-animation-zero-duration,
var(--sbb-animation-duration-4x)
);
--sbb-teaser-image-animation-easing: var(--sbb-animation-easing);
--sbb-teaser-product-border-radius: var(--sbb-border-radius-4x);

// Simulate link color optically
Expand Down
6 changes: 0 additions & 6 deletions src/elements/teaser/teaser.scss
Original file line number Diff line number Diff line change
Expand Up @@ -17,12 +17,6 @@
--sbb-teaser-gap: var(--sbb-spacing-fixed-4x);
--sbb-teaser-width: fit-content;
--sbb-teaser-border-radius: var(--sbb-border-radius-4x);
--sbb-teaser-image-brightness-hover: var(--sbb-hover-image-brightness);
--sbb-teaser-image-animation-duration: var(
--sbb-disable-animation-zero-duration,
var(--sbb-animation-duration-4x)
);
--sbb-teaser-image-animation-easing: var(--sbb-animation-easing);

@include sbb.if-forced-colors {
--sbb-teaser-description-color: LinkText;
Expand Down

0 comments on commit b7ed26c

Please sign in to comment.