lists in "do text from xhtml()"

Bug #777890 reported by Luc Saffre
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Appy
Fix Released
Undecided
Unassigned

Bug Description

The xhtml() function now works fine for a lot of cases.
But there is still at least one problem: it doesn't show bulletted lists (<UL><LI>).
I made a test case which you can download from the Lino source repository:

http://code.google.com/p/lino/source/browse/#hg%2Ftests%2Fappy%2F1

See also comments in test.py

Revision history for this message
Gaëtan Delannay (gaetan-delannay) wrote :

Hi Luc,
Strange, if the root tag is "p" instead of "div" is works. I will try to solve this bug quickly!
Cheers
Gaetan

Revision history for this message
Luc Saffre (luc-saffre) wrote :

Tilt, so I can work around this by simply not giving a DIV tag as topmost element in my XHTML string. That's okay for me (I just need to find out how to have docutils not write this surrounding DIV tag). So for me the bug can be closed.

Revision history for this message
Luc Saffre (luc-saffre) wrote :

Oops, I changed my mind: don't close this bug.

Because the same problem occurs with XHTML fragments saved by TinyMCE. And there it would be really strange to remove the surrounding DIV tag from these fragments.

I have the feeling that it should not be necessary to to remove the <div> tag.
Isn't it strange that the XHTML() function wraps the given string into a <P>...</P>.
I tried to wrap it into a <DIV>...</DIV> instead of <P>, but that didn't change anything.

Revision history for this message
Gaëtan Delannay (gaetan-delannay) wrote :

I Luc,
Don't worry, this bug is still in the pipe. I've added a unit test for it; will try to solve it soon.
Cheers
Gaetan

Revision history for this message
Gaëtan Delannay (gaetan-delannay) wrote :

Hi Luc,
I've committed a fix for this, check http://bazaar.launchpad.net/~appy-dev/appy/trunk/revision/152, will be included in forthcoming 0.6.7...
Does it work??
Cheers
Gaetan

Changed in appy:
status: New → Fix Committed
Revision history for this message
Luc Saffre (luc-saffre) wrote :

Yes, it works! Thanks for the fix!

Changed in appy:
milestone: none → 0.6.7
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.