-
Notifications
You must be signed in to change notification settings - Fork 86
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
текст в кнопке #555
Comments
не входит в зону ответственности библиотеки, сделайте собственный хелпер, который будет этим промышлять |
в telegraf.js реализовано так и не нужно постоянно об этом помнить и писать хэлперы. По моему так удобнее. |
библиотека же реализована для удобства разработки по мне моё предложение этим и является |
телеграф говно, вопросы? |
а если мне нужна ошибка, а не чтоб текст обрезался? |
А что тогда не говно? |
А можете пример кейса привести? |
Это плохая идея, как показывает практика, взгляните на то как он MySQL обрезал строки вместо ошибки и сколько данных из-за этого билось. Если в кнопке может быть больше 40 символов, это явно не то что нужно обрезать, вместо этого лучше быть более кратким или вынести описание в текст сообщения. |
В моём кейсе название кнопок это названия сообществ. Изначально вынес это в дискуссию, но там никто не реагирует. Поэтому создал ишью. |
Discussed in #554
Originally posted by Jhon-Mosk October 4, 2023
При длине лэйбла в кнопке более 40, сделать сокращение его до 40 символов с заменой последних трёх ... Вместо проброса ошибки
vk-io/packages/vk-io/src/structures/keyboard/builder.ts
Line 72 in 1604d19
The text was updated successfully, but these errors were encountered: