nildarar
(Nildarar)
September 10, 2020, 5:39am
1
The list opened in Select-Kit is not displayed correctly in RTL.
Current layout:
+--------+
| target |
+-------------+
| |
| popper |
| |
+-------------+
Expected layout:
+--------+
| target |
+-------------+
| |
| popper |
| |
+-------------+
I checked the code and saw that the settings are only for LTR.
https://github.com/discourse/discourse/blob/52672b9eabccb1184d85dc7f08062d5a7c18cb73/app/assets/javascripts/select-kit/addon/components/select-kit.js#L272
And used here:
https://github.com/discourse/discourse/blob/52672b9eabccb1184d85dc7f08062d5a7c18cb73/app/assets/javascripts/select-kit/addon/components/select-kit.js#L779
I tried to solve the problem by adding some CSS codes, but unfortunately, this causes more problems. I think if we use placement: "bottom-end"
when the page is RTL, the problem will be solved.
Also, this has caused the opened poppers to be displayed off-screen on some mobile phones. If you need, I can send you some screenshots of the problem in a pm.
1 Like
nildarar
(Nildarar)
October 22, 2020, 8:30am
3
Can we apply this change to our community or must the change be applied by the Discourse team?
j.jaffeux
(Joffrey Jaffeux)
October 22, 2020, 8:31am
4
Yes you need to wait for a change, sorry, will try to have a look soon, should be trivial to fix.
1 Like
nildarar
(Nildarar)
October 22, 2020, 8:37am
5
Thank you for your response
Do you think it is possible to change such a thing through JavaScript? As we inject this change through a theme component to temporarily solve the problem.
j.jaffeux
(Joffrey Jaffeux)
October 22, 2020, 8:42am
6
Everything is possible but I wouldn’t recommend it, I will have a look shortly.
1 Like
nildarar
(Nildarar)
October 22, 2020, 8:46am
7
On desktops larger than 1711px, this problem is easily seen in the notification settings at the end of the topic page.
1 Like
j.jaffeux
(Joffrey Jaffeux)
October 22, 2020, 8:32pm
8
I pushed a fix on master for this, let me know if it improves the situation for you. If not please make sure you take a screenshot.
5 Likes
nildarar
(Nildarar)
October 31, 2020, 6:38am
9
Due to the deletion of the gif profile pictures, we had to update with a slight delay. As it turns out, this problem is completely solved.
Thanks @j.jaffeux
2 Likes