Making WordPress.org

Opened 8 months ago

Last modified 3 months ago

#4507 new defect

Glossary tooltips break HTML markup (take 2)

Reported by: SergeyBiryukov Owned by:
Milestone: Priority: normal
Component: Make (Get Involved) / P2 Keywords: has-screenshots


Previously: #4473

On https://make.wordpress.org/polyglots/2019/06/02/pte-request-for-kirki/#comment-283767, there's this part of a sentence:

[username] can not be the translation editor for the […] language

The new glossary tooltips interfere with the page, causing invalid markup and display issues:

… can not be the <span tabindex="0" class="glossary-item-container">translation editor<span class="glossary-item-hidden-content"><span class="glossary-item-header">Translation Editor</span> <span class="glossary-item-description">Translation editors can approve translations for projects. The GTE (General Translation Editor) and LM (Locale Manager) roles can add new users with the "Project Translation Editor" role that can approve translations for specific projects.</p>
<p>There are two different Translation Editor roles, see:</p>
<p> 	<a href="#general-translation-editor">General Translation Editor</a><br />
 	<a href="#project-translation-editor">Project Translation Editor</a><br />
<br /><a href="https://make.wordpress.org/polyglots/wp-admin/post.php?post=5951&amp;action=edit">Edit Entry</a></span></span></span> for the <a href="https://make.wordpress.org/polyglots/teams/?locale=uk" class="tag"><span class="tag-prefix">#</span>uk</a> Ukrainian language.

Two issues here:

  • <p> inside <span> is invalid.
  • Extra text is displayed on the page, instead of being in a tooltip.

See the screenshot. Tested in the latest Google Chrome and Microsoft Edge on Windows 10.

Attachments (3)

4507.PNG (42.9 KB) - added by SergeyBiryukov 8 months ago.
Screen Shot 2019-12-03 at 8.59.17 AM.png (157.0 KB) - added by dufresnesteven 3 months ago.
Screen Shot 2019-12-03 at 8.58.40 AM.png (241.8 KB) - added by dufresnesteven 3 months ago.

Download all attachments as: .zip

Change History (4)

8 months ago

#1 @dufresnesteven
3 months ago

Hi @SergeyBiryukov, :)

Is this still an issue? See attached.

Note: See TracTickets for help on using tickets.