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

bug: Support when border-spacing returns a single value #606

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cyril-sf
Copy link

@cyril-sf cyril-sf commented Jan 10, 2025

This is mostly a workaround for a potential bug introduced with Firefox 134

Based on the spec, getComputedStyle(el).borderSpacing should always return 2 values, even when they are identical.

On Firefox 134, it returns a single value when they are the same.

Fixes #607

Copy link

netlify bot commented Jan 10, 2025

Deploy Preview for ember-sortable canceled.

Name Link
🔨 Latest commit 96f0116
🔍 Latest deploy log https://app.netlify.com/sites/ember-sortable/deploys/6781a878722f2a0008ee87da

NullVoxPopuli
NullVoxPopuli previously approved these changes Jan 10, 2025
@NullVoxPopuli NullVoxPopuli dismissed their stale review January 10, 2025 22:13

ope, lint issue

This is mostly a workaround for a [potential bug introduced with Firefox 134](https://bugzilla.mozilla.org/show_bug.cgi?id=1941066)

Based on the spec, `getComputedStyle(el).borderSpacing` should always return 2 values, even when they are identical.

On Firefox 134, it returns a single value when they are the same.
@cyril-sf cyril-sf force-pushed the bug/border-spacing-firefox branch from c4e04e9 to 96f0116 Compare January 10, 2025 23:08
@cyril-sf
Copy link
Author

@NullVoxPopuli lint fixed!

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.

Vertical sort is broken on Firefox 134
2 participants