Skip to content

fix pageURL logic to not put double slash before slugs in some cases#53

Merged
paul90 merged 1 commit intofedwiki:mainfrom
Bortseb:patch-3
Feb 23, 2025
Merged

fix pageURL logic to not put double slash before slugs in some cases#53
paul90 merged 1 commit intofedwiki:mainfrom
Bortseb:patch-3

Conversation

@Bortseb
Copy link
Contributor

@Bortseb Bortseb commented Feb 21, 2025

Currently, if a page in an activity item's rendered body is only on a remote site, then it's href gets a double slash before the slug. This doesn't break how wiki works for some reason (if you just click the link normally), but it does break the ability to open any of those links in a new tab.

I have tested this fix locally, and it seems to work in all cases.

This is how hovering over links looked before:
image

And this is how they look now after my fix:
image

Currently, if a page in an activity item's rendered body is only on a remote site, then it's href gets a double slash before the slug. This doesn't break how wiki works for some reason (if you just click the link normally), but it does break the  ability to open any of those links in a new tab.

I have tested this fix locally, and it seems to work in all cases.

This is how hovering over links looked before:
@paul90 paul90 merged commit d62c3c8 into fedwiki:main Feb 23, 2025
3 checks passed
@paul90
Copy link
Member

paul90 commented Feb 23, 2025

@Bortseb, thank you for the fix. As main is currently a prerelease of the decaffeinated code this has been change is including in wiki-plugin-activity@0.7.0-rc.4.

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.

2 participants