Skip to content

Commit f66a4cd

Browse files
committed
content: Render KaTeX by default, as planned for the upcoming launch
1 parent 84186c6 commit f66a4cd

File tree

3 files changed

+30
-5
lines changed

3 files changed

+30
-5
lines changed

lib/model/settings.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ enum BoolGlobalSetting {
133133
placeholderIgnore(GlobalSettingType.placeholder, false),
134134

135135
/// An experimental flag to toggle rendering KaTeX content in messages.
136-
renderKatex(GlobalSettingType.experimentalFeatureFlag, false),
136+
renderKatex(GlobalSettingType.experimentalFeatureFlag, true),
137137

138138
/// An experimental flag to enable rendering KaTeX even when some
139139
/// errors are encountered.

test/model/content_test.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -512,7 +512,7 @@ class ContentExample {
512512
static final mathInline = ContentExample.inline(
513513
'inline math',
514514
r"$$ \lambda $$",
515-
expectedText: r'\lambda',
515+
expectedText: r'λ',
516516
'<p><span class="katex">'
517517
'<span class="katex-mathml"><math xmlns="http://www.w3.org/1998/Math/MathML"><semantics><mrow><mi>λ</mi></mrow>'
518518
'<annotation encoding="application/x-tex"> \\lambda </annotation></semantics></math></span>'
@@ -532,7 +532,7 @@ class ContentExample {
532532
static const mathBlock = ContentExample(
533533
'math block',
534534
"```math\n\\lambda\n```",
535-
expectedText: r'\lambda',
535+
expectedText: r'λ',
536536
'<p><span class="katex-display"><span class="katex">'
537537
'<span class="katex-mathml"><math xmlns="http://www.w3.org/1998/Math/MathML" display="block"><semantics><mrow><mi>λ</mi></mrow>'
538538
'<annotation encoding="application/x-tex">\\lambda</annotation></semantics></math></span>'

test/widgets/content_test.dart

Lines changed: 27 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -558,7 +558,11 @@ void main() {
558558
group('MathBlock', () {
559559
testContentSmoke(ContentExample.mathBlock);
560560

561-
testWidgets('displays KaTeX source; experimental flag default', (tester) async {
561+
testWidgets('displays KaTeX source; experimental flag disabled', (tester) async {
562+
addTearDown(testBinding.reset);
563+
final globalSettings = testBinding.globalStore.settings;
564+
await globalSettings.setBool(BoolGlobalSetting.renderKatex, false);
565+
562566
await prepareContent(tester, plainContent(ContentExample.mathBlock.html));
563567
tester.widget(find.text(r'\lambda', findRichText: true));
564568
});
@@ -1143,6 +1147,23 @@ void main() {
11431147
testContentSmoke(ContentExample.mathInline);
11441148

11451149
testWidgets('maintains font-size ratio with surrounding text', (tester) async {
1150+
const html = '<span class="katex">'
1151+
'<span class="katex-mathml"><math xmlns="http://www.w3.org/1998/Math/MathML"><semantics><mrow><mi>λ</mi></mrow>'
1152+
'<annotation encoding="application/x-tex"> \\lambda </annotation></semantics></math></span>'
1153+
'<span class="katex-html" aria-hidden="true"><span class="base"><span class="strut" style="height:0.6944em;"></span><span class="mord mathnormal">λ</span></span></span></span>';
1154+
await checkFontSizeRatio(tester,
1155+
targetHtml: html,
1156+
targetFontSizeFinder: mkTargetFontSizeFinderFromPattern(r'λ'));
1157+
}, skip: true // TODO(#46): adapt this test
1158+
// (it needs a more complex targetFontSizeFinder;
1159+
// see other uses in this file for examples.)
1160+
);
1161+
1162+
testWidgets('maintains font-size ratio with surrounding text, when showing TeX source', (tester) async {
1163+
addTearDown(testBinding.reset);
1164+
final globalSettings = testBinding.globalStore.settings;
1165+
await globalSettings.setBool(BoolGlobalSetting.renderKatex, false);
1166+
11461167
const html = '<span class="katex">'
11471168
'<span class="katex-mathml"><math xmlns="http://www.w3.org/1998/Math/MathML"><semantics><mrow><mi>λ</mi></mrow>'
11481169
'<annotation encoding="application/x-tex"> \\lambda </annotation></semantics></math></span>'
@@ -1152,7 +1173,11 @@ void main() {
11521173
targetFontSizeFinder: mkTargetFontSizeFinderFromPattern(r'\lambda'));
11531174
});
11541175

1155-
testWidgets('displays KaTeX source; experimental flag default', (tester) async {
1176+
testWidgets('displays KaTeX source; experimental flag disabled', (tester) async {
1177+
addTearDown(testBinding.reset);
1178+
final globalSettings = testBinding.globalStore.settings;
1179+
await globalSettings.setBool(BoolGlobalSetting.renderKatex, false);
1180+
11561181
await prepareContent(tester, plainContent(ContentExample.mathInline.html));
11571182
tester.widget(find.text(r'\lambda', findRichText: true));
11581183
});

0 commit comments

Comments
 (0)