Skip to content

Shortcut For Opening Welcome Tab - #2773

Merged
UnschooledGamer merged 8 commits into
Acode-Foundation:mainfrom
Game-Dev-Shubham:main
Sep 5, 2026
Merged

Shortcut For Opening Welcome Tab #2773
UnschooledGamer merged 8 commits into
Acode-Foundation:mainfrom
Game-Dev-Shubham:main

Conversation

@Game-Dev-Shubham

Copy link
Copy Markdown
Contributor

Added new shortcut Ctrl-Shift-W to open welcome window

@greptile-apps

greptile-apps Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR adds a Ctrl-Shift-W shortcut for opening the welcome tab and corrects the command-palette label identified in the prior review.

  • Registers acode:showWelcome in the application key-binding configuration.
  • Associates the command with the existing welcome action.
  • Displays the command as “Show Welcome.”

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
src/lib/keyBindings.js Adds a correctly labeled welcome-tab shortcut whose command name and action align with the existing command registration.

Reviews (2): Last reviewed commit: "Rename welcomeTab to acode:showWelcome" | Re-trigger Greptile

Comment thread src/lib/keyBindings.js Outdated
@Game-Dev-Shubham

Copy link
Copy Markdown
Contributor Author

As a beginner I was struggling to get welcome tab again but I didn't find so I added new shortcut Ctrt-Shift-W for opening Welcome Tab , maybe it is good decision or not I don't know because this is my First Contribution ever

@RohitKushvaha01

This comment was marked as resolved.

@RohitKushvaha01 RohitKushvaha01 added the needs info / awaiting response The issue needs more information from the user before proceeding. label Aug 25, 2026
fix cmd description and rename the command name to "acode:showWelcome" so that it matches the existing command and adds the keybind to it instead of a possibly new command.
@UnschooledGamer UnschooledGamer added community area:editor Feedback, improvements related to the editor (code editing, editor iteration, formatting, etc) and removed needs info / awaiting response The issue needs more information from the user before proceeding. labels Sep 5, 2026
@UnschooledGamer

Copy link
Copy Markdown
Collaborator

Thanks for the contribution! Merging via squash. Since this was sent from your fork's main branch, your local history is now desynced from upstream. To fix your fork, recreate the main branch from this repo's main branch or (use with caution) recreate your fork if it does NOT have any unmerged changes/branch.

Thanks Again,
UG.

@UnschooledGamer

This comment was marked as resolved.

@UnschooledGamer
UnschooledGamer added this pull request to the merge queue Sep 5, 2026
Merged via the queue into Acode-Foundation:main with commit c3e6f92 Sep 5, 2026
14 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in The Code Board - Acode Sep 5, 2026
@UnschooledGamer UnschooledGamer moved this from Done to 🫡 Done by Community in The Code Board - Acode Sep 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:editor Feedback, improvements related to the editor (code editing, editor iteration, formatting, etc) community

Projects

Status: 🫡 Done by Community

Development

Successfully merging this pull request may close these issues.

3 participants