Separate the title from the description #452
Conversation
frontend/src/pages/about/Faq.jsx
Outdated
| <br /> | ||
| <br /> | ||
| <strong>{t('faq.a23')}</strong> | ||
| {t('faq.a24')} | ||
| {convertToHtml(t('faq.a23'), t('faq.a24'))} |
There was a problem hiding this comment.
@IVANn84 у вас в итоге получилось, что строка разделилась на две части. Я имел ввиду, что у вас имеено сам тег strong будет внутри строки-локализации. Т.е. a23 будет в одном ключе с a24. Ладно, раз решили делать так, то давайте ревьювить +)
frontend/src/pages/about/Faq.jsx
Outdated
| @@ -4,6 +4,14 @@ import './Faq.module.css'; | |||
|
|
|||
| export function Faq() { | |||
| const { t } = useTranslation(); | |||
| function convertToHtml(header, text) { | |||
There was a problem hiding this comment.
Имя функции не говорит о том, что мы делаем. Какая именно конвертация? Непонятно. Внутри может что угодно конвертироваться в html, а по факту здесь возвращается две строки - одна ссылкой (кстати почему? Никакой ссылки жеж здесь нет), другая выделенный текст.
frontend/src/locales/ru.json
Outdated
| @@ -62,26 +62,26 @@ | |||
| "a0": "Интегрированная среда разработки (IDE) — это программное приложение, которое позволяет разработчикам писать, отлаживать и тестировать код, используя при этом один из нескольких языков программирования. Интегрированные среды разработки популярны среди разработчиков, потому что они ускоряют и упрощают работу по написанию кода. Так же как писатели используют текстовые редакторы, а бухгалтеры – электронные таблицы, разработчики программного обеспечения применяют IDE, чтобы повысить продуктивность.", | |||
| "a11": "Большинство IDE включают в себя функции, выходящие за рамки обычного редактора кода. В одном приложении объединено множество различных инструментов, благодаря которым разработчики могут быстро приступить к программированию новых приложений вместо того, чтобы вручную интегрировать и настраивать различное ПО. Вот несколько важных функций IDE.", | |||
| "a110": "Автоматизация локальной сборки", | |||
| "a111": "IDE повышают производительность труда программистов, выполняя повторяющиеся задачи разработки, которые обычно являются частью каждого изменения кода", | |||
| "a111": " IDE повышают производительность труда программистов, выполняя повторяющиеся задачи разработки, которые обычно являются частью каждого изменения кода", | |||
There was a problem hiding this comment.
вот чего я боялся - появился пробел, который надо учитывать. Здесь проще сделать что-то типа
"a110": "<strong>Автоматизация локальной сборки</strong> IDE повышают производительность труда программистов, выполняя повторяющиеся задачи разработки, которые обычно являются частью каждого изменения кода",
```
тогда вы это просто как html рендерите. Мб во фреймворке можно посмотреть список разрешенных тегов. Например таким образом выводятся форматированные тексты в Хекслете, когда нужны например формулы
There was a problem hiding this comment.
Здравствуйте, исправил замечания , согласно статье Hexlet перезаписал предыдущий PR , но теперь никак не могу понять отправились ли Вам изменения или нет..( . Подскажите пожалуста пришел ли новый PR. спасибо
|
Здравствуйте, исправил замечания , согласно статье Hexlet перезаписал предыдущий PR , но теперь никак не могу понять отправились ли Вам изменения или нет..( . Подскажите пожалуста пришел ли новый PR. спасибо |
|
Да, вроде отправились =) |
According to Issues 440, the headings are separated from the description in the help section. P/S: I automatically started the linter after the changes I made and it automatically corrected some more files.. Is it really not very good?.. I looked at the diff, there seem to be only cosmetic changes.