hosna
5월 31, 2023, 12:36오후
1
사진에서 보시는 것처럼 창밖으로 나가 있어서 아무것도 볼 수 없습니다.
아래 머지 리퀘스트와 관련이 있을 수 있습니다:
main ← fix/layout-problems-rtl
merged 10:27PM - 30 May 23 UTC
We currently have some CSS rules in `common/base/rtl.scss` that were added to wo… rkaround shortcomings of the R2 gem that we used to use to generate a version of our CSS that's suitable for RTL layouts. Those workarounds are mostly duplicate rules with the only difference being that they're flipped to suit RTL layouts.
However, we've recently replaced R2 with `rtlcss` which doesn't have those shortcomings of R2 (see https://github.com/discourse/discourse/commit/f94951147ea9d1c48ea68f0dabd568e70cb94674) which means those workarounds/duplicate rules need to be removed because they're getting flipped by `rltcss`, essentially reverting them to their original LTR version and causing issues with RTL layouts.
This PR removes those workarounds that are no longer needed, and cleans up the the file that contains our RTL-specific CSS.
Meta topic: https://meta.discourse.org/t/avatar-in-rtl-website-in-wrong-place/264676?u=osama.
@Osama
5개의 좋아요
olivia
5월 31, 2023, 12:59오후
2
Discourse에서 오른쪽에서 왼쪽으로(RTL) 레이아웃에서 멘션 제안 목록 박스의 배치 관련 문제를 재현하려면 다음 단계를 따를 수 있습니다:
우선순위/심각도: 중간 - 이 버그는 사용자 인터페이스와 사용자 경험에 영향을 미치며, 특히 RTL 언어 사용자에게 해당됩니다. 기능이 깨지지는 않지만 사용에 불편을 초래할 수 있습니다.
플랫폼: 데스크톱에서 테스트, OS: Windows 10, 브라우저: Google Chrome.
설명: Discourse에서 RTL 언어가 활성화된 상태에서 사용자가 다른 사용자를 멘션하려고 할 때(이름의 앞부분을 입력하기 전에 '@'를 입력), 제안 목록 박스가 표시 가능한 창 영역 밖에 나타납니다. 이 문제로 인해 사용자는 사용자명 제안 목록을 볼 수 없습니다.
재현 단계:
Discourse를 RTL 언어 설정으로 구성합니다. (관리자 설정에서 이를 설정할 수 있습니다.)
테스트 사용자로 로그인합니다.
댓글을 남길 수 있는 아무 주제(Topic)로 이동합니다.
새 게시물이나 답변을 입력하기 시작하여 다른 사용자를 멘션하려고 합니다(이름의 앞부분을 입력하기 전에 '@'를 입력).
나타나는 멘션 제안 목록 박스의 위치를 관찰합니다.
6개의 좋아요
Canapin
5월 31, 2023, 2:11오후
3
오늘 5일 된 인스턴스를 업그레이드한 후 재현할 수 있으니, 이는 최근 커밋과 관련이 있을 가능성이 높습니다
6개의 좋아요
Osama
6월 2, 2023, 1:07오전
4
이 버그를 보고해 주셔서 감사합니다 @hosna ! 실제로 최근 발생한 회귀 문제이며, 이 PR으로 수정됩니다:
main ← fix/mention-autocomplete-in-rtl
merged 04:55PM - 02 Jun 23 UTC
Meta topic: https://meta.discourse.org/t/mention-suggestion-list-box-in-the-rtl-… website-in-wrong-place/266763?u=osama.
Our autocomplete box doesn't currently take into account the user's locale and places itself off-screen when using an RTL locale. This PR changes the placement logic for the autocomplete box when an RTL locale is used to make sure that:
1. the autocomplete box's right side is near and to the left of the caret
2. the autocomplete box doesn't go outside the composer's left side
Before:
<img src="https://github.com/discourse/discourse/assets/17474474/71b375b3-6f05-49b5-a4ee-910a96010b53" width=500>
After:
<img src="https://github.com/discourse/discourse/assets/17474474/3209d8a0-16b4-45a1-953d-355c73ce3224" width=500>
<img src="https://github.com/discourse/discourse/assets/17474474/92215653-bdf0-49cd-b38d-13301c2a7adb" width=500>
<img src="https://github.com/discourse/discourse/assets/17474474/f4d157d8-6355-457d-9328-35620c450f6c" width=500>
3개의 좋아요
