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.

Sign up for GitHub

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

Jump to bottom

RTL Column Support #574

Closed
andreia-telenav opened this issue Feb 13, 2018 · 4 comments
Closed

RTL Column Support #574

andreia-telenav opened this issue Feb 13, 2018 · 4 comments
Labels
bug Existing features not working as expected
Milestone
43

Comments

@andreia-telenav
Copy link

Hi,
we are using this library to generate the arabic version of our reports.
It works very well so far.
But we encountered a problem with column display.
We are using the following css coding:
.columns {
column-count: 2;
column-gap: 10px;
}
with configuration.
When it comes to display the columns in RTL direction, the columns are displaced to right.
I came up with a solution changing the blocks.py file
line 241 doing the following changes:

if containing_block.style['direction'] == 'rtl':
                column_box.position_x -= i * (width + style['column_gap'])
            else:
                column_box.position_x += i * (width + style['column_gap'])

If you consider it to be a good fix, could it be included in the current version?
Or if you are planning to come up with an other solution, please let me know.

Thank you.
@liZe
Copy link
Member

liZe commented Feb 13, 2018

Hi,
we are using this library to generate the arabic version of our reports.
It works very well so far.

Wow, that's good news as RTL support is pretty bad in WeasyPrint!

If you consider it to be a good fix, could it be included in the current version?

That's a good start, thank you! I'll push a slightly different fix, but that's really helpful. Tell me if it's OK for you!

liZe added a commit that referenced this issue Feb 13, 2018
@liZe
Respect text direction for multicolumn layout
897d440 
Related to #574.
liZe added a commit that referenced this issue Feb 13, 2018
@liZe
Respect text direction for multicolumn layout
1b62ca5 
Related to #574.
@andreia-telenav
Copy link
Author

Hi, thank you very much for your reply.
Unfortunately the content is still displaced.
I would like to send you some images to show you how it looks on my side, but it seems that I can't attach them here.
Can you take a look please?
Thank you again for your help.

@liZe
Copy link
Member

liZe commented Feb 16, 2018

@andreia-telenav could you provide an HTML+CSS sample?

@andreia-telenav
Copy link
Author

Hi,
here is the html + css code:

<html dir="rtl">
    <head>
        <style>
            @page {
                size: A4 portrait;
                margin: 10mm;
            }
            .columns {
                column-count: 2;
                column-gap: 10mm;
            }
        </style>
    </head>
    <body>
        <h1>Example</h1>
        <div class="columns">
                <p>
                        يتضمن هذا التقرير ثلاثة متغيرات CYP2C19 المرتبطة
         مع انخفاض وظيفة انزيم (تسمى * 2، * 3،)
         ومتغير واحد يرتبط مع زيادة
         وظيفة انزيم (* 17). الناس دون أي من هذه
         المتغيرات هي الأكثر احتمالا أن يكون ما يسمى * 1
         الإصدار، الإصدار الأكثر شيوعا من CYP2C19
         الجينات. ومن الممكن أيضا أن تكون أخرى نادرة جدا
         CYP2C19 المتغيرات غير المدرجة في هذا التقرير
         قد تؤثر على فعالية كلوبيدوغريل.
                     </p>
                     <p>
                         تنفيذ السريرية علم الوراثة الدوائي
         كونسورتيوم (كبيك) (2) يصنف الناس مع اثنين
                     </p>
                     <p>
                             (مثل * 2 / * 2، * 2 / * 3 وما إلى ذلك)
             "الفقراء" CYP2C19 الأيض والناس مع واحد
             انخفاض النشاط النشاط كمستقلبات "وسيطة"
             (* 1 / * 2، * 1 / * 3،). العلاج كلوبيدوغريل قد لا
             أن تكون فعالة تماما للناس في أي من هذه
             المجموعات. وبسبب هذا، تشير إرشادات كبيك
             أن مقدمي الرعاية الصحية النظر في بديل ل
             كلوبيدوغريل عند علاج أعراض النوبات القلبية
             في الناس مع هذه المتغيرات. انتشار
             تختلف المتغيرات ذات النشاط المنخفض حسب العرق، ولكنها تختلف
             تميل إلى أن تكون أكثر شيوعا في سكان شرق آسيا.
             بعض الناس لديهم مزيج من انخفاض وظيفة واحدة
             ومتغير واحد واحد متغير الوظيفة
             (مثل * 2 / * 17). على الرغم من أن تأثير هذا الجمع
             على فعالية كلوبيدوغريل ليست مفهومة تماما
             ويمكن أن تختلف بين الأفراد، المبادئ التوجيهية كبيك
             أيضا تصنيف هؤلاء الناس باسم "الأيض وسيطة"
             والتوصية بأن مقدمي الرعاية الصحية
             النظر في دواء بديل لهم.
                         </p>
                         <p>
                            الناس الذين ليس لديهم أي من انخفاض وظيفة
             من المرجح أن تكون المتغيرات نموذجية أو متزايدة
             CYP2C19 الأيض وبالتالي نموذجية أو ربما
             زيادة فعالية كلوبيدوغريل. هذه الأنماط الجينية
             تشمل * 1 / * 1 و * 1 / * 17 و * 17 / * 17. اعتمادا
             على العرق، حوالي 35 في المئة الى 50 في المئة من
             الناس لديهم * النمط الجيني * 1 / *، وتصنف كبيك
             لهم "مستقلبات واسعة". الناس
             مع النظر في النمط الجيني * 1 / * 17 أو * 17 / * 17
             "مستقلبات أولترارابيد." تدعم إرشادات كبيك
             الجرعات القياسية (كما هو مبين في التسمية المخدرات) ل
             سواء الأيض واسعة و أولترارابيد.
                         </p>
        </div>
    </body>
</html>

Here is the pdf result:
screen shot 2018-02-16 at 11 33 01 am

Thank you for your help.

liZe added a commit that referenced this issue Feb 20, 2018
@liZe
Don't translate rtl columns twice
092b03b 
Fix #574.
@liZe liZe closed this as completed in 3610016 Feb 20, 2018
@liZe liZe added this to the 43 milestone Feb 20, 2018
liZe added a commit that referenced this issue Mar 27, 2018
@liZe
Don't translate rtl columns twice
19342fb 
Fix #574.
@liZe liZe added the bug Existing features not working as expected label Dec 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Existing features not working as expected
Projects
None yet
Milestone
43
Development

No branches or pull requests
2 participants
@liZe @andreia-telenav