when accepting, offer to place {{Db-afc-move}} when the destination is a redirect #338
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Feature
In addition
pointer-events: 'none'
. easier than removing then re-adding the click listenerNotes
addFormSubmitHandler( handleAcceptOverRedirect )
because we already didaddFormSubmitHandler( handleAccept )
, and it's buggy if you do it twice{{Db-afc-move}}
, because if the redirect is tagged with this at the top (normal place for CSD tags), then it won't technically be a redirect anymore and our earlier check for "does page exist and is not a redirect?" will catch thisFixes #243