Skip to content

Remove pagehide event from sidebar persistence script - #4088

Merged
delucis merged 3 commits into
mainfrom
chris/pagehide
Jul 21, 2026
Merged

delucis merged 3 commits into
mainfrom
chris/pagehide

Conversation

@delucis

@delucis delucis commented Jul 21, 2026 •

Copy link
Copy Markdown
Member

Description

@changeset-bot

changeset-bot Bot commented Jul 21, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 3601b56

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@astrojs/starlight Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@netlify

netlify Bot commented Jul 21, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for astro-starlight ready!

Name Link
🔨 Latest commit 3601b56
🔍 Latest deploy log https://app.netlify.com/projects/astro-starlight/deploys/6a5f5f819ea7b800082c8b7d
😎 Deploy Preview https://deploy-preview-4088--astro-starlight.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 100 (no change from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the 🌟 core Changes to Starlight’s main package label Jul 21, 2026
@delucis delucis changed the title Chris/pagehide Remove pagehide event from sidebar persistence script Jul 21, 2026
@astrobot-houston

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size
/index.html 6.14 KB (-0.12% 🔽)
/_astro/*.js 25.35 KB (0%)
/_astro/*.css 14.7 KB (0%)

@HiDeoo HiDeoo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Makes total sense to me and like you said, Safari 14.5 is way below our minimum supported version.

Great call 🧹

@delucis
delucis merged commit 4486ba4 into main Jul 21, 2026
22 of 25 checks passed
@delucis
delucis deleted the chris/pagehide branch July 21, 2026 12:14
@astrobot-houston astrobot-houston mentioned this pull request Jul 21, 2026
dadezzz pushed a commit to dadezzz/university_notes that referenced this pull request Jul 26, 2026
This PR contains the following updates:

| Package | Change | [Age](https://docs.renovatebot.com/merge-confidence/) | [Confidence](https://docs.renovatebot.com/merge-confidence/) |
|---|---|---|---|
| [@astrojs/starlight](https://starlight.astro.build) ([source](https://github.com/withastro/starlight/tree/HEAD/packages/starlight)) | [`0.41.3` → `0.41.4`](https://renovatebot.com/diffs/npm/@astrojs%2fstarlight/0.41.3/0.41.4) | ![age](https://developer.mend.io/api/mc/badges/age/npm/@astrojs%2fstarlight/0.41.4?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@astrojs%2fstarlight/0.41.3/0.41.4?slim=true) |

---

### Release Notes

<details>
<summary>withastro/starlight (@&#8203;astrojs/starlight)</summary>

### [`v0.41.4`](https://github.com/withastro/starlight/blob/HEAD/packages/starlight/CHANGELOG.md#0414)

[Compare Source](https://github.com/withastro/starlight/compare/@astrojs/starlight@0.41.3...@astrojs/starlight@0.41.4)

##### Patch Changes

- [#&#8203;3936](withastro/starlight#3936) [`712eedd`](withastro/starlight@712eedd) Thanks [@&#8203;miichom](https://github.com/miichom)! - Fixes support for modifying Zod enums when passing an [`extend` option](https://starlight.astro.build/reference/frontmatter/#extend) to Starlight’s `docsSchema()`

- [#&#8203;4092](withastro/starlight#4092) [`0896b91`](withastro/starlight@0896b91) Thanks [@&#8203;delucis](https://github.com/delucis)! - Fixes support for links containing a protocol like `mailto:` in the sidebar

- [#&#8203;4088](withastro/starlight#4088) [`4486ba4`](withastro/starlight@4486ba4) Thanks [@&#8203;delucis](https://github.com/delucis)! - Simplifies Starlight’s client-side sidebar state persistence script slightly

</details>

---

### Configuration

📅 **Schedule**: (UTC)

- Branch creation
  - At any time (no schedule defined)
- Automerge
  - At any time (no schedule defined)

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Mend Renovate](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4yNzIuNiIsInVwZGF0ZWRJblZlciI6IjQzLjI3Mi42IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌟 core Changes to Starlight’s main package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SidebarPersistState listens for 'pageHide' (camelCase), so it never fires

3 participants