Bug 215418 – Add Copy WebKit Permalink items to Sublime Text command palette

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Devin Rousso]

I use this all the time, and it would be nice to have a a way to invoke "Copy WebKit Permalink" without having to right-click :)

[Devin Rousso]

Created attachment 406466 [details]
Patch

[Keith Miller]

Comment on attachment 406466 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=406466&action=review

rs=me

> Tools/CopyPermalink/Sublime Text/CopyWebKitPermalink/Context.sublime-commands:3
> +    { "caption": "Copy WebKit Permalink", "command": "copy_web_kit_permalink" },
> +    { "caption": "Copy WebKit Permalink to Blame", "command": "copy_web_kit_permalink", "args": { "annotate_blame": true } },

What's with the underscores between web and kit? Don't we normally use one word even when not capitalizing?

[Devin Rousso]

Comment on attachment 406466 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=406466&action=review

>> Tools/CopyPermalink/Sublime Text/CopyWebKitPermalink/Context.sublime-commands:3
>> +    { "caption": "Copy WebKit Permalink to Blame", "command": "copy_web_kit_permalink", "args": { "annotate_blame": true } },
> 
> What's with the underscores between web and kit? Don't we normally use one word even when not capitalizing?

I believe Sublime Text snakecases any `sublime_plugin.Command` using capital letters as delimiters.  I don't know if it's possible to control over that.

FWIW this is what `Tools/CopyPermalink/Sublime Text/CopyWebKitPermalink/Context.sublime-menu` does too ¯\_(ツ)_/¯

[EWS]

Committed r265557: <https://trac.webkit.org/changeset/265557>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 406466 [details].

[Radar WebKit Bug Importer]

<rdar://problem/66925840>