Closed (won't fix)
Project:
Popup
Version:
6.x-3.0
Component:
Miscellaneous
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Feb 2010 at 20:28 UTC
Updated:
22 Aug 2012 at 00:59 UTC
Hi,
When I insert a popup filter (ie [popup ...]) inside a text (block body), the part of text ABOVE this popup filter is wrapped in
tags. So a popup filter can't be really inline.
Is there a solution?
Thanks in advance!
Comments
Comment #1
brulain commentedHi,
I have created an input format with 'Line break converter' unchecked.
Bye.
Comment #2
boclodoa commented(I need the line break converter :S)
I'm making a digital version of an impressed document and I want to use this module to replace the footnotes. My problem is that sometimes before the [popup ...] appears a line break, so I get my lines broken. ¿can I control this behavior with a configuration parameter?
Example:
and the result is
Comment #3
boclodoa commentedhelp please
Comment #4
boclodoa commentedI've tried to solve it my self but is not as easy as I suspected.
The base problem is: this module substitutes [popup ...] to
and a 'div' tag inside a 'p' tag is (in principle) not allowed, then the browser tries to fix it but don't display what I want.
other discussions on this topic:
http://www.webmasterworld.com/html/3560080.htm
http://blog.programmingsolution.net/html/html-div-tag-inside-html-p-para...
http://www.webmasterworld.com/forum83/2481.htm
http://www.cs.tut.fi/~jkorpela/html/nesting.html
So, even if the drupal filter configuration is working fine, the browser gets confused and the firebug too, you can see that is different the HTML code in firebug, and the HTML you can see with Ctrl+u.
Then my suggestion is to change the code of this module for use 'span' instead of 'div'. This is not single line change because it's necessary to change all the references (css and js) to the 'div' tags with references to 'span' tags.
Comment #5
Scheepers de Bruin commentedActually it is a two line change.
Edit popup-element.tpl.php in whatever popup style you are using, and change:
to
I cannot recall why I specifically made that a div (there was some reason, possibly cross-browseriness), but if you're satisfied that that works, I may change it in the new release.
Please let me know.
Comment #6
Scheepers de Bruin commentedNo longer supported