Opened 21 months ago

Closed 19 months ago

Last modified 19 months ago

#18419 closed defect (bug) (fixed)

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 (1)

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

Download all attachments as: .zip

Change History (10)

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

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

  • Resolution fixed deleted
  • Status changed from closed to reopened

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   jakemgold19 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.

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

In [19190]:

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

comment:9 in reply to: ↑ 4   SergeyBiryukov19 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.