Skip to content

android - fix parsing headings #69

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
exploIF merged 6 commits into from
Jun 10, 2025
Merged

android - fix parsing headings #69

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
9f975fc
fix: html output, setting up paragraph spans, parser
exploIF Jun 6, 2025
f68d684
fix: android parser - headings
exploIF Jun 9, 2025
f42469c
Merge remote-tracking branch 'origin/main' into android-fix-block-ele…
exploIF Jun 9, 2025
172a9c4
Merge remote-tracking branch 'origin/main' into android-fix-headings-…
exploIF Jun 9, 2025
e4b9aa5
Merge remote-tracking branch 'origin/android-fix-block-elements-parse…
exploIF Jun 9, 2025
731a618
Merge remote-tracking branch 'origin/main' into android-fix-headings-…
exploIF Jun 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
62 changes: 38 additions & 24 deletions android/src/main/java/com/swmansion/reactnativerichtexteditor/utils/EditorParser.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
import android.text.style.BackgroundColorSpan;
import android.text.style.ForegroundColorSpan;
import android.text.style.ParagraphStyle;
import android.text.style.RelativeSizeSpan;
import android.text.style.TypefaceSpan;

import com.swmansion.reactnativerichtexteditor.spans.EditorBlockQuoteSpan;
Expand Down Expand Up @@ -700,11 +699,11 @@ private void handleStartTag(String tag, Attributes attributes) {
} else if (tag.equalsIgnoreCase("strike")) {
start(mSpannableStringBuilder, new Strikethrough());
} else if (tag.equalsIgnoreCase("h1")) {
start(mSpannableStringBuilder, new H1());
startHeading(mSpannableStringBuilder, attributes, 1);
} else if (tag.equalsIgnoreCase("h2")) {
start(mSpannableStringBuilder, new H2());
startHeading(mSpannableStringBuilder, attributes, 2);
} else if (tag.equalsIgnoreCase("h3")) {
start(mSpannableStringBuilder, new H3());
startHeading(mSpannableStringBuilder, attributes, 3);
} else if (tag.equalsIgnoreCase("img")) {
startImg(mSpannableStringBuilder, attributes, mImageGetter, mStyle);
} else if (tag.equalsIgnoreCase("code")) {
Expand Down Expand Up @@ -747,11 +746,11 @@ private void handleEndTag(String tag) {
} else if (tag.equalsIgnoreCase("s")) {
end(mSpannableStringBuilder, Strikethrough.class, new EditorStrikeThroughSpan(mStyle));
} else if (tag.equalsIgnoreCase("h1")) {
end(mSpannableStringBuilder, H1.class, new EditorH1Span(mStyle));
endHeading(mSpannableStringBuilder, mStyle, 1);
} else if (tag.equalsIgnoreCase("h2")) {
end(mSpannableStringBuilder, H2.class, new EditorH2Span(mStyle));
endHeading(mSpannableStringBuilder, mStyle, 2);
} else if (tag.equalsIgnoreCase("h3")) {
end(mSpannableStringBuilder, H3.class, new EditorH3Span(mStyle));
endHeading(mSpannableStringBuilder, mStyle, 3);
} else if (tag.equalsIgnoreCase("code")) {
end(mSpannableStringBuilder, Code.class, new EditorInlineCodeSpan(mStyle));
} else if (tag.equalsIgnoreCase("mention")) {
Expand Down Expand Up @@ -902,18 +901,41 @@ private static void endCodeBlock(Editable text, RichTextStyle style) {

private void startHeading(Editable text, Attributes attributes, int level) {
startBlockElement(text, attributes, getMarginHeading());
start(text, new Heading(level));
}

private static void endHeading(Editable text, RichTextStyle style) {
// RelativeSizeSpan and StyleSpan are CharacterStyles
// Their ranges should not include the newlines at the end
Heading h = getLast(text, Heading.class);
if (h != null) {
setSpanFromMark(text, h, new RelativeSizeSpan(HEADING_SIZES[h.mLevel]),
new EditorBoldSpan(style));
switch (level) {
case 1:
start(text, new H1());
break;
case 2:
start(text, new H2());
break;
case 3:
start(text, new H3());
break;
default:
throw new IllegalArgumentException("Unsupported heading level: " + level);
}
}

private static void endHeading(Editable text, RichTextStyle style, int level) {
endBlockElement(text);

switch (level) {
case 1:
H1 lastH1 = getLast(text, H1.class);
setParagraphSpanFromMark(text, lastH1, new EditorH1Span(style));
break;
case 2:
H2 lastH2 = getLast(text, H2.class);
setParagraphSpanFromMark(text, lastH2, new EditorH2Span(style));
break;
case 3:
H3 lastH3 = getLast(text, H3.class);
setParagraphSpanFromMark(text, lastH3, new EditorH3Span(style));
break;
default:
throw new IllegalArgumentException("Unsupported heading level: " + level);
}
}

private static <T> T getLast(Spanned text, Class<T> kind) {
Expand Down Expand Up @@ -1254,14 +1276,6 @@ public Background(int backgroundColor) {
}
}

private static class Heading {
private final int mLevel;

public Heading(int level) {
mLevel = level;
}
}

private static class Newline {
private final int mNumNewlines;

Expand Down
Loading