- Notifications
You must be signed in to change notification settings - Fork669
add: star command in shortcuts#601
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Thanks@yihong0618 I would suggest to use |
@jdneo |
Ok, given the current rendering result, seems using text is better. But I think there is no need to put it as one of the default value |
Maybe in setting that user can config it open it or not? |
Do you mean using a setting to toggle between text and icon? |
Forget it, I think for now, keep the text maybe a better way. |
I think so too. |
package.json Outdated
| @@ -621,7 +621,8 @@ | |||
| "type": "array", | |||
| "default": [ | |||
| "submit", | |||
| "test" | |||
| "test", | |||
| "star" | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
No need to add this shortcut by default. But please add it in the items, and also documents in readme
@jdneo Fixed. |
README.md Outdated
| - `Submit`: Submit your answer to LeetCode. | ||
| - `Test`: Test your answer with customized test cases. | ||
| - `Star/Unstar`: Star or Unstar the current problem. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
nit: unstar
| @@ -629,6 +629,7 @@ | |||
| "enum": [ | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Could you by the way add"enumDescriptions" for this setting here? 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Fixed, and optimize star logic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thanks@yihong0618 !
@jdneo@zry656565
Support star/unstar in codelen shortcuts/