Skip to content

feat: DLT-2419 hideEdges + watcher on active page prop#675

Merged
Sanjeev Suresh (sanjeevdialpad) merged 2 commits intostagingfrom
DLT-2419
Mar 20, 2025
Merged

feat: DLT-2419 hideEdges + watcher on active page prop#675
Sanjeev Suresh (sanjeevdialpad) merged 2 commits intostagingfrom
DLT-2419

Conversation

@sanjeevdialpad
Copy link
Copy Markdown
Contributor

feat: hideEdges for lazy loading pagination and watcher on active page prop

Obligatory GIF (super important!)

test

test

🛠️ Type Of Change

These types will increment the version number on release:

  • Fix
  • Feature
  • Performance Improvement
  • Refactor

These types will not increment the version number, but will still deploy to documentation site on release:

  • Documentation
  • Build system
  • CI
  • Style (code style changes, not css changes)
  • Test
  • Other (chore)

📖 Jira Ticket

https://dialpad.atlassian.net/browse/DLT-2419

📖 Description

activePage prop is not reactive since its assigned to data on init, so we cannot control the pagination from outside this component which is needed sometimes.

Sometimes you may need to hide start and end page number buttons when moving in between.
This prop will be used to hide the first and last page buttons when not near the edges.
This is useful when your backend does not support offset and you can only use cursor based pagination.

💡 Context

📝 Checklist

For all PRs:

  • I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
  • I have reviewed my changes.
  • I have added all relevant documentation.
  • I have considered the performance impact of my change.

For all Vue changes:

  • I have added / updated unit tests.
  • I have made my changes in Vue 2 and Vue 3. Note: you may sync your changes from Vue 2 to Vue 3 (or vice versa) using the ./scripts/dialtone-vue-sync.sh script. Read docs here: Dialtone Vue Sync Script
  • I have validated components with a screen reader.
  • I have validated components keyboard navigation.

@sanjeevdialpad Sanjeev Suresh (sanjeevdialpad) changed the title feat: hideEdges + watcher on active page prop feat: DLT-2419 hideEdges + watcher on active page prop Mar 19, 2025
Comment thread packages/dialtone-vue2/components/pagination/pagination_variants.story.vue Outdated
Comment thread packages/dialtone-vue2/components/pagination/pagination.vue
Comment thread packages/dialtone-vue2/components/pagination/pagination.vue Outdated
Comment thread packages/dialtone-vue2/components/pagination/pagination.vue Outdated
@github-actions
Copy link
Copy Markdown
Contributor

Please add either the visual-test-ready or no-visual-test label to this PR depending on whether you want to run visual tests or not.
It is recommended to run visual tests if your PR changes any UI. ‼️

@sanjeevdialpad
Copy link
Copy Markdown
Contributor Author

Brad Paugh (@braddialpad) made all the changes!

@github-actions
Copy link
Copy Markdown
Contributor

✔️ Deploy previews ready!
😎 Dialtone preview: https://dialtone.dialpad.com/deploy-previews/pr-675/
😎 Dialtone-vue 2 preview: https://dialtone.dialpad.com/vue/deploy-previews/pr-675/
😎 Dialtone-vue 3 the preview: https://dialtone.dialpad.com/vue3/deploy-previews/pr-675/

Copy link
Copy Markdown
Contributor

@braddialpad Brad Paugh (braddialpad) left a comment

Choose a reason for hiding this comment

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

Looks great thanks!

@braddialpad Brad Paugh (braddialpad) added the visual-test-ready Add this tag when the PR is ready for visual test, to trigger GHA visual tests label Mar 20, 2025
@sanjeevdialpad Sanjeev Suresh (sanjeevdialpad) merged commit 67ef1e2 into staging Mar 20, 2025
14 checks passed
Julio Ortega (juliodialpad) pushed a commit that referenced this pull request Mar 24, 2025
# [2.179.0](dialtone-vue2/v2.178.2...dialtone-vue2/v2.179.0) (2025-03-20)

### Features

* DLT-2419 hideEdges + watcher on active page prop ([#675](#675)) ([67ef1e2](67ef1e2))
Julio Ortega (juliodialpad) pushed a commit that referenced this pull request Mar 24, 2025
# [3.172.0](dialtone-vue3/v3.171.2...dialtone-vue3/v3.172.0) (2025-03-20)

### Features

* DLT-2419 hideEdges + watcher on active page prop ([#675](#675)) ([67ef1e2](67ef1e2))
Julio Ortega (juliodialpad) pushed a commit that referenced this pull request Mar 24, 2025
# [9.106.0](dialtone/v9.105.2...dialtone/v9.106.0) (2025-03-20)

### Features

* DLT-2419 hideEdges + watcher on active page prop ([#675](#675)) ([67ef1e2](67ef1e2))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

visual-test-ready Add this tag when the PR is ready for visual test, to trigger GHA visual tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants