Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐞 The visually impaired are doomed #41

Closed
wants to merge 1 commit into from

Conversation

szepeviktor
Copy link
Contributor

@szepeviktor szepeviktor commented May 1, 2024

@adamlaki kidebugoltam, ha a transform-ot kikapcsolom, megjelenik a tartalom Firefox v115-ben is

Mit gondolsz?

Copy link

netlify bot commented May 1, 2024

‼️ Deploy request for sprucecss rejected.

Name Link
🔨 Latest commit d572f03

@szepeviktor
Copy link
Contributor Author

valami van az animációval

image

@szepeviktor
Copy link
Contributor Author

szepeviktor commented May 1, 2024

így van megadva az animáció

animation: slidein config('duration', $transition) forwards;

ezt a sorrendet nem találom: https://developer.mozilla.org/en-US/docs/Web/CSS/animation

@szepeviktor
Copy link
Contributor Author

szepeviktor commented May 1, 2024

Annyira engedékeny a szabvány, hogy már zavaros.
https://developer.mozilla.org/en-US/docs/Web/CSS/animation#formal_syntax
Mi lenne, ha explicit megadnánk a 3 CSS property nevét??

animation-name: slidein;
animation-duration: config('duration', $transition);
animation-fill-mode: forwards;

@szepeviktor
Copy link
Contributor Author

szepeviktor commented May 1, 2024

Vagy a prefers-reduced-motion-t nem támogatja a böngészőm??

@media(prefers-reduced-motion: no-preference) {
  :root{
    --spruce-duration: 0.15s;
    --spruce-timing-function: ease-in-out;
  }
}

A gépemen minden is ki van kapcsolva/le van tiltva!

image

FALSE ❗

Erre varrjál gombot!

image

@szepeviktor szepeviktor changed the title fix cookie-consent visibility 🐞 The visually impaired are doomed May 1, 2024
@szepeviktor szepeviktor marked this pull request as draft May 1, 2024 16:19
@adamlaki
Copy link
Member

adamlaki commented May 1, 2024

Viktor, nem semmi CSS debugban vagy!

@adamlaki
Copy link
Member

adamlaki commented May 1, 2024

@szepeviktor raktam ki egy új verziót, nézd meg kérlek!

Ha most okés nálad, akkor találtál egy bugot.

image

@szepeviktor
Copy link
Contributor Author

Fixed in 478527c

@szepeviktor szepeviktor closed this May 1, 2024
@szepeviktor szepeviktor deleted the patch-1 branch May 1, 2024 18:00
@szepeviktor
Copy link
Contributor Author

@adamlaki Csak 1 baj van: az én reduced beállításommal is lejátssza az animációt.

@szepeviktor
Copy link
Contributor Author

... pont azért állítottam be a Windows 10-ben, hogy minden animáció stop,
hogy ne animáljon semmit

@szepeviktor
Copy link
Contributor Author

szepeviktor commented May 1, 2024

Lehetne úgy, hogy a "reduced-osoknak" ne animáljon?

@adamlaki
Copy link
Member

adamlaki commented May 1, 2024

Szuper!

Igen, mert ez csak egy átmeneti fix. A jó fix majd a spruceba kell menjen közvetlenül, felírtam. Át kell variálni a @media-queryt ami nézi a user beállítást, ide kell egy fallback, különben nem látja a változót.

Szép találat Viktor, köszi!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants