Skip to content

Conversation

@github-actions
Copy link
Contributor

No description provided.

@github-actions github-actions bot requested a review from a team as a code owner December 11, 2023 16:46
1. In the component's initialization method (`OnInitializedAsync`), load the data and set the `PageSize` property to the total count of the records.
2. Add the "All" option (`null`) to the `PageSizes` collection.
3. Bind the `PageSize` property to the `@bind-PageSize` attribute of the `TelerikGrid` component.
4. Specify the `PageSizes` collection in the `GridPagerSettings` component to display the dropdown with the available page sizes.
Copy link
Contributor

Choose a reason for hiding this comment

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

There should be a step about setting PageSize to the data count

Copy link
Contributor

Choose a reason for hiding this comment

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

This is listed in the first step:

  1. In the component's initialization method (OnInitializedAsync), load the data and set the PageSize property to the total count of the records.
Copy link
Contributor

@dimodi dimodi left a comment

Choose a reason for hiding this comment

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

  • Refactor tags
  • Rename the slug
  • Add a step for pagesize = data count
  • Add caption for the code snippet
ntacheva and others added 5 commits January 2, 2024 16:35
Co-authored-by: Dimo Dimov <961014+dimodi@users.noreply.github.com>
Co-authored-by: Dimo Dimov <961014+dimodi@users.noreply.github.com>
Co-authored-by: Dimo Dimov <961014+dimodi@users.noreply.github.com>
Co-authored-by: Dimo Dimov <961014+dimodi@users.noreply.github.com>
@ntacheva ntacheva requested a review from dimodi January 2, 2024 14:43
@ntacheva ntacheva added the merge-to-production Use this label to get a comment to choose whether to merge the PR to production label Jan 3, 2024
@github-actions
Copy link
Contributor Author

github-actions bot commented Jan 3, 2024

Hello @ntacheva,

Check the below option if you would like to automatically generate PR to production. The automation uses the branch for the cherry-pick, and then will delete the branch. Please, do not delete it manually.

  • create PR to production
@ntacheva ntacheva merged commit 8ad0855 into master Jan 3, 2024
@github-actions
Copy link
Contributor Author

github-actions bot commented Jan 3, 2024

Here you are the link to your PR to production: 1832

@github-actions github-actions bot deleted the new-kb-set-all-page-size-option-as-default-77088c6f035547d699b97a8568b79b0b branch January 3, 2024 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-to-production Use this label to get a comment to choose whether to merge the PR to production

4 participants