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

Shortcut refactor and fix UX issues with keyboard shortcuts #1257

Merged
merged 12 commits into from
Jan 1, 2024
Merged

Conversation

abose
Copy link
Member

@abose abose commented Dec 28, 2023

  1. Add Keyboard shortcuts on hover of top nav bar image
  2. Change nav shortcuts- Alt-Left arrow for nav back and Alt-Right arrow to nav forward
  3. Ctrl-Shift-0 - show in file tree
  4. new file - Ctrl-N - desktop only and Ctrl-1(all platforms)
  5. new folder - Ctrl-3(all platforms)
  6. new window - Ctrl-Shift-N - desktop only. In browser, use can do ctrl-T and type phcode.dev using browser shortcuts itself. So we dont make a browser shortcut for this.
  7. file Close - Ctrl-W - desktop only and Ctrl-` (all platforms)
  8. Open file - Ctrl-O in desktop only as browser doesnt support open file feature
  9. Open Folder - Ctrl-O browser only, open folder has no shortcut in desktop builds for now as its only ever used when the user opens a project once in a while.
  10. Code Folding Shortcuts:
    1. Collapse: Alt-Shift-Left instead of Ctrl-Alt-[
    2. Expand: Alt-Shift-Right instead of Ctrl-Alt-]
    3. Collapse All- Ctrl-Alt-[ instead of Alt-1 in win and Cmd-1 in mac
    4. Expand all - Ctrl-Alt-] instead of Alt-Shift-1 in win/linux and Cmd-Shift-1 in mac

Test suite enhancements

This pull request introduces a significant enhancement to our browser-based test suites by adding a 'Simulate OS' dropdown menu. This feature is designed to streamline the process of debugging and running tests across different operating systems without the need to physically switch or boot into another machine.

The feature is currently limited to browser tests and is not implemented in desktop builds. This is due to potential compatibility issues with the Tauri framework, where simulating different OS environments in desktop builds could lead to unpredictable behaviors or break the functionality.

image

Remaining shortcut related fixes for native builds beta is covered in: #1259

@phoenixide
Copy link
Member

Thank You for making this pull request.

@phoenixide
Copy link
Member

Thank You for making this pull request.

@phoenixide
Copy link
Member

Thank You for making this pull request.

@phoenixide
Copy link
Member

Thank You for making this pull request.

@abose abose mentioned this pull request Dec 30, 2023
4 tasks
@phoenixide
Copy link
Member

Thank You for making this pull request.

@phoenixide
Copy link
Member

Thank You for making this pull request.

@phoenixide
Copy link
Member

Thank You for making this pull request.

@phoenixide
Copy link
Member

Thank You for making this pull request.

Copy link

sonarcloud bot commented Jan 1, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

4 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@abose abose merged commit a9b6415 into main Jan 1, 2024
17 checks passed
@abose abose deleted the short branch January 1, 2024 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants