wordpress-mobile / wordpress-mobile/AztecEditor-iOS
Generated HTML for edited links containing multibyte characters is weird.
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Swift
- Sterne
- 669
- Forks
- 153
- Ø Merge
- 11 Std. 27 Min.
- Gemergte PRs (30 T.)
- 4
Beschreibung
This is another side-effect of the underlying bug behind #853. #854 improves how the editor behaves when tapping on the "link" button in the toolbar, but the output HTML stills isn't great.
Expected behavior
After creating a link containing multibyte characters and then editing it, there's only one <a> element in the HTML output:
<a href="https://example.com">❤️ test 👍</a>
Actual behavior
Multiple <a> elements are present in the output:
<p>
<a href="http://example.com">❤️</a><a href="http://example.com"> test </a><a href="http://example.com">👍</a>
</p>
Steps to reproduce the behavior
- Insert a link with any URL, consisting of multi-byte characters (two emojis is probably the easiest).
- Put a cursor between the two characters.
- Start typing something, I usually add
testin between the two. - Look at the resulting HTML.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, das Problem in AztecEditor-iOS mit der Link-Schaltfläche der Symbolleiste zu reproduzieren: Erstelle einen Link um Multibyte-Zeichen, bearbeite ihn zwischen den Zeichen und überprüfe das erzeugte HTML. Die Korrektur ist vollständig, wenn der bearbeitete Text in einem einzigen -Element mit der erwarteten URL enthalten ist, statt in mehreren Elementen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- swift
- Bereich
- mobile
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100