Skip to content

Commit 5d0b003

Browse files
authored
Merge pull request #75 from swup/fix/update-link-to-fragment-for-same-url
2 parents 6f0f43d + 1c5101e commit 5d0b003

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

src/inc/defs.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ export type Route = {
77
};
88

99
/** The interface for an augmented Fragment Element */
10-
export interface FragmentElement extends Element {
10+
export interface FragmentElement extends HTMLElement {
1111
__swupFragment?: {
1212
url?: string;
1313
selector?: string;

src/inc/functions.ts

+1-2
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,6 @@ function handleLinksToFragments({ logger, swup }: FragmentPlugin): void {
7373
if (isEqualUrl(fragmentUrl, swup.getCurrentUrl())) {
7474
// prettier-ignore
7575
if (__DEV__) logger?.warn(`The fragment URL of ${selector} is identical to the current URL. This could mean that [data-swup-fragment-url] needs to be provided by the server.`);
76-
return;
7776
}
7877

7978
el.href = fragmentUrl;
@@ -372,7 +371,7 @@ export function queryFragmentElement(
372371
): FragmentElement | undefined {
373372
for (const containerSelector of swup.options.containers) {
374373
const container = document.querySelector(containerSelector);
375-
if (container?.matches(fragmentSelector)) return container;
374+
if (container?.matches(fragmentSelector)) return container as FragmentElement;
376375

377376
const fragment = container?.querySelector<FragmentElement>(fragmentSelector);
378377
if (fragment) return fragment;

0 commit comments

Comments
 (0)