Dotclear

Ticket #886 (closed defect: fixed)

Opened 14 years ago

Last modified 14 years ago

Editeur xhtml et les balises de type <hr />, <br />

Reported by: MathieuM Owned by: xave
Priority: normal Milestone: 2.2
Component: module:core Version: 2.1
Severity: minor Keywords:
Cc:

Description

Lorsque l'on se sert de l'éditeur en mode xhtml et "source", si l'on saisit une balise du type <hr class="maclasse" />, l'éditeur va automatiquement transformer cette dernière en <hr class="maclasse"> après enregistrement.

Ce n'est pas méchant mais ça casse la validation xhtml.

Attachments

jsToolBar.wysiwyg.js.diff Download (268 bytes) - added by lmagnin 14 years ago.
patch
jsToolBar.wysiwyg.js.patch Download (1.2 KB) - added by lmagnin 14 years ago.
patch fonctionnel

Change History

comment:1 Changed 14 years ago by lmagnin

Celui-là doit semble bien tordu. Je pensais le résoudre en 5 minutes, et au bout d'une heure, je ne sais pas trop ce qui se passe. Au début, je pensais que c'était dans la classe hmtlfilter, mais je n'ai rien trouvé là…

Quelqu'un aurait une intuition ?

À noter aussi, que si la balise <hr /> n'a pas d'attribut, tout se passe bien, et que ce bug ne concerne pas toutes les balises "single". <img> fonctionne très bien...

comment:2 follow-up: ↓ 3 Changed 14 years ago by lmagnin

Ah ah, c'est l'éditeur qui fait ça. Où peut-on trouver les fichiers JsToolBar?.* non stupidement compressés ?

comment:3 in reply to: ↑ 2 Changed 14 years ago by Moe

Replying to lmagnin:

Où peut-on trouver les fichiers JsToolBar?.* non stupidement compressés ?

Là : http://dev.dotclear.org/2.0/browser/trunk/admin/js/jsToolBar

Changed 14 years ago by lmagnin

patch

comment:4 Changed 14 years ago by lmagnin

Merci Moe.

Je joins un patch qui corrige pour <hr /> et <br />.

En fait pour <img>, <input>, <param>, <area> et <col> DotClear? corrigeait déjà ça — c'est un bug dans innerHTML. Cette solution ne fonctionne que sur les balises qui sont toujours "single" et ne pourra pas s'appliquer aux balises qui parfois s'utilisent seules, parfois non. Je pense en particulier aux <canvas>, <video>, <audio> et consorts qui peuvent ou non avoir plusieurs encodage, ou un fallback…

Changed 14 years ago by lmagnin

patch fonctionnel

comment:5 Changed 14 years ago by xave

  • Status changed from new to closed
  • Resolution set to fixed

(In [3065]) Correcting <hr /> & <br /> in the wysiwyg editor (thanks to MathieuM.) Closes #886.

Note: See TracTickets for help on using tickets.

Sites map