Skip to content

fix(notes): open critical warnings only for the links they guard - #7188

Open
eli32-vlc wants to merge 1 commit into
fmhy:mainfrom
eli32-vlc:fix/critical-note-targets
Open

eli32-vlc wants to merge 1 commit into
fmhy:mainfrom
eli32-vlc:fix/critical-note-targets

Conversation

@eli32-vlc

Copy link
Copy Markdown
Contributor

The warning dialog used to open for the first link in any list item with a critical note. On the Beginners Guide, AudiobookBay's warning opened on Mobilism Audiobooks and not on AudiobookBay, and mirrors and peers such as WinRAR [2], MyInsta, HushGram, Instafel and Betterfox skipped their warning.

Guarded links are now tagged with data-critical-note at build time: every primary peer and numbered mirror before the description, or, on the Beginners Guide, the link right before the warning. Layout.vue opens the dialog only for tagged links.

The warning dialog used to open for the first link in any list item with a
critical note. On the Beginners Guide, AudiobookBay's warning opened on
Mobilism Audiobooks and not on AudiobookBay, and mirrors and peers such as
WinRAR [2], MyInsta, HushGram, Instafel and Betterfox skipped their warning.

Guarded links are now tagged with data-critical-note at build time: every
primary peer and numbered mirror before the description, or, on the
Beginners Guide, the link right before the warning. Layout.vue opens the
dialog only for tagged links.
@github-actions github-actions Bot added the core Core changes to our VitePress setup label Nov 11, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Core changes to our VitePress setup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant