WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 3 years ago

#15111 closed defect (bug) (fixed)

Color selection spans no longer nest inside links

Reported by: markel Owned by: nacin
Milestone: 3.1 Priority: normal
Severity: normal Version: 3.1
Component: TinyMCE Keywords:
Focuses: Cc:

Description

Tested in most recent nightly (as of 10/13/10).

When adding a text color to a link, the span tags are placed outside the link, like this:

<span style="color: #339966;"><a href="http://wordpress.org/">link</a></span>

The previous behavior (confirmed in 2.9.2) was to nest the span tags within the link, like this:

<a href="http://wordpress.org/"><span style="color: #339966;">link</span></a>

Users can no longer influence the color of a link using only the Visual Editor.

Steps to reproduce/test against other versions:

  1. Add some text.
  2. Make it into a link using the Visual Editor.
  3. Select the text again and select a color for it using the Visual Editor.

Change History (14)

comment:1 andrewspittle4 years ago

  • Cc andrewspittle added

comment:2 westi4 years ago

  • Cc westi added
  • Milestone changed from Awaiting Review to 3.1

comment:3 ryan4 years ago

Still an issue.

comment:4 ocean904 years ago

  • Component changed from Editor to TinyMCE

comment:5 ryan4 years ago

  • Milestone changed from 3.1 to Future Release

comment:6 ryan4 years ago

  • Milestone changed from Future Release to 3.1

comment:7 nacin4 years ago

Request for help sent upstream.

comment:8 nacin4 years ago

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

(In [16950]) TinyMCE 3.3.9.3-dev. fixes #15111.

comment:10 demetris4 years ago

Reopening to note that TinyMCE 3.3.9.3 final was released yesterday.

Do we want to update to that or are we staying with our 3.3.9.3-dev?

comment:11 demetris4 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:12 nacin4 years ago

  • Owner changed from azaozz to nacin
  • Status changed from reopened to accepted

Thanks for the heads up. I just got the notification this morning.

Will inspect the changes (if any) and handle today.

comment:13 nacin4 years ago

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

There were no additional changes. We're running 3.3.9.3 final.

Note: See TracTickets for help on using tickets.