Issue

https://otwarchive.atlassian.net/browse/AO3-4989

Purpose

This makes it so you return to referer if you delete a bookmark unless you came from a page specifically editing or deleting it. I saw the pattern used in the comment controller but I'm not sure it it's the best way of implementing this.

Testing

Delete bookmark from all possible places. If page still exists (user bookmarks second page, recent bookmarks, etc.) then you should still be at that page.

Credit

Tal Hayon

Please use he.

Can keep it like it originally was and just add the page parameter to the path (i.e. redirect_to user_bookmarks_path(current_user, page: params[:page])), or does that lead to an error?

Yeah, I think we can consider that behavior separately and focus on the problem described in the issue.

Looking at the code again, I really don't think the optimal solution would be to add the page somehow... the deletion link is deep inside the rendering parts and has no idea where it is rendered, nor I think it should know.

If this is not acceptable, I recommend closing this pull request and extending the requirements of this feature.