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

右上の残り字数カウンタが0なのにInvalid Param.エラーが出て投稿できないケースがある #14410

Closed
1 task
kyoka-h opened this issue Aug 15, 2024 · 2 comments · Fixed by #14466
Labels
1️⃣ Good First Issue This is a good problem for your first contribution. 🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR

Comments

@kyoka-h
Copy link

kyoka-h commented Aug 15, 2024

💡 Summary

先頭が改行またはブランクで始まる3001字の文章を入力した際、右上に表示される残り字数カウンタの表示が0と表示されてしまうため、ユーザは3000字ちょうどだと誤認して投稿しようとするが、実際は1字オーバーのため「Invalid Param.」のエラーポップアップが出て投稿できない。

🥰 Expected Behavior

先頭が改行またはブランクであっても、先頭の改行またはブランクを含めた字数でカウントする。
(先頭が改行またはブランクで始まる3001字の文章を入力した際は、右上の残り字数カウンタは「-1」と表示されてほしい)
または、投稿時に先頭の改行およびブランクがtrimされる挙動を正とするならば、postDataにはtrim後の文字列をセットして「Invalid Param.」エラーになることなく正常に投稿できるようにする。

🤬 Actual Behavior

MkPostForm.vueのtextLengthは入力内容をtrim()した後のlengthを返しているが、Post関数ではtrim前の入力内容をそのままpostDataに設定しているため、残り字数カウンタと実際の残り字数に差異が生じている。

📝 Steps to Reproduce

MkPostForm.vueのtextLengthを入力内容をtrim()せずにlengthを返すようにする。
またはPost関数で入力内容をtrim()してからpostDataに設定するようにする。

💻 Frontend Environment

* Model and OS of the device(s): Windows11
* Browser: FireFox 129.0.1
* Server URL: misskey.io、kasei.ski、dosei.fun
* Misskey: 2024.5.0(misskey.io, kasei.ski), 2024.7.0(dosei.fun)

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
@kyoka-h kyoka-h added the ⚠️bug? This might be a bug label Aug 15, 2024
@KisaragiEffective KisaragiEffective added the packages/frontend Client side specific issue/PR label Aug 26, 2024
@KisaragiEffective
Copy link
Collaborator

2024.8.0でも再現

@KisaragiEffective KisaragiEffective added 🐛Bug Unexpected behavior and removed ⚠️bug? This might be a bug labels Aug 26, 2024
@KisaragiEffective
Copy link
Collaborator

permalink:

return (text.value + imeText.value).trim().length;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1️⃣ Good First Issue This is a good problem for your first contribution. 🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR
Projects
None yet
2 participants