Skip to content

Conversation

Sub6Resources
Copy link
Owner

@Sub6Resources Sub6Resources commented May 16, 2023

This involved adding more options to the Margins class, as well as refactoring the padding option to take a new HtmlPaddings object.

Fixes #1257

@codecov
Copy link

codecov bot commented May 16, 2023

Codecov Report

Patch coverage: 73.68% and project coverage change: +4.09 🎉

Comparison is base (496d1aa) 60.02% compared to head (103ce62) 64.11%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1270      +/-   ##
==========================================
+ Coverage   60.02%   64.11%   +4.09%     
==========================================
  Files          37       38       +1     
  Lines        2699     2979     +280     
==========================================
+ Hits         1620     1910     +290     
+ Misses       1079     1069      -10     
Impacted Files Coverage Δ
lib/src/style/padding.dart 51.02% <51.02%> (ø)
lib/src/style/margin.dart 64.00% <61.44%> (-19.79%) ⬇️
lib/src/style.dart 85.64% <76.92%> (+0.84%) ⬆️
lib/src/css_parser.dart 35.92% <89.92%> (+18.44%) ⬆️
lib/src/css_box_widget.dart 64.95% <91.30%> (+0.61%) ⬆️
lib/src/processing/margins.dart 93.87% <92.85%> (+0.85%) ⬆️
lib/src/builtins/styled_element_builtin.dart 92.42% <100.00%> (+0.20%) ⬆️
lib/src/style/length.dart 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

… Update CssBoxWidget to handle rtl marker boxes
@Sub6Resources Sub6Resources force-pushed the fix/rtl-list-markers branch from 18db545 to d9fe7d5 Compare May 18, 2023 02:36
@Sub6Resources Sub6Resources added this to the 3.0.0 milestone May 18, 2023
@Sub6Resources Sub6Resources self-assigned this May 18, 2023
@Sub6Resources Sub6Resources added the bugfix Fixes a bug in the package label May 18, 2023
@Sub6Resources Sub6Resources merged commit d709199 into master May 23, 2023
@Sub6Resources Sub6Resources deleted the fix/rtl-list-markers branch May 23, 2023 14:44
Luca-CZ added a commit to CZ-App/flutter_html_lite that referenced this pull request Nov 11, 2024
…rsion

 - **FIX**: start list items on a new line ([Sub6Resources#1281](Sub6Resources#1281)). ([496d1aa](Sub6Resources@496d1aa))
 - **FIX**: Add "display: Display.block" to table ([Sub6Resources#1278](Sub6Resources#1278)). ([6350f02](Sub6Resources@6350f02))
 - **FIX**: improve API for ExtensionContext and export marker.dart ([Sub6Resources#1273](Sub6Resources#1273)). ([27e33a9](Sub6Resources@27e33a9))
 - **FIX**: Cleaned up whitespace processing and added whitespace tests ([Sub6Resources#1267](Sub6Resources#1267)). ([cc00406](Sub6Resources@cc00406))
 - **FIX**: a tag should not style as link if href is not provided ([Sub6Resources#1265](Sub6Resources#1265)). ([d7247cb](Sub6Resources@d7247cb))
 - **FEAT**: Update CssBoxWidget to handle rtl marker boxes ([Sub6Resources#1270](Sub6Resources#1270)). ([d709199](Sub6Resources@d709199))
 - **FEAT**: support vertical-align in inline styles ([Sub6Resources#1266](Sub6Resources#1266)). ([fe896de](Sub6Resources@fe896de))
 - **FEAT**: Add WrapperExtension helper ([Sub6Resources#1264](Sub6Resources#1264)). ([2ffa1dd](Sub6Resources@2ffa1dd))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Fixes a bug in the package
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BUG] UL list dots rendered on the left for rtl text
1 participant