Skip to content

Add DataCollection guides#56

Open
mafanya23 wants to merge 18 commits intomasterfrom
add-datacollection-guide-4800
Open

Add DataCollection guides#56
mafanya23 wants to merge 18 commits intomasterfrom
add-datacollection-guide-4800

Conversation

@mafanya23
Copy link
Contributor

No description provided.

@mafanya23 mafanya23 requested a review from Sorokin-Oleg March 11, 2025 08:21
Copy link
Contributor

Choose a reason for hiding this comment

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

@mafanya23 кажется тут не то описание. silent - если использовать метод с включенным данным флагом, метод будет работать в "тихом" режиме, то есть - без вызова событий. Метод сработает, события будут проигнорированы.

Copy link
Contributor

Choose a reason for hiding this comment

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

@mafanya23 откуда тут перерисовка?)) Это частный случай, а может и сработает. Можно в примечаниях написать, что:

после использования данного метода с включенным флагом silent вам потребуется принудительная перерисовка компонента при помощи метода paint()

Флаг silent вызывает метод без вызова событий.

Copy link
Contributor

Choose a reason for hiding this comment

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

@mafanya23 понимаю, что правка локальная. Если есть возможность, поправ пожалуйста отступы в примерах:
image

Copy link
Contributor

Choose a reason for hiding this comment

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

@mafanya23 почти во всех примерах есть такая проблема

Copy link
Contributor

Choose a reason for hiding this comment

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

@mafanya23 метод возвращает длину коллекции. Название переменной можно было назвать dataLength

Copy link
Contributor

Choose a reason for hiding this comment

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

@mafanya23 по описанию подавлю. В TreeCollection этот метод коварен, и вместо длинны всех данных отобразит длину уровня. Вроде есть об этом запись в TreeCollection. Но я подумал, что в гайдах стоит упомянуть, если не будешь делать отдельный гайд по TreeCollection

Copy link
Contributor

Choose a reason for hiding this comment

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

@mafanya23 index не опционален.


**Related sample**: [Data. Parse](https://snippet.dhtmlx.com/0zrxtmvi)

## Checking whether data is loaded
Copy link
Contributor

Choose a reason for hiding this comment

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

@mafanya23 Хорошо, мы проверяем загруженные данным. А как мы их загрузили? Нет описания ленивой загрузки.

grid.data.save("http://userurl/");

//or
grid.data.save(new DataProxy({ url:"http://userurl/" }));
Copy link
Contributor

Choose a reason for hiding this comment

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

@mafanya23 это ведь тоже к загрузке относится, в гайде инфы нет DataProxy

Copy link
Contributor

Choose a reason for hiding this comment

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

@mafanya23 будет просто ошибка. Grid без свойства columns не инициализируется

@@ -0,0 +1,321 @@
---
Copy link
Contributor

Choose a reason for hiding this comment

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

@mafanya23 все примеры необходимо привести в порядок

@@ -0,0 +1,182 @@
---
Copy link
Contributor

Choose a reason for hiding this comment

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

@mafanya23 описание с методов тут не годится. Это просто дублирование информации. Про фильтрацию есть, что рассказать и показать. Особенно про smartSorting, которая по факту мультисортировка. Есть даже примеры в snippet tool

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

Comments