-
Notifications
You must be signed in to change notification settings - Fork 58
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
Доработана статья Flexberry Groupedit для 3 версии эмбера по замечаниям #221
base: master
Are you sure you want to change the base?
Доработана статья Flexberry Groupedit для 3 версии эмбера по замечаниям #221
Conversation
Quality Gate passedIssues Measures |
pages/products/flexberry-ember/3.x/components/ef3_groupedit.ru.md
Outdated
Show resolved
Hide resolved
pages/products/flexberry-ember/3.x/components/ef3_groupedit.ru.md
Outdated
Show resolved
Hide resolved
} | ||
``` | ||
|
||
Реализацию примера можно посмотреть на [тестовом стенде](https://flexberry.github.io/ember-flexberry/dummy/dummy-test-2/#/components-examples/flexberry-groupedit/dynamic-groupedit). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Пустая строка в конце статьи
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
вроде так и надо?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ну в смысле она там есть
Quality Gate passedIssues Measures |
WalkthroughДокумент компонента Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (4)
pages/products/flexberry-ember/3.x/components/ef3_groupedit.ru.md (4)
13-13
: Улучшите описание компонента.Предлагаю уточнить описание, добавив информацию о том, что компонент предназначен для работы с табличными данными:
-`flexberry-groupedit` – компонент на базе технологии Ember.js для работы с [детейлами](fo_detail-associations-properties.html) на форме редактирования. +`flexberry-groupedit` – табличный компонент на базе технологии Ember.js для работы с [детейлами](fo_detail-associations-properties.html) на форме редактирования.
59-60
: Исправьте форматирование таблицы свойств.Между заголовком таблицы и содержимым отсутствует пустая строка, что нарушает правила оформления Markdown.
Свойство | Краткое описание :--------|:---------------- + `defaultSettingsButton`| Флаг, определяющий, будет ли показана кнопка настроек по умолчанию на панели инструментов. Значение по умолчанию: true. Нажатие кнопки применит настройки отображение табличных данных, заданные по умолчанию `DEFAULT` и определяющие ширину столбцов, их отображение и порядок сортировки.
🧰 Tools
🪛 Markdownlint
59-59: Expected: 0 or 2; Actual: 1
Trailing spaces(MD009, no-trailing-spaces)
326-342
: Улучшите структуру раздела о пользовательских кнопках.Текст содержит избыточные повторы и может быть структурирован более эффективно.
-EditFormController - расширяет стандартный контроллер формы редактирования и позволяет создавать пользовательские кнопки для панели управления groupedit. -Для создания пользовательских кнопок на панели управления groupedit нужно: -В шаблоне для компонента groupedit задать значение свойства `customButtons` и события для создаваемых кнопок. +### Создание пользовательских кнопок + +Для добавления пользовательских кнопок на панель управления groupedit необходимо: + +1. В шаблоне компонента задать значение свойства `customButtons` +2. Определить обработчики событий для создаваемых кнопок
568-607
: Исправьте форматирование последних разделов.В конце документа нарушено форматирование:
- Отсутствуют пустые строки между заголовками и содержимым
- Неконсистентное использование пустых строк
## Ordered свойства в groupedit + При добавлении в представление, groupedit ordered-свойства в нем появятся некоторые особенности, которые следует учитывать: ... ## Динамическое обновление модели в groupedit + Существует возможность реализации динамического изменения groupedit в зависимости, например, от выбранного значения в object-list-view, расположенного на той же форме, что и groupedit.🧰 Tools
🪛 Markdownlint
575-575: Expected: 0 or 2; Actual: 1
Trailing spaces(MD009, no-trailing-spaces)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
- pages/products/flexberry-ember/3.x/components/ef3_groupedit.ru.md (10 hunks)
🧰 Additional context used
🪛 LanguageTool
pages/products/flexberry-ember/3.x/components/ef3_groupedit.ru.md
[uncategorized] ~61-~61: Избыточные определения. В правильно составленном предложении не должно быть больше одного придаточного определения.
Context: ...tFormRoute`| Роут формы редактирования, который будет открыт после клика по соответствующей строке (именно тот, по которому был клик). **Дополнительные свойства:*...(OPREDELENIA)
[uncategorized] ~65-~65: Перед знаками препинания не должно быть пробела: «описание:»
Context: ...тельные свойства:** Свойство | Краткое описание :--------|:---------------- `editOnSepara...(Punct_and_spaces)
🪛 Markdownlint
pages/products/flexberry-ember/3.x/components/ef3_groupedit.ru.md
59-59: Expected: 0 or 2; Actual: 1
Trailing spaces(MD009, no-trailing-spaces)
208-208: null
Lists should be surrounded by blank lines(MD032, blanks-around-lists)
412-412: Expected: 0 or 2; Actual: 1
Trailing spaces(MD009, no-trailing-spaces)
575-575: Expected: 0 or 2; Actual: 1
Trailing spaces(MD009, no-trailing-spaces)
@@ -449,8 +499,7 @@ console.log(cellComponent); // Выведет объект с настройка | |||
Настройка данного варианта аналогичная описываемому выше способу. В шаблоне контрола при этом нужно выставить один дополнительный флаг `saveBeforeRouteLeave`. | |||
|
|||
```hbs | |||
{% raw %} | |||
{{flexberry-groupedit | |||
{% raw %}{{flexberry-groupedit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codebase verification
Требуется обновление ссылок в документации
В файле найдены следующие проблемы с ссылками:
- Ссылка на
fo_detail-associations-properties.html
указывает на устаревшую документацию - Ссылка на
efd3_template.html
некорректна, соответствующий файл отсутствует в директории 3.x - Некоторые ссылки на Ember.js guides указывают на устаревшие версии (v3.1.0, v3.12.0)
- Ссылки на GitHub репозиторий должны указывать на актуальную ветку
Необходимо:
- Обновить ссылки на документацию Flexberry на актуальные версии для 3.x
- Обновить ссылки на Ember.js guides до актуальной версии
- Проверить и обновить ссылки на GitHub репозиторий
🔗 Analysis chain
Обновите ссылки на документацию.
Ссылки на статьи должны вести на актуальную версию документации для Ember 3.x.
Let me check the actual links in the specific file to verify if they need updating.
Let me check if these links are pointing to the correct 3.x documentation versions by checking if the referenced files exist in the 3.x directory.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Find all markdown links in the documentation
rg -l "\[.*\]\(.*\)" pages/products/flexberry-ember/
Length of output: 10083
Script:
#!/bin/bash
# Check all markdown links in the groupedit documentation file
rg "\[.*\]\(.*\)" pages/products/flexberry-ember/3.x/components/ef3_groupedit.ru.md
Length of output: 6208
Script:
#!/bin/bash
# Check existence of referenced files in 3.x directory
fd "editform.ru.md|template.ru.md|router.ru.md|model.ru.md|serializer.ru.md" pages/products/flexberry-ember/3.x/
Length of output: 397
Задача 42.
fixedHeader
- не внесены изменения, т.к. нужны уточнения по этой настройки. Будет вынесено в отдельную задачу.useSidePageMode
- свойство удалено из описания, т.к. у групедита нет модальных окон.Summary by CodeRabbit
Новые функции
flexberry-groupedit
, включая расширенные описания свойств и примеры использования.groupedit
, с примерами реализации.Документация