Add shortTitle to execSelectionInTerminal command #25007
Conversation
|
@Dinesh-0813 Can you remove the package.lock. it might be using the wrong version of npm. |
package.json
Outdated
| @@ -1567,7 +1569,7 @@ | |||
| "cross-env": "^7.0.3", | |||
| "cross-spawn": "^6.0.5", | |||
| "del": "^6.0.0", | |||
| "download": "^8.0.0", | |||
| "download": "^3.3.0", | |||
There was a problem hiding this comment.
undo this version change, and revert the package-lock.json
|
@microsoft-github-policy-service agree |
|
Hi, I've reverted the package-lock.json and package.json changes as requested. Please review again. Thank you! |
|
You will still want to keep the title addition in package.json, just without the module version bump |
|
Hi, I’ve added the title field back to package.json as %shortTitle% without modifying the version. Let me know if anything else is needed. Thanks! |
package.json
Outdated
| @@ -2,6 +2,7 @@ | |||
| "name": "python", | |||
| "displayName": "Python", | |||
| "description": "Python language support with extension access points for IntelliSense (Pylance), Debugging (Python Debugger), linting, formatting, refactoring, unit tests, and more.", | |||
| "title": "%shortTitle%", | |||
There was a problem hiding this comment.
It should really be on the command not the extension.
|
Thanks for the feedback! I've moved the shortTitle to the command as suggested. |
There was a problem hiding this comment.
"Hi @connor4312 @karthiknadig, I have fixed the missing shortTitle in package.nls.json as suggested. Kindly review. Thanks!"
package.json
Outdated
| @@ -325,7 +325,8 @@ | |||
| { | |||
| "category": "Python", | |||
| "command": "python.execSelectionInTerminal", | |||
| "title": "%python.command.python.execSelectionInTerminal.title%" | |||
| "title": "%python.command.python.execSelectionInTerminal.title%", | |||
| "shortTitle": "%python.menu.execSelectionInTerminal.title%" | |||
There was a problem hiding this comment.
This should be python.command.python.execSelectionInTerminal.shortTitle
…lectionInTerminal.shortTitle
…lectionInTerminal.shortTitle
|
@karthiknadig Thank you for the review. I've fixed the shortTitle value in package.json to use the correct reference "%python.command.python.execSelectionInTerminal.shortTitle%" as you suggested. This now properly references the existing localization string in package.nls.json instead of using the incorrect menu-based reference. The changes have been pushed to my branch. Please let me know if there are any other adjustments needed. |
…lectionInTerminal.shortTitle
package.nls.json
Outdated
| @@ -5,8 +5,9 @@ | |||
| "python.command.python.createNewFile.title": "New Python File", | |||
| "python.command.python.createTerminal.title": "Create Terminal", | |||
| "python.command.python.execInTerminal.title": "Run Python File in Terminal", | |||
| "python.menu.execSelectionInTerminal.title": "Run Selection", | |||
There was a problem hiding this comment.
This is not used anywhere. Please make sure to only make changes that are relevant to this issue.
|
sorry for that i fixed that can you please review now |
|
@connor4312 i completed your request changes please review |
karthiknadig
added
bug
|
@Dinesh-0813 Please undo the package-lock changes. |
|
Undo package-lock.json changes as requested |
|
@connor4312 please review |
hi iam a begineer from tamilnadu if i do anything wrong please let me know
thankyou!
Add shortTitle to execSelectionInTerminal command