Modify

Opened 16 years ago

Last modified 9 years ago

#3340 new defect

Wysiwyg: Dislikes some nested definitions

Reported by: anonymous Owned by: Jun Omae
Priority: normal Component: TracWysiwygPlugin
Severity: normal Keywords: WYSIWYG definition
Cc: Trac Release: 0.11

Description

 * C
  l(s)::

 * C

results in Failed to activate the wysiwyg editor.

Attachments (0)

Change History (6)

comment:1 Changed 14 years ago by Adrian Fritz

Keywords: WYSIWYG added; Wysiwyg removed

comment:2 Changed 13 years ago by anonymous

Same error occurs with a table where one cell contains an empty bold text (i.e. '''''').

comment:3 in reply to:  2 Changed 13 years ago by Jun Omae

Replying to anonymous:

Same error occurs with a table where one cell contains an empty bold text (i.e. '''''').

It works for me. What version of tracwysiwyg do you use?

I tried the latest;

  1. Input || '''''' || to textarea
  2. Click wysiwyg radio button
  3. Wysiwyg editor is shown without errors

comment:4 Changed 9 years ago by Jun Omae

#12450 was closed as duplicate.

comment:5 Changed 9 years ago by xoneca@…

So, I tried to debug this with the following snippet:

 First def:: 
  - first list
 Second def:: 
  - second list

I have tracked the bug to the function handleList() when parsing the second list (last line of that snippet). There the holder is the last dd element, so when calling

list = getSelfOrAncestor(holder, "li");

That returns null and blows at the next line, on appendBogusLineBreak().

comment:6 Changed 9 years ago by xoneca@…

Tried also the snippet in the ticket description and fails in the same function, but this time holder is a DocumentFragment element.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain Jun Omae.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.