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

Hover over Glossary term adds text to base of page #758

Closed
garretthyder opened this issue Jun 28, 2017 · 5 comments
Closed

Hover over Glossary term adds text to base of page #758

garretthyder opened this issue Jun 28, 2017 · 5 comments
Labels
[Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature is broken.
Milestone

Comments

@garretthyder
Copy link

Hello,

Minor bug I came across when you hover over a Glossary term the tooltip appears but also at the base of the page some text gets injected;
screen shot 2017-06-28 at 2 52 01 pm
(find the verbcolour as the base of the screenshot)

Inspecting that section here's the HTML that string comes from;

  • verbcolour

This only happens when you hover so must be a JS issue, from looking at the code it looks like that's tooltip template HTML, so maybe the .ui-helper-hidden-accessible class should have a display none on it??

Cheers
P.S. Here's the string I was looking at when it occurred;
https://translate.wordpress.org/projects/wp-plugins/the-events-calendar/stable/en-ca/default?filters%5Bstatus%5D=either&filters%5Boriginal_id%5D=3887469&filters%5Btranslation_id%5D=48304327
P.P.S. I made sure to disable GlotDict to check the issue was actually GlotPress and not GlotDict

@toolstack
Copy link
Contributor

Confirmed, I've replicated this on my development install.

@toolstack toolstack added the [Type] Bug An existing feature is broken. label Jun 28, 2017
@toolstack toolstack added this to the 2.4 milestone Jun 28, 2017
@toolstack
Copy link
Contributor

It's a bug in jQuery UI, version 1.10.4 works fine, both 1.11.4 and 1.12.4 exhibit this behaviour.

@garretthyder
Copy link
Author

Thanks @toolstack I guess a proper fix will come once jQuery UI resolves the issue. For now I found on forums it's suggested to just do a display none on the .ui-helper-hidden-accessible class.

@toolstack
Copy link
Contributor

Could you provide the link to the work around?

@garretthyder
Copy link
Author

@toolstack toolstack added the [Status] In Progress Tracking issues with work in progress label Jun 30, 2017
ocean90 added a commit that referenced this issue Jul 1, 2017
ocean90 added a commit that referenced this issue Jul 1, 2017
@toolstack toolstack modified the milestones: 3.0, 2.4 Aug 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature is broken.
Projects
None yet
Development

No branches or pull requests

2 participants