Jefferson49 / Joomla_plugin_sexypolling_reloaded

Migration of the joomla plugin "Sexy Polling" to joomla 4.x
GNU General Public License v3.0
2 stars 3 forks source link

Right to Left Middle Eastern #51

Closed brilliu closed 11 months ago

brilliu commented 11 months ago

Hi, Thank you for the amazing Sexy Polling Joomla Extension, it's just fab. I am having issues with Right to Left text appearing as left to right. Thank you

Jefferson49 commented 11 months ago

Welcome to the plugin and thank you for reporting the issue!

Until now, I have no experience with right to left text. On first sight, I see two options:

Poll name text align

brilliu commented 11 months ago

Hi, thank you for the swift reply. Answers Text Align = right does not do anything. I have some knowledge of CSS, will try to do that. The problem it is creating is that I can't control the order of questions, 1, 2 ,3 4, 5 etc... you can see it in the attached screen 2023-10-09_13-52-31 shot

Jefferson49 commented 11 months ago

I also do not understand why the first 2 answers are not ordered properly. Starting from the third answer, everything from "01/" up to "19/" seems to be o.k.

I checked in the code and the ordering seems to be performed by a database query: ORDER BY sexy_answers.name

You could use phpMyAdmin to check the answers in the database. Open the "[prefix]_sexy_answers" table with phpMyAdmin and check the "name" column. Try to sort the data according to the "name" column and check if you can find some explaination for the ordering.

Jefferson49 commented 11 months ago

Answers Text Align = right does not do anything.

I was able to identify the root cause and fixed it in the code. The bugfix will be available with the next release.

However, I found out that the bug was only in the template configuration animation. If you save the changes in the template, the "Answers Text Align" setting will be saved properly and the new text alignment is shown in the site module. Therefore, you can use the feature also with the existing plugin version.

Jefferson49 commented 11 months ago

Fixed with 8eeb9df