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.

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

名前にUnicode制御文字が入っているとUIが破壊されることがある #12714

Open
kakkokari-gtyih opened this issue Dec 19, 2023 · 11 comments
Labels
🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR

Comments

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Dec 19, 2023

💡 Summary

Unicode制御文字(RTLマークなど)が入っていると周りの要素にも影響が出る

特定の条件下で文字列が左右逆方向に流れるようになっている

🥰 Expected Behavior

image

🤬 Actual Behavior

https://misskey.io/notes/9nfnrifkaug400r2

image

📝 Steps to Reproduce

不明(特定のユーザーのリノートメッセージで発生)

💻 Frontend Environment

* Model and OS of the device(s): Windows 11 22H2
* Browser: Chrome v119.0.6045.200 (Official Build) (64 ビット)
* Server URL: misskey.io
* Misskey: 2023.11.1-io.5a

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:
@kakkokari-gtyih kakkokari-gtyih added the ⚠️bug? This might be a bug label Dec 19, 2023
@kakkokari-gtyih kakkokari-gtyih changed the title 特定の条件下でメッセージがRTLになる 特定の条件下で文言がRTLになる Dec 19, 2023
@syuilo
Copy link
Member

syuilo commented Dec 19, 2023

名前にUnicodeのright-to-left mark埋め込んでるとかではなく?

@kakkokari-gtyih
Copy link
Contributor Author

その可能性はある

@kakkokari-gtyih
Copy link
Contributor Author

ただ、その場合であっても「トーノリが」にはならないようにしたい

@syuilo
Copy link
Member

syuilo commented Dec 19, 2023

名前に制御文字は使えなくしたい

@GrapeApple0
Copy link
Contributor

名前にUnicodeのright-to-left mark埋め込んでるとかではなく?

users/showでみた感じ\u202e\u202cが入っていました

@GrapeApple0
Copy link
Contributor

\u202eを先頭に入れて名前を逆順に入れると再現できました

@kakkokari-gtyih kakkokari-gtyih changed the title 特定の条件下で文言がRTLになる 名前にUnicode制御文字が入っているとUIが破壊される Dec 19, 2023
@kakkokari-gtyih kakkokari-gtyih changed the title 名前にUnicode制御文字が入っているとUIが破壊される 名前にUnicode制御文字が入っているとUIが破壊されることがある Dec 19, 2023
@WinLinux1028
Copy link

本人登場
名前にしてる文字列はこれね
a.txt

@SINtyanneru
Copy link

(おもしろいから残していいと思ったりした)

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Dec 19, 2023

(おもしろいから残していいと思ったりした)

フィールドに制御文字を許可したとしても「トーノリが」にはならないようにしたい(UIが壊れると混乱は避けられないので)

@SINtyanneru
Copy link

なるほろ

@acid-chicken
Copy link
Member

名前に制御文字は使えなくしたい

どちらかというとちゃんと unicode-bidi の指定を整備したほうがいい

@syuilo syuilo added 🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR and removed ⚠️bug? This might be a bug labels Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR
Projects
None yet
Development

No branches or pull requests

6 participants