-
Notifications
You must be signed in to change notification settings - Fork 9
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
チェックインにつけられるタグ数に制限がない #709
Comments
これは考慮漏れです。タグ数は制限する方向で行きたいです。 Note
|
タグ1つあたりの文字数に触れていなかったですが、こっちは現状のまま表示で切り落としてしまっても良いかなと思っています。 ちなみに本番のtagsテーブルの最長は70文字ですが、これはユーザー入力では無さそうです。
|
Closed
flex, gapなら思考放棄できる .バッジのコンテナ {
display: flex;
flex-wrap: wrap;
gap: .5ch;
}
.badge {
text-align: left;
max-width: 100%;
white-space: normal;
word-break: break-all;
} .badgeは1行前提でline-heightが1なので、line-height設定ついでに余白を足してあげてもいいかもしれない(よく触るコンポーネントにしては小さい気がしてたので) .バッジのコンテナ {
...
gap: .6ch;
}
.badge {
...
line-height: 1.5;
padding: .2em .6em;
} |
This was referenced Aug 4, 2021
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
webhookを通じて200文字のタグを5000個つけられることを確認しました。このようなチェックインがお惣菜コーナーなどに出てくるとレイアウトが壊滅してしまうので、どこかで制限したほうがいい気がします。
対策
資料: 別サービスでは?
ユーザーが自由な文字列でタグをつけられるサービスでの最大タグ数と1タグの最大文字数
The text was updated successfully, but these errors were encountered: