#1741 comment text area character limit fixed (500 chars) #1762

RasmuS2024 · 2025-09-29T19:33:35Z

Pull request details

RU

В CommenRequest добавил константу MAX_CONTENT_LENGTH = 500.
Поля ввода комментария, редактирования комментария и ответа на комментарий ограничено 500 символами, значение берется из константы.
Изменил label полей ввода комментария, редактирования комментария и ответа на комментарий, где выводится максимальное количество символов ( в локали передается :max, который в шаблонах берется из константы).

EN

Added the constant MAX_CONTENT_LENGTH = 500 to CommenRequest.
The comment input, comment editing, and comment reply fields are limited to 500 characters; the value is taken from the constant.
Changed the label for the comment input, comment editing, and comment reply fields to display the maximum number of characters (in the locale, :max is passed, which is taken from the constant in templates).

Issues fixed

#1741

fey · 2025-09-30T08:43:11Z

resources/views/components/comments.blade.php

  </ul>
  @auth
-    @include('components.comment._form')
+    @include('components.comment._form', ['maxLength' => \App\Http\Requests\CommentRequest::MAX_CONTENT_LENGTH])


Давай эту константу использовать прямо внутри формы, т.е. не передавать снаружи, а обращаться к ней в компоненте.
Тк форма комментария и логика они неразрывно связаны. Ну и так код будет почище. То, что относится к комментам, будет внутри.

Давай эту константу использовать прямо внутри формы, т.е. не передавать снаружи, а обращаться к ней в компоненте. Тк форма комментария и логика они неразрывно связаны. Ну и так код будет почище. То, что относится к комментам, будет внутри.

Поправил

fey · 2025-10-01T06:31:13Z

resources/views/components/comment/_comment.blade.php

   */
  use App\Helpers\MarkdownHelper;
+  if (!defined('MAX_CONTENT_LENGTH')) {
+      define('MAX_CONTENT_LENGTH', 500);


не, у тебя до этого верно было что константабыла в реквесте определена. Ведь получается, что у нас константа 500, которая обозачает длину коммента, расползается по двум местам.

тут как будто бы нет лучшего решения, но определять константы внутри вью - зло.

До этого верный путь был - константа определена в реквесте, и использована во вью. Просто не нужно ее прокидывать снаружи (из другой вью) в этот компонент, а прямо тут обращаться к контснанте, как щас делаешь.

Хорошо, делаю.

@fey Не понял как обращаться к константе реквеста из вью. Можно ли сделать константу в config\comment ? а она будет глобально доступна. Для Laravel я так понял это лучший вариант.

@fey Не понял как обращаться к константе реквеста из вью. Можно ли сделать константу в config\comment ? а она будет глобально доступна. Для Laravel я так понял это лучший вариант.

А у тебя ведь было. \App\Http\Requests\CommentRequest::MAX_CONTENT_LENGTH - в принципе нам так допустимо. Почему? Потому что это не настройка уровня приложения прям ,чтобы в конфиг уносить. Это локальная штука конкретно к комментам. Это может быть внутри реквеста, внутри ДТО, или модели, ведь мы на уровне предметной области задаем бизнес-правило "комментарий не может быть больше 500 символов".
Константа не будет магической, и ее можно использовать во вью, реквесте, в переводах.

fey · 2025-10-01T06:32:33Z

resources/views/components/comment/_comment.blade.php

    @if ($comment->isReply())
      <div class="border my-2">
-        <div class="small text-muted ml-2">{{ $comment->parent->content }}</div>
+        <div class="small text-muted ml-2" style="overflow-wrap: break-word;">{{ $comment->parent->content }}</div>


Вот это как будто бы лишнее изменение. Для чего оно?
Во первых это не решает нашу текущую задачу (показать лимит сообщения), во вторых это инлайн-стиль, который мы избегаем. Скорее всего для бутстрапа есть нужный класс-утилита. Используй его. Если нет - определени кастом.
Похожа проблема где-то уже была, поэтому можешь поискать по вьюхам.

Я понял, исправлю. Это решает проблему, когда отвечаешь на комментарий, ответ показывается нормально с переносами, а сам исходный коммент показывается в другом стиле и длинные слова не переносятся. Вот, нашел: <div class="small text-muted ml-2 text-break">.

хм, верно понял, что break-word решает это, а text-break - нет?

RasmuS2024 · 2025-10-01T21:58:31Z

Что-то пошло не так...

fey · 2025-10-02T04:33:06Z

resources/views/components/comment/_comment.blade.php

  use App\Helpers\MarkdownHelper;
+  if (!defined('MAX_CONTENT_LENGTH')) {
+      define('MAX_CONTENT_LENGTH', 500);
+  }


@RasmuS2024 константа вернулась/осталась)

В config/comment.php у тебя определенеа длина коммента.

А почему решил через конфиг это делать? Мне кажется через константу в CreateCommentRequest кмк вполне норм было. Конфиг возможно стоит оставить для более системных штук. А так, если константа будет в реквесте, то будет +- локализовано это (т.е. не ползет по остальному риложению).

Еще вариант - как раз использовать DTO, когда ты из реквеста (тела запроса) создаешь ридонли класс типа CommentData, и переменная тоже там может быть.

Можно еще прям в модель это впиндюрить. Если у нас "бизнесовое" требовани ограничить длину коммента, то это справедливо туда положить.

fey · 2025-10-02T04:34:14Z

resources/views/components/comment/_comment.blade.php

    @if ($comment->isReply())
      <div class="border my-2">
-        <div class="small text-muted ml-2">{{ $comment->parent->content }}</div>
+        <div class="small text-muted ml-2 text-break">{{ $comment->parent->content }}</div>


Вот тут в итоге это изменение - для чего?)

Т.е. мы тут фиксим и верстку и вывод сообщения о длине коммента? Я думаю, лучше разделить и делать это разными ПРами, тк по смыслам это именно разные вещи. И если что-то пойдет не так, то будет легче откатить одно из изменений.

RasmuS2024 · 2025-10-04T14:45:44Z

@ddm14159 подправил, но правильно ли? Лишние правки не относящиеся к данному ищщусу убрал.

…nd fixed resorces/lang comment files

RasmuS2024 changed the title ~~1741/feature/comment character limit~~ #1741 feature: comment character limit fixed Sep 29, 2025

RasmuS2024 changed the title ~~#1741 feature: comment character limit fixed~~ #1741 comment text area character limit fixed Sep 29, 2025

RasmuS2024 changed the title ~~#1741 comment text area character limit fixed~~ #1741 comment text area character limit fixed (500 chars) Sep 29, 2025

fey reviewed Sep 30, 2025

View reviewed changes

RasmuS2024 force-pushed the 1741/feature/comment-character-limit branch from 0df6ddb to 537e493 Compare September 30, 2025 19:29

fey reviewed Oct 1, 2025

View reviewed changes

RasmuS2024 force-pushed the 1741/feature/comment-character-limit branch from 7dbbe8b to 55413af Compare October 1, 2025 22:23

fey reviewed Oct 2, 2025

View reviewed changes

RasmuS2024 force-pushed the 1741/feature/comment-character-limit branch from 55413af to f576f79 Compare October 4, 2025 13:36

fix: update character limit handling per review without other fixes

f41f9ce

RasmuS2024 force-pushed the 1741/feature/comment-character-limit branch from f576f79 to f41f9ce Compare October 4, 2025 14:36

fix: update character limit handling per review without other fixes a…

0e1c2a0

…nd fixed resorces/lang comment files

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

#1741 comment text area character limit fixed (500 chars) #1762

#1741 comment text area character limit fixed (500 chars) #1762

Uh oh!

RasmuS2024 commented Sep 29, 2025 •

edited

Loading

Uh oh!

fey Sep 30, 2025

Uh oh!

RasmuS2024 Sep 30, 2025

Uh oh!

fey Oct 1, 2025

Uh oh!

RasmuS2024 Oct 1, 2025

Uh oh!

RasmuS2024 Oct 1, 2025 •

edited

Loading

Uh oh!

fey Oct 3, 2025

Uh oh!

fey Oct 1, 2025

Uh oh!

RasmuS2024 Oct 1, 2025 •

edited

Loading

Uh oh!

fey Oct 3, 2025

Uh oh!

RasmuS2024 commented Oct 1, 2025

Uh oh!

fey Oct 2, 2025

Uh oh!

fey Oct 2, 2025

Uh oh!

RasmuS2024 commented Oct 4, 2025 •

edited

Loading

Uh oh!

Uh oh!

#1741 comment text area character limit fixed (500 chars) #1762

Are you sure you want to change the base?

#1741 comment text area character limit fixed (500 chars) #1762

Uh oh!

Conversation

RasmuS2024 commented Sep 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pull request details

Issues fixed

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

RasmuS2024 Oct 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

RasmuS2024 Oct 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

RasmuS2024 commented Oct 1, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

RasmuS2024 commented Oct 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

RasmuS2024 commented Sep 29, 2025 •

edited

Loading

RasmuS2024 Oct 1, 2025 •

edited

Loading

RasmuS2024 Oct 1, 2025 •

edited

Loading

RasmuS2024 commented Oct 4, 2025 •

edited

Loading