Skip to content

added cameron/frontend_v4#33

Open
camerontarget14 wants to merge 3 commits intoAcademySoftwareFoundation:mainfrom
camerontarget14:main
Open

added cameron/frontend_v4#33
camerontarget14 wants to merge 3 commits intoAcademySoftwareFoundation:mainfrom
camerontarget14:main

Conversation

@camerontarget14
Copy link
Collaborator

Includes:

  • Side bar from @jspada200
  • Removed upper settings tray

Signed-off-by: camerontarget14 <127965975+camerontarget14@users.noreply.github.com>
@camerontarget14 camerontarget14 linked an issue Dec 11, 2025 that may be closed by this pull request
Signed-off-by: camerontarget14 <127965975+camerontarget14@users.noreply.github.com>
This was linked to issues Jan 25, 2026
@jspada200
Copy link
Collaborator

We can go the way of doing it manually but I will say that I am a bit concerned about not using a existing package and global manager. We are leaving some functionality on the table that would be easier to maintain and implement. For example, there is a lot of duplicate logic. Take a look at the long render statments in settings and the check for hotkeys effect. There is a lot of logic that needs to be implemented every time a hotkey is added. For things that are discoverable and configurable, a context provider works best. The way I see this MR, it should implement A) The callbacks for the hotkeys, B) add hotkeysConfig.ts file that has the hostkeys and mapped methods and C) the package implemented.

There are some other things the packages offer as well such as displaying the hotkey on tools tips on thier corresponding buttons that we would get for free. There are also some lower level guards agienst conflicting events and focus traps.

All that being said, not going to die on hotkey hill XD If there is somthing we are not getting from using an off the shelf package, then we can reimplement.

@camerontarget14
Copy link
Collaborator Author

@jspada200 at a high level, that all makes a lot of sense to me! I'd be happy to take another pass so it's off your plate, but won't be till the weekend..

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.

Centeralized Hotkey Manager Settings Dialog Wireframe Prototypes

2 participants