[wp-trac] [WordPress Trac] #26778: Visual editor not showing wysywig editor and white text on white background
WordPress Trac
noreply at wordpress.org
Mon Mar 31 01:20:35 UTC 2014
#26778: Visual editor not showing wysywig editor and white text on white background
--------------------------+----------------------
Reporter: marcreck | Owner:
Type: defect (bug) | Status: closed
Priority: normal | Milestone: 3.9
Component: TinyMCE | Version: 3.8
Severity: normal | Resolution: invalid
Keywords: | Focuses:
--------------------------+----------------------
Comment (by tollmanz):
Replying to [comment:29 toscho]:
> I know it isn’t valid. But it breaks backwards-compatibility, and it
doesn’t even throw a JavaScript error to explain it. Just leaving people
alone with such a change is not a good solution.
I would argue that this is not a regression. In
[https://github.com/WordPress/WordPress/blob/3.8.1/wp-includes/general-
template.php#L1878 3.8.1], the docblock states that the {{{$editor_id}}}
value can only be {{{/[a-z]+/}}}. Before I realized this in 3.8, I used a
bracket in {{{$editor_id}}} in an instance of {{{wp_editor}}} (not
realizing that I could set the {{{textarea_name}}} value independently).
This broke the link button for me because the {{{$editor_id}}} value is
used as part of a [https://github.com/WordPress/WordPress/blob/3.8.1/wp-
includes/js/wplink.js#L66 jQuery selector], which does not allow brackets.
I had to override this function for it to work.
WordPress 3.9 broke this further and I experienced the problems that you
did; however, because WordPress' docs clearly advised that brackets are
not allowed and because the editor did not work fully with brackets in the
previous version, I do not see this as a regression given that the updated
code is acting on assumptions that are articulated for appropriate use of
the API. My solution was to update my code as it is clear that there is a
good reason why brackets are not allowed for the {{{$editor_id}}} value.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/26778#comment:31>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list