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

Foundation tooltips broken on hover #145

Open
khawkins98 opened this issue Feb 14, 2019 · 5 comments
Open

Foundation tooltips broken on hover #145

khawkins98 opened this issue Feb 14, 2019 · 5 comments
Labels

Comments

@khawkins98
Copy link
Contributor

As discovered by @esanzgar , Foundation's tooltips are busted in v1.3 when using the minified CSS.

This appears to be a byproduct of how csso compresses the CSS.

image

@khawkins98
Copy link
Contributor Author

I've tried

-    "csso": "^2.1.1",
+    "csso": "^3.5.1",

But it doesn't seem to improve, investigating further.

@khawkins98 khawkins98 added the bug label Feb 14, 2019
@khawkins98
Copy link
Contributor Author

khawkins98 commented Feb 14, 2019

Looks like if we use the --no-restructure option it prevents the optimisation that results in the bug.

Compression diff:

  • Before: 114 575 bytes (over the wire compressed: 19.4KB)
  • After: 119 286 bytes (over the wire compressed: 19.5KB)

5KB is probably a reasonable difference. I'll commit and we can debate about releasing to prod/test further.

https://dev.ebi.emblstatic.net/web_guidelines/EBI-Framework/v1.3/css/ebi-global.min.css

@esanzgar
Copy link
Contributor

esanzgar commented Feb 21, 2019

I was about to file an issue with csso, but found many related issues with the structural optimization. I am surprised we haven't found issues earlier.

Anyway, this is our issue with the reordering:

Before optimisation
After optimisation

@khawkins98
Copy link
Contributor Author

Oof, well good that we've turned if off. Surprised it defaults to on.

@esanzgar
Copy link
Contributor

esanzgar commented Feb 21, 2019

Oof, well good that we've turned if off. Surprised it defaults to on.

Well, if no body else seem to notice... then maybe we should keep it on. It is a nice compression.

(My guess is that people notice but don't raise any issue, and they apply local patches.)

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

No branches or pull requests

2 participants