Skip to content
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.

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

feat: Update CssBoxWidget to handle rtl marker boxes #1270

Merged
merged 4 commits into from
May 23, 2023

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