Ticket #18419 (closed defect (bug): fixed)

Opened 6 months ago

Last modified 4 months ago

TinyMCE Anchor doesn't render in Visual Editor

Reported by: MadtownLems Owned by: jakemgold
Priority: normal Milestone: 3.3
Component: TinyMCE Version: 3.2.1
Severity: normal Keywords:
Cc:

Description

I'm adding the Anchor button to TinyMCE via plugin with the normal method.

Excerpt: array_push($buttons, "anchor"); return $buttons;

The Anchor button displays fine. Clicking it presents a dialog box fine. Completing the dialog box inserts the necessary HTML. However, the problem is that a little picture of an anchor is no longer visible in the Visual Editor

Attachments

mce_anchor_style.patch Download (1.1 KB) - added by jakemgold 4 months ago.
Fixes anchor background image

Change History

  • Status changed from new to closed
  • Resolution set to fixed
  • Milestone changed from Awaiting Review to 3.3

Should be fixed in [18660], feel free to reopen if that still happens.

  • Status changed from closed to reopened
  • Resolution fixed deleted

3.3-beta-1 still has this issue. Re-opening like you said I could :)

  • Owner set to jakemgold
  • Status changed from reopened to accepted

comment:4 follow-up: ↓ 9   jakemgold4 months ago

Turns out this regressed circa 3.2 from 3.1; the CSS background image stopped pointing up a level to the default skin image folder. Attaching patch.

Fixes anchor background image

The other issue is that the anchor is rendered, but the default CSS places it too low, since it expects it to sit in the autosave/word count table markup. The default CSS should place it inside the editor. The main editor should customize its position for the custom case.

Whoops, my comment is a completely different bug.

Confirmed that the background image is missing when you add in an anchor button.

  • Status changed from accepted to closed
  • Resolution set to fixed

In [19190]:

Correct TinyMCE anchor (as in <a name>) image. props jakemgold, fixes #18419.

comment:9 in reply to: ↑ 4   SergeyBiryukov4 months ago

Replying to jakemgold:

Turns out this regressed circa 3.2 from 3.1; the CSS background image stopped pointing up a level to the default skin image folder.

http://core.trac.wordpress.org/changeset/17625#file64

Note: See TracTickets for help on using tickets.