Sub6Resources / flutter_html

A Flutter widget for rendering static html as Flutter widgets (Will render over 80 different html tags!)
https://pub.dev/packages/flutter_html
MIT License
1.79k stars 860 forks source link

[BUG] UL list dots rendered on the left for rtl text #1257

Closed dzsonni closed 1 year ago

dzsonni commented 1 year ago

Describe the bug: The list indicator is drawn on the left of UL(and possibly OL).

HTML to reproduce the issue:

<ul dir="rtl">
<li>عند تلقي المشورة أو الخضوع لفحص جسدي، يحق للمستفيد/للمستفيدة من الخدمات أن يعرف/تعرف دور كل فرد في الغرفة (مثل الأفراد الذين يخضعون للتدريب، والمشرفين، والمدربين، والباحثين، إلخ). إذا كان وجود أفراد يخضعون للتدريب ضروريًا، ينبغي الحصول على إذن مسبق من المستفيد/المستفيدة من الخدمات.</li>
<li>للمستفيد(ة) من الخدمات الحق في معرفة نوع الفحص الجسدي الذي سيتم إجراؤه مسبقًا. يحق للمستفيد(ة) من الخدمات أيضًا رفض أي فحص إذا لم يشعر/تشعر بالراحة تجاهه أو أن يطلب/تطلب قيام مقدم رعاية صحية آخر بإجراء هذا الفحص.</li>
<li>المناقشات المتعلقة بالحالة التي تُعقد بحضور المستفيد(ة) (لا سيما في مرافق التدريب) ينبغي أن يشارك/تشارك فيها المستفيد(ة) ويقر/تقر بها.</li>
</ul>

Html widget configuration: Default configuration. From latest master branch.

Expected behavior: The indicators should be on the right, or the beginning of the content.

Screenshots: image

Device details and Flutter/Dart/flutter_html versions: Latest master. flutter_html-c75e0dfb1be6cb79748f719487043d12bc330c60

Stacktrace/Logcat

Additional info:

A picture of a cute animal (not mandatory but encouraged)

image

Sub6Resources commented 1 year ago

Opened a PR with a fix: #1270

dzsonni commented 1 year ago

@Sub6Resources Checked the PR branch, fixes the issue. Thank you. This can be closed after it is merged.