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

Fixed: Fix tooltips display, styles leaks, performance issues #851

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

florianMo
Copy link
Contributor

Use natives browser tooltips instead of handmade tooltips. Fixes at least one glitch (OFBIZ-12730) due to styles leaking.

Removes the global "transition: .2s" which leads to animate every single DOM element alteration (causes OFBIZ-13104, and possibly other problems/lagging). Only apply these transitions to buttons to keep a nice effect on hover. We may have to add other selectors if needed.

OFBIZ-12638
OFBIZ-12730
OFBIZ-13104

Thanks: Nereide team

Use natives browser tooltips instead of handmade tooltips.
Fixes at least one glitch (OFBIZ-12730) due to styles leaking.
Removes the global "transition: .2s" which leads to
animate every single element modification (causes OFBIZ-13104,
and possibly other problems/lagging).
Only apply these transitions to buttons to keep a nice effect
on hover. We may have to add other selectors if needed.

OFBIZ-12638
OFBIZ-12730
OFBIZ-13104
Copy link

sonarcloud bot commented Nov 6, 2024

@JacquesLeRoux JacquesLeRoux merged commit 2113d21 into apache:trunk Nov 7, 2024
6 checks passed
asfgit pushed a commit that referenced this pull request Nov 7, 2024
Use natives browser tooltips instead of handmade tooltips.
Fixes at least one glitch (OFBIZ-12730) due to styles leaking.
Removes the global "transition: .2s" which leads to
animate every single element modification (causes OFBIZ-13104,
and possibly other problems/lagging).
Only apply these transitions to buttons to keep a nice effect
on hover. We may have to add other selectors if needed.

OFBIZ-12638
OFBIZ-12730
OFBIZ-13104
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