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

Дятлов Е.А. Пашин Н.А. Курсовой проект Компонент создания класса сообщений и шаблона фраз #343

Open
wants to merge 18 commits into
base: main
Choose a base branch
from

Conversation

EugeneDyatl0v
Copy link

@EugeneDyatl0v EugeneDyatl0v commented Mar 26, 2024

ZCOJ245IPMZ3H7WMIOIAU7THLFE24ZUH

Copy link
Collaborator

@PetrochukKsenija PetrochukKsenija left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я бы предложила переименовать немного системный идентификатор, который предлагается для класса фраз. Сделать его по одному из вариантов:

  1. concept_phrase_about_ (по такому принципу мы делали в шаблонах, там было concept_phrase_about_your_theme)
  2. concept_phrase_with_template_about_ (это если нужно обязательно отметить, что тут шаблон)

Хотя мне кажется, что под шаблоном должна иметься поисковая конструкция с переменной во фразе. Тут этого нет, поэтому и упоминать про шаблон не нужно, поэтому я предлагаю всё же первый вариант
@MikhailSadovsky

Copy link
Collaborator

@PetrochukKsenija PetrochukKsenija left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

И ещё момент, когда добавляем шаблон фразы. Не сразу понятно, что нужно нажать на enter. Было бы хорошо это тоже подправить
И если уже прям сильно заморочится, то этот блок возможно лучше назвать "Ответные фразы", а в строках, где мы пишем фразу, сделать, чтобы было написано "Фраза" или "Напишите фразу", так как тут нет шаблонов поиска при формировании ответа. Можно даже в строке, куда фразу вписываем, сразу написать "Напишите фразу и нажмите enter" или что-то по типу этого
Тут тоже надо мнение @MikhailSadovsky

Copy link
Collaborator

@PetrochukKsenija PetrochukKsenija left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Есть ещё предложение переименовать идентификаторы на русском языке:
"Сообщение о" -> "Класс сообщений о"
"Шаблон ответа о" -> "Класс ответных фраз о"

И название самого окна сделать "Создание класса сообщение и класса ответных фраз"
Сообщение пользователя тоже было бы хорошо поменять в соответствии с названием окна на "Создай класс сообщений и класс ответных фраз"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants