Skip to content

Conversation

hoaaah
Copy link
Contributor

@hoaaah hoaaah commented Jan 30, 2022

I have a problem when create datatable without id. Datatable id also used in bootstrap4 accordion item in Yii2. And from code, I've found you can assign different datatable id, but It is not explain enough in readme.

I have a problem when create datatable without id. Datatable id also used in bootstrap4 accordion item in Yii2. And from code, I've found you can assign different datatable id, but It is not explain enough in readme.
@dmytrokarpovych
Copy link

Hello 👋🏻
Thanks for PR
Could you explain problem with missing ID in details?

We didn’t add this part to readme because this ability have all widgets at Yii2

@hoaaah
Copy link
Contributor Author

hoaaah commented Jan 30, 2022

Yii2 Widget create id for rendered widget. I use Accordion Widget at my page, and this widget rendered one of accordion item with id #w5.

I create datatable in modal that use renderAjax, and datatable use the same id with accordion item (#w5). So it fail to rendered with this warning. In other pages it stacked with other id too, like breadcrumb or menu.

I realized there is no explanation how to change this id (#w5) from readme, but when I read the code, you actually can set your datatable id. So it's nice for other to know how to set datatable id when their datatable id stacked with other widget id.

@ZAYEC77
Copy link
Contributor

ZAYEC77 commented Jan 30, 2022

Yii2 Widget create id for rendered widget. I use Accordion Widget at my page, and this widget rendered one of accordion item with id #w5.

I create datatable in modal that use renderAjax, and datatable use the same id with accordion item (#w5). So it fail to rendered with this warning. In other pages it stacked with other id too, like breadcrumb or menu.

I realized there is no explanation how to change this id (#w5) from readme, but when I read the code, you actually can set your datatable id. So it's nice for other to know how to set datatable id when their datatable id stacked with other widget id.

Yeah, there is nothing that we could do in this case that just set Id manually

Good point to add this section to readme

@ZAYEC77 ZAYEC77 merged commit 276e85b into NullRefExcep:master Jan 30, 2022
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.

3 participants