[FF] Image2 in Firefox: Problems with autoparagraphing

[Olek Nowodziński]

1. Open samples/plugins/image2/image2.html
2. Use the following code:

   <figure class="caption" style="float:right"><img alt="Saturn V" src="assets/image1.jpg" width="200" />
   <figcaption>Roll out of Saturn V on launch pad</figcaption>
   </figure>
   

3. Switch to WYSIWYG and back to source:

   <p>&nbsp;
   <figure class="caption" style="float:right"><img alt="Saturn V" src="assets/image1.jpg" width="200" />
   <figcaption>Roll out of Saturn V on launch pad</figcaption>
   </figure>
   </p>
   

4. Switch to WYSIWYG and back to source:

   <p>&nbsp;</p>
   
   <figure class="caption" style="float:right"><img alt="Saturn V" src="assets/image1.jpg" width="200" />
   <figcaption>Roll out of Saturn V on launch pad</figcaption>
   </figure>
   
   <p>&nbsp;</p>
   

5. 3. and 4. should never happen.

[Piotrek Koszuliński]

This is caused by FF's bug which I reported in ​https://bugzilla.mozilla.org/show_bug.cgi?id=911201

Firefox appends bogus <br> before non-editable element what then affects parser which wraps this in a paragraph. We cannot fix (without hacking) what FF does, but we should fix the second part - broken parser. We can also add a small hack which will remove bogus <br> from the beginning of document.

[Piotrek Koszuliński]

Related issue: #4638.

[Marek Lewandowski]

Added keyword for firefox

[Piotrek Koszuliński]

Pushed ​t/10854 on dev and tests.

We decided to strip leading <br> in htmlDataProcessor#toDataFormat.

[Frederico Caldeira Knabben]

Please just have the regex simplified to this:

/^<br *\/?>/i

[Piotrek Koszuliński]

Fixed on major with ​git:2d9c3ff on dev and with 2e1b8cb on tests.