content: Support parsing and handling inline styles for KaTeX content

Author: rajveermalviya

lib/model/katex.dart

@@ -1,6 +1,9 @@
+import 'package:csslib/parser.dart' as css_parser;
+import 'package:csslib/visitor.dart' as css_visitor;
 import 'package:flutter/foundation.dart';
 import 'package:html/dom.dart' as dom;
 
+import '../log.dart';
 import 'content.dart';
 
 class KatexParser {
@@ -225,11 +228,72 @@ class KatexParser {
     }
     if (text == null && spans == null) throw KatexHtmlParseError();
 
+    final inlineStyles = _parseSpanInlineStyles(element);
+
     return KatexSpanNode(
       text: text,
-      styles: styles,
+      styles: inlineStyles != null
+        ? styles.merge(inlineStyles)
+        : styles,
       nodes: spans ?? const []);
   }
+
+  KatexSpanStyles? _parseSpanInlineStyles(dom.Element element) {
+    if (element.attributes case {'style': final styleStr}) {
+      final stylesheet = css_parser.parse('*{$styleStr}');
+      final topLevels = stylesheet.topLevels;
+      if (topLevels.length != 1) throw KatexHtmlParseError();
+      final topLevel = topLevels.single;
+      if (topLevel is! css_visitor.RuleSet) throw KatexHtmlParseError();
+      final rule = topLevel;
+
+      double? marginLeftEm;
+      double? marginRightEm;
+      double? paddingLeftEm;
+
+      for (final declaration in rule.declarationGroup.declarations) {
+        if (declaration is! css_visitor.Declaration) throw KatexHtmlParseError();
+        final property = declaration.property;
+
+        final expressions = declaration.expression;
+        if (expressions is! css_visitor.Expressions) throw KatexHtmlParseError();
+        if (expressions.expressions.length != 1) throw KatexHtmlParseError();
+        final expression = expressions.expressions.single;
+
+        switch (property) {
+          case 'margin-left':
+            marginLeftEm = _getEm(expression);
+            if (marginLeftEm != null) continue;
+
+          case 'margin-right':
+            marginRightEm = _getEm(expression);
+            if (marginRightEm != null) continue;
+
+          case 'padding-left':
+            paddingLeftEm = _getEm(expression);
+            if (paddingLeftEm != null) continue;
+
+          default:
+            // TODO handle more CSS properties
+            assert(debugLog('Unsupported CSS property: $property of type ${expression.runtimeType}'));
+        }
+      }
+
+      return KatexSpanStyles(
+        marginLeftEm: marginLeftEm,
+        marginRightEm: marginRightEm,
+        paddingLeftEm: paddingLeftEm,
+      );
+    }
+    return null;
+  }
+
+  double? _getEm(css_visitor.Expression expression) {
+    if (expression is css_visitor.EmTerm && expression.value is num) {
+      return (expression.value as num).toDouble();
+    }
+    return null;
+  }
 }
 
 enum KatexSpanFontWeight {
@@ -248,13 +312,20 @@ enum KatexSpanTextAlign {
 }
 
 class KatexSpanStyles {
+  double? marginLeftEm;
+  double? marginRightEm;
+  double? paddingLeftEm;
+
   String? fontFamily;
   double? fontSizeEm;
   KatexSpanFontStyle? fontStyle;
   KatexSpanFontWeight? fontWeight;
   KatexSpanTextAlign? textAlign;
 
   KatexSpanStyles({
+    this.marginLeftEm,
+    this.marginRightEm,
+    this.paddingLeftEm,
     this.fontFamily,
     this.fontSizeEm,
     this.fontStyle,
@@ -265,6 +336,9 @@ class KatexSpanStyles {
   @override
   int get hashCode => Object.hash(
     'KatexSpanStyles',
+    marginLeftEm,
+    marginRightEm,
+    paddingLeftEm,
     fontFamily,
     fontSizeEm,
     fontStyle,
@@ -275,6 +349,9 @@ class KatexSpanStyles {
   @override
   bool operator ==(Object other) {
     return other is KatexSpanStyles &&
+      other.marginLeftEm == marginLeftEm &&
+      other.marginRightEm == marginRightEm &&
+      other.paddingLeftEm == paddingLeftEm &&
       other.fontFamily == fontFamily &&
       other.fontSizeEm == fontSizeEm &&
       other.fontStyle == fontStyle &&
@@ -289,6 +366,9 @@ class KatexSpanStyles {
     if (this == _zero) return '${objectRuntimeType(this, 'KatexSpanStyles')}()';
 
     final args = <String>[];
+    if (marginLeftEm != null) args.add('marginLeftEm: $marginLeftEm');
+    if (marginRightEm != null) args.add('marginRightEm: $marginRightEm');
+    if (paddingLeftEm != null) args.add('paddingLeftEm: $paddingLeftEm');
     if (fontFamily != null) args.add('fontFamily: $fontFamily');
     if (fontSizeEm != null) args.add('fontSizeEm: $fontSizeEm');
     if (fontStyle != null) args.add('fontStyle: $fontStyle');
@@ -299,6 +379,9 @@ class KatexSpanStyles {
 
   KatexSpanStyles merge(KatexSpanStyles other) {
     return KatexSpanStyles(
+      marginLeftEm: other.marginLeftEm ?? marginLeftEm,
+      marginRightEm: other.marginRightEm ?? marginRightEm,
+      paddingLeftEm: other.paddingLeftEm ?? paddingLeftEm,
       fontFamily: other.fontFamily ?? fontFamily,
       fontSizeEm: other.fontSizeEm ?? fontSizeEm,
       fontStyle: other.fontStyle ?? fontStyle,

lib/widgets/content.dart

@@ -916,13 +916,33 @@ class _KatexSpan extends StatelessWidget {
       };
     }
 
+    var margin = EdgeInsets.zero;
+    final marginRightEm = styles.marginRightEm;
+    final marginLeftEm = styles.marginLeftEm;
+    if (marginRightEm != null && !marginRightEm.isNegative) {
+      margin += EdgeInsets.only(right: marginRightEm * em);
+    }
+    if (marginLeftEm != null && !marginLeftEm.isNegative) {
+      margin += EdgeInsets.only(left: marginLeftEm * em);
+    }
+
+    var padding = EdgeInsets.zero;
+    final paddingLeftEm = styles.paddingLeftEm;
+    if (paddingLeftEm != null && !paddingLeftEm.isNegative) {
+      padding += EdgeInsets.only(right: paddingLeftEm * em);
+    }
+
     if (textStyle != null || textAlign != null) {
       widget = DefaultTextStyle.merge(
         style: textStyle,
         textAlign: textAlign,
         child: widget);
     }
-    return widget;
+    return Container(
+      margin: margin != EdgeInsets.zero ? margin : null,
+      padding: padding != EdgeInsets.zero ? padding : null,
+      child: widget,
+    );
   }
 }