search

facelessuser / ApplySyntax

Syntax detector for Sublime Text
https://facelessuser.github.io/ApplySyntax/
343 stars 48 forks source link

Use standardized name for palette entry #154

Open FichteFoll opened 1 year ago

FichteFoll commented 1 year ago

I keep not finding them because entering prefapply doesn't yield the expected result.

facelessuser commented 1 year ago

I'm not necessarily opposed to preference vs settings, but I generally try to prefix all plugin commands with the plugin name.

facelessuser commented 1 year ago

Also, it is mentioned "standardized". Where is this standard? Generally, I've standardized my commands as described above.

FichteFoll commented 1 year ago

Mostly it's something that I noticed as a trend in many packages being used and also adapted into the template file for PackageDev. Though I just saw that I forgot to add "Settings" at the end (also in the template). Another argument to group it behind "Preferences" is that this is where it would go in the main menu file.

2023-01-08_19-39-49

https://github.com/SublimeText/PackageDev/blob/742c8154f15d1f9fdb2bc868ca4a4b086ac304c6/plugins/new_resource_file/templates.py#L37-L50

facelessuser commented 1 year ago

Okay, let me think about this. It definitely breaks the convention I've been following, specifically for preferences. I may look at a bigger sample size. I'll take a serious look and consider this as a real possibility. If I end up making the change in this plugin, I imagine it I'll be making this change in all my plugins as I update them, so I want to make sure this is something I want to do.