Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't abbrieviate product name? #5

Closed
rchl opened this issue Nov 15, 2021 · 3 comments
Closed

Don't abbrieviate product name? #5

rchl opened this issue Nov 15, 2021 · 3 comments
Labels
enhancement New feature or request fixed
Milestone

Comments

@rchl
Copy link

rchl commented Nov 15, 2021

I feel that using ST and SM abbreviations could potentially sound cryptic to some:

"caption": "CommandsBrowser: Browse available ST core commands.",
"command": "commands_browser_core_commands",
"args": {
"application": "st"
},
},
{
"caption": "CommandsBrowser: Browse available SM core commands.",
"command": "commands_browser_core_commands",
"args": {
"application": "sm"
},
},

Maybe use full product names - Sublime Text and Sublime Merge instead?

@UltraInstinct05
Copy link
Contributor

I don't have a strong opinion on this one as it's a Sublime Text specific package and is not that difficult to understand what ST & SM stand for, but sure, I can change it to Sublime Text and Sublime Merge. The more explicit, the better.

UltraInstinct05 added a commit that referenced this issue Nov 15, 2021
This commit uses full forms of ST & SM in command palette description
to make it more explicit, rather than using abbreviated forms.

This should address #5
@UltraInstinct05
Copy link
Contributor

Should be fixed in the referenced commit & will be available when a new tagged release comes out. Will close once the next tagged release is out.

@UltraInstinct05 UltraInstinct05 modified the milestones: v1.0.3, v1.1.0 Nov 15, 2021
UltraInstinct05 added a commit that referenced this issue Nov 16, 2021
This commit uses full forms in the quick panel placeholder. This is a
follow up for the previous commit. Forgot to update the placeholders to
not have abbreviated forms.

Related #5
@UltraInstinct05
Copy link
Contributor

Fixed in v1.1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request fixed
Development

No branches or pull requests

2 participants