? ? Pending

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
17 Jul 2019

Summary of Changes

Postinstall component uses the admin mod_feed to display the Release News from the Joomla Project.
But it does not take into account a RTL language.
Alignment and direction of date are set by default to LTR
This PR corrects that by:

  1. Settting the module rssrtl to RTL if lang is RTL for the envelopping <div> in postinstall default.php
  2. Adding a specific style to the <ul to force the feed texts to be aligned left as these are in English in mod_feed.

Testing Instructions

Install Persian language and switch admin to Persian.
Load index.php?option=com_postinstall&eid=700

Before patch

The date in the left column is not displayed as RTL and is aligned left
Screen Shot 2019-07-17 at 11 30 41

After patch

The date is now RTL and alignment is right
Screen Shot 2019-07-17 at 11 29 38

Note

For 4.0 the modified code for mod_feed will be slightly different as we have no more
<?php echo $params->get('moduleclass_sfx'); ?>
and the class for the feed texts is now newsfeed list-group

avatar infograf768 infograf768 - open - 17 Jul 2019
avatar infograf768 infograf768 - change - 17 Jul 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Jul 2019
Category Administration com_postinstall Modules
0e91a57 18 Jul 2019 avatar infograf768 cs
avatar infograf768 infograf768 - change - 18 Jul 2019
Labels Added: ?
avatar franz-wohlkoenig franz-wohlkoenig - change - 20 Jul 2019
Title
[staging] Correcting RTL in Postinstall
Correcting RTL in Postinstall
avatar franz-wohlkoenig franz-wohlkoenig - edited - 20 Jul 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 20 Jul 2019
The description was changed
avatar franz-wohlkoenig franz-wohlkoenig - edited - 20 Jul 2019
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 20 Jul 2019

I have tested this item ? unsuccessfully on 0e91a57

without PR: Bullet is on Start of Title

ohne-pr

with PR: Bullet is on right End of Title (as Date).

mit-pr


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25604.
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 20 Jul 2019 - Tested unsuccessfully
avatar infograf768
infograf768 - comment - 20 Jul 2019

I see other errors with the feed text which should be ltr. Looking at this now.
Also, the padding was in error before patch.

avatar infograf768
infograf768 - comment - 20 Jul 2019

@Quy @franz-wohlkoenig
Should be OK now.

Screen Shot 2019-07-20 at 11 39 22

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 21 Jul 2019

I have tested this item ? unsuccessfully on ce28993

All Paragraphs are <strong>.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25604.

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 21 Jul 2019 - Tested unsuccessfully
avatar infograf768
infograf768 - comment - 23 Jul 2019

@franz-wohlkoenig

This is not due to this patch but an error in the Persian ini file: a <strong> not closed properly:
(</ strong>)

COM_PRIVACY_POSTINSTALL_BODY="<p>با معرفی GDPR برای شهروندان اتحادیه اروپا و مقررات مشابه در سایر نقاط جهان، ممکن است لازم باشد قبل از ذخیره هر <strong>اطلاعات شخصی</ strong> از کاربر رضایت بخواهید.</p><p>جوملا 3.9 قابلیت های جدیدی را برای کمک به شما در ایجاد سیاست های حفظ حریم خصوصی سایت و جمع آوری رضایت کاربر معرفی می کند. علاوه بر این، یک گردش کار برای کمک به مدیریت درخواستهای اطلاعات کاربر مانند درخواست حذف اطلاعات شخصی خود از سایت شما در دسترس است.</p><p>برای اطلاعات بیشتر در مورد این ویژگی جدید لینک <a href='https://docs.joomla.org/J3.x:Privacy' target='_new'>مستندات حریم خصوصی</a> را مطالعه کنید.</p>"

once corrected to

COM_PRIVACY_POSTINSTALL_BODY="<p>با معرفی GDPR برای شهروندان اتحادیه اروپا و مقررات مشابه در سایر نقاط جهان، ممکن است لازم باشد قبل از ذخیره هر <strong>اطلاعات شخصی</strong> از کاربر رضایت بخواهید.</p><p>جوملا 3.9 قابلیت های جدیدی را برای کمک به شما در ایجاد سیاست های حفظ حریم خصوصی سایت و جمع آوری رضایت کاربر معرفی می کند. علاوه بر این، یک گردش کار برای کمک به مدیریت درخواستهای اطلاعات کاربر مانند درخواست حذف اطلاعات شخصی خود از سایت شما در دسترس است.</p><p>برای اطلاعات بیشتر در مورد این ویژگی جدید لینک <a href='https://docs.joomla.org/J3.x:Privacy' target='_new'>مستندات حریم خصوصی</a> را مطالعه کنید.</p>"

all is fine:

Screen Shot 2019-07-23 at 11 55 00

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 23 Jul 2019

I have tested this item successfully on 5346788


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25604.

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 23 Jul 2019 - Tested successfully
avatar infograf768
infograf768 - comment - 25 Jul 2019

@Quy
Please confirm so that we get this in in next release.

avatar infograf768
infograf768 - comment - 25 Jul 2019

Modified as requested.
@franz-wohlkoenig No need to test again.

avatar Quy
Quy - comment - 25 Jul 2019

I have tested this item successfully on 0317860


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25604.

avatar Quy Quy - test_item - 25 Jul 2019 - Tested successfully
avatar Quy Quy - change - 25 Jul 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 25 Jul 2019

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25604.

avatar infograf768 infograf768 - change - 26 Jul 2019
Labels Added: ?
avatar infograf768
infograf768 - comment - 28 Jul 2019

@HLeithner
Please merge.

avatar HLeithner HLeithner - change - 31 Jul 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-07-31 09:58:25
Closed_By HLeithner
avatar HLeithner HLeithner - close - 31 Jul 2019
avatar HLeithner HLeithner - merge - 31 Jul 2019
avatar HLeithner
HLeithner - comment - 31 Jul 2019

thx

Add a Comment

Login with GitHub to post a comment