Skip to content

docs(refresher): add new section for pull start and end events#4421

Open
brandyscarney wants to merge 3 commits intofeature-8.8from
FW-6591
Open

docs(refresher): add new section for pull start and end events#4421
brandyscarney wants to merge 3 commits intofeature-8.8from
FW-6591

Conversation

@brandyscarney
Copy link
Member

@brandyscarney brandyscarney commented Mar 4, 2026

⚠️ Remove dev builds before merging ⚠️

Adds a new section for ionPullStart and ionPullEnd events for Refresher.

Preview

@vercel
Copy link

vercel bot commented Mar 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ionic-docs Ready Ready Preview, Comment Mar 4, 2026 11:05pm

Request Review

Comment on lines +16 to +17
// TODO: this needs to be exported by angular
import { RefresherPullEndCustomEvent } from '@ionic/core';
Copy link
Member Author

Choose a reason for hiding this comment

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

I can remove this if we plan to merge & release 8.8.1 with the fix in it before merging the docs.

@brandyscarney brandyscarney marked this pull request as ready for review March 4, 2026 22:24
@brandyscarney brandyscarney requested a review from a team as a code owner March 4, 2026 22:24
@brandyscarney brandyscarney requested review from thetaPC and removed request for a team March 4, 2026 22:24
Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

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

Plus what Shane found


The `ionPullEnd` event is emitted when the refresher returns to an inactive state, with a reason property of `'complete'` or `'cancel'` indicating whether the refresh operation completed successfully or was canceled.

import PullStartEndEvents from '@site/static/usage/v8/refresher/pull-start-end-events/index.md';
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't this be at the top? For consistency? Or is this the new structure?

@@ -0,0 +1,81 @@
```html
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
```html
```vue

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