Skip to content

feat: Copy with Redirect context menu#448

Open
brian6932 wants to merge 1 commit intoeinaregilsson:masterfrom
brian6932:feat/copy-with-redirect
Open

feat: Copy with Redirect context menu#448
brian6932 wants to merge 1 commit intoeinaregilsson:masterfrom
brian6932:feat/copy-with-redirect

Conversation

@brian6932
Copy link

@brian6932 brian6932 commented Jul 24, 2025

This has no AI slop :), the only issue's that if you check for matches when showing the context menu, it's a bit inconsistent to have it append. I'll see if I can improve that, I don't think I can hold the rendering of the context menu with a promise. Append ?w=1 to the diff to get a less crazy looking diff, there was a lot of trailing whitespace that my editor decided to trim.

closes #443
closes #414


Challenges:

  • Writing to Chromium's clipboard requires moving some logic out of background, or using the different legacy API.
  • After taking a look at Chrome's contextMenus API for this, it's a lot more basic than Firefox's menus, so it's not possible to have match based contexts across browsers (e.g. https://issues.chromium.org/issues/40467152). It'd be cool if targetUrlPatterns worked on link types. As a workaround, I added a global toggle.

@SmartManoj
Copy link

SmartManoj commented Jul 24, 2025

Append ?w=1 to the diff to get a less crazy looking diff, there was a lot of trailing whitespace that my editor decided to trim.

For audience, one can use Refined GitHub for hide whitespace button.

image

@brian6932
Copy link
Author

Yea, it's also under the cog without any extensions.

@brian6932 brian6932 force-pushed the feat/copy-with-redirect branch 8 times, most recently from 7d227f9 to 38480aa Compare July 24, 2025 04:48
@brian6932 brian6932 force-pushed the feat/copy-with-redirect branch from 38480aa to 8c4d2c8 Compare July 24, 2025 09:31
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.

Feature Request: Copy with Redirect RMB Context Menu Action

2 participants