Skip to content

Support fragments in redirects #2256

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ehuss opened this issue Dec 2, 2023 · 1 comment
Open

Support fragments in redirects #2256

ehuss opened this issue Dec 2, 2023 · 1 comment
Labels
A-redirect Area: the HTML redirect table C-enhancement Category: Enhancement or feature request

Comments

@ehuss
Copy link
Contributor

ehuss commented Dec 2, 2023

Problem

The redirect table does not support fragments. It would be helpful when section headings are changed to provide a way to redirect to the new location (whether or the same page, or a different page).

Proposed Solution

The redirect table should support # in the source and destination.
This is tricky because AFAIK native HTML does not support that. In my books, I have been using javascript redirects.
This would need to handle both scenarios where the original page still exists, and where it has been removed.
This should also consider any security issues like open redirects.

Notes

cc original issue #430

@ehuss ehuss added C-enhancement Category: Enhancement or feature request A-redirect Area: the HTML redirect table labels Dec 2, 2023
@KFearsoff
Copy link
Contributor

Seems related: #1760, #1715

While renames are different from permalinks, I feel like permalinks to the heading attributes would cover 80% of the use cases. Let's do that first, maybe?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-redirect Area: the HTML redirect table C-enhancement Category: Enhancement or feature request
Projects
None yet
Development

No branches or pull requests

2 participants