-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
fix: reset individual shortcut #5756
base: main
Are you sure you want to change the base?
fix: reset individual shortcut #5756
Conversation
🥷 Ninja i18n – 🛎️ Translations need to be updatedProject
|
lint rule | new reports | level | link |
---|---|---|---|
Missing translation | 112 | warning | contribute (via Fink 🐦) |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5756 +/- ##
=======================================
Coverage 57.63% 57.63%
=======================================
Files 320 320
Lines 14043 14043
Branches 2038 2038
=======================================
Hits 8094 8094
Misses 5686 5686
Partials 263 263
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Should we change the opacity (eg. 70%), change cursor to default, and change the tooltip to "Keybinding is already the default", when the keybinding is already the one it would be after an eventual reset? |
Yes, that sounds right. On it! |
Hey @MayurSMahajan , I've added the behavior I spoke about previously for disabling the button when the keybinding is already the default. |
Oh Thanks! |
@LucasXu0 what is stopping this PR from getting merged? |
Nothing. Give me some time to review it. |
Solves: #5684
Feature Preview
Here is a feature preview:
individual_shortcut.mp4
PR Checklist