Gut. Wenn da wenigstens Übereinstimmung besteht, ist es nur Gewöhnungssache.Holger wrote:... kann aber zum CKEditor etwas sagen: momentan wir da auch ein <span> genommen:
Auf der Beispiel-Seite steht gleich im ersten Beispiel:frase wrote:Es ist zu überlegen, ob <span> die beste Wahl ist. Alle FA-Beispiele, die so zu finden sind, verwenden <i> bzw. <em>. Es wird natürlich auch erwähnt, dass auch andere Tags möglich sind. Soweit alles OK - falsch ist es auf keinen Fall. Wie wird das im CKEditor? Haben wir dann Unterschiede? Warum macht CMSimple_XH wieder alles anders als die anderen? Gibt das nicht Verwirrung?
Ob nun also ein <i> oder <span> verwendet wird, sollte nicht wirklich verwirrend sein. Die Plugins für TinyMCE 4 und CKEditor verwenden beide <span>; warum, weiß ich gar nicht, ich habe sie ja nicht geschrieben, sondern nur ein wenig angepasst. Ich könnte das bei Bedarf wohl auch ändern – bin selbst nicht sicher, was nun besser ist.Font Awesome is designed to be used with inline elements (we like the <i> tag for brevity, but using a <span> is more semantically correct).
"<span> is more semantically correct" ist doch ein gutes Argument.cmb wrote:Ich könnte das bei Bedarf wohl auch ändern – bin selbst nicht sicher, was nun besser ist.
Das liegt daran, dass zumindest der TinyMCE 4 so konfiguriert ist, dass er <i> einfach entfernt. Da kann ich in init_fontawesome.js nachbessern; warum sollte <i> nicht erlaubt sein! Weiß aber nicht wie das (aktuell) beim CKEditor ist. Zumindest wenn beide <i> akzeptieren, müsste wohl auch nichts dokumentiert werden. Siehe https://github.com/cmb69/fa_xh/issues/22.frase wrote:Mir ist es bei Tests passiert, dass meine hineinkopierten Beispiele sang- und klanglos vom Tiny entfernt wurden.
Code: Select all
extended_valid_elements: 'span[*],i[*]'Du bist gut, Christoph. Die extended_valid_elements verwende ich schon, seit ich mit FA rumspiele.cmb wrote:Quick-Fix für init_fontawesome.js
Der wird leere <i> - Tags auch entfernen. Das sollte man ihm aber mittelscmb wrote:Weiß aber nicht wie das (aktuell) beim CKEditor ist.
Code: Select all
CKEDITOR.dtd.$removeEmpty[\'i\']Dass <span> in TinyMCE verwendet wird anstatt <i> war sofort klar, man musste es nur mal in einem i-Tag probieren und der wurde dann gelöscht. So war das bein TinyMCE3 auch mit html5-Tags.frase wrote:Knollsen und Ludwig wollen sich mit FA beschäftigen. Da ist eine bessere Erklägung nötig.
Ludwig's XH-Templates for MultiPage & OnePageDas sollte nur eingefügt werden, wenn ein FA-Icon in einen leeren Absatz eingefügt wird. Das ist ein Workaround für einen Bug in TinyMCE 4.lck wrote:Mich hat eher das Leerzeichen hinter den FA-Icons nachdenklich gemacht, sollte/muss das sein? Ich denke es kann nicht schaden und wird in den meisten Fällen sowieso benötigt.