Revert some change from #2002
This commit is contained in:
parent
9389cfe7a3
commit
d0532bb9a9
@ -10,7 +10,6 @@ Improvements 🙌:
|
|||||||
|
|
||||||
Bugfix 🐛:
|
Bugfix 🐛:
|
||||||
- Display name not shown under Settings/General (#1926)
|
- Display name not shown under Settings/General (#1926)
|
||||||
- Wrong markdown parsing (#350, #1375, #1939, #1982)
|
|
||||||
- Words containing my name should not trigger notifications (#1781)
|
- Words containing my name should not trigger notifications (#1781)
|
||||||
- Fix changing language issue
|
- Fix changing language issue
|
||||||
- Fix FontSize issue (#1483, #1787)
|
- Fix FontSize issue (#1483, #1787)
|
||||||
|
@ -53,7 +53,7 @@ internal class MarkdownParser @Inject constructor(
|
|||||||
// According to https://matrix.org/docs/spec/client_server/latest#m-room-message-msgtypes:
|
// According to https://matrix.org/docs/spec/client_server/latest#m-room-message-msgtypes:
|
||||||
// The plain text version of the HTML should be provided in the body.
|
// The plain text version of the HTML should be provided in the body.
|
||||||
// But it caused too many problems so it has been removed in #2002
|
// But it caused too many problems so it has been removed in #2002
|
||||||
TextContent(text, cleanHtmlText.trim())
|
TextContent(text, cleanHtmlText.postTreatment())
|
||||||
} else {
|
} else {
|
||||||
TextContent(text)
|
TextContent(text)
|
||||||
}
|
}
|
||||||
@ -61,4 +61,15 @@ internal class MarkdownParser @Inject constructor(
|
|||||||
|
|
||||||
private fun isFormattedTextPertinent(text: String, htmlText: String?) =
|
private fun isFormattedTextPertinent(text: String, htmlText: String?) =
|
||||||
text != htmlText && htmlText != "<p>${text.trim()}</p>\n"
|
text != htmlText && htmlText != "<p>${text.trim()}</p>\n"
|
||||||
|
|
||||||
|
/**
|
||||||
|
* The parser makes some mistakes, so deal with it here
|
||||||
|
*/
|
||||||
|
private fun String.postTreatment(): String {
|
||||||
|
return this
|
||||||
|
// Remove extra space before and after the content
|
||||||
|
.trim()
|
||||||
|
// There is no need to include new line in an html-like source
|
||||||
|
.replace("\n", "")
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user