MainHUD UI Redesign [SYNTH-63]#1377
Conversation
Only the UI components; not touching the functionality just yet Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
…ning viewcube and scoreboard to match Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
…cons Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
added some tooltips Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
…menu, cancel button for multiplayer
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
… into alexey/claude-aps-icon-fixes
…-redesign Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
AlexD717
left a comment
AlexD717
left a comment
There was a problem hiding this comment.
Apart from this minor issue lgtm.
|
|
||
| <Box flexGrow={1} /> | ||
|
|
||
| {import.meta.env.DEV && ( |
There was a problem hiding this comment.
I guess this is up for debate, but don't we only want the developer tools to be unavailable. Debug tools should still be there as they provide useful features (ex, drag mode).
rutmanz
left a comment
rutmanz
left a comment
There was a problem hiding this comment.
Looks like the navbar doesn't correctly detect when to go into the dropdown
Ellie's screenshot from Slack
And my own reproduction
The feedback on the joints panel was never implemented
It's still massive
I know you asked for some other input on this and nobody ever replied, @azaleacolburn agrees it's way too big. To me, about 75% size is about right.
Sidebar space seems kind of wasted
This is a lot of the screen being covered for six buttons that could fit in a much smaller space. Is there a reason that it's so wide (compared to like 1 column with less vertical spacing)
Scope
The configuration panel has been kind of co-opted but still works, but you can only get there by pressing the back arrow after entering a configuration mode from the main hud or through the context menu or command palette. There are also some configuration modes that don't appear in the main hud. To me this makes it all feel half-done, and maybe just a "more" button would do the trick in the config subpanel
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
The joints config merged two sub-panels, with Joint Sequencing collapsed inside an Accordion. Since the panel is already reached through the config mode selector, that extra collapsible read as a nested dropdown. Render Joint Sequencing directly under a matching Label header (spaced via sx, since Tailwind margin utilities are overridden by MUI Typography styles) so both sections read as one integrated panel.
GameplayControls (desktop) and MobileHUD each defined an identical MultiplayerStartModal startWorldCallback. Extract it into startMultiplayerWorld() as the single source of truth and reference it from both entry points, dropping the now-unused MultiplayerSystem, PreferencesSystem, and globalAddToast imports from each.
4c9fefa to
224e5b4
Compare
|
Note: There is currently no button for the recently added camera positions. Since I don't want this to be stuck in review forever, I am fine merging this and then later fixing the minor issues. |
615fe0b to
0d44711
Compare
Preserve the Toggle Joysticks control (SYNTH-183) in the redesigned HUD: add it to TopBar and MobileHUD, gated on touch-capable devices.
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
0d44711 to
cbab725
Compare
Signed-off-by: Dhruv Arora <dhruv.arora1@autodesk.com>
rutmanz
left a comment
rutmanz
left a comment
There was a problem hiding this comment.
I think it was better to show a sidebar than to show this. I feel like it's reasonable to use synthesis split screen or otherwise in a narrow way and even if it looks a little worse we shouldn't just stop them from using it (well really it should adjust to have an overflow menu imo).
It also looks like you missed my comment on accessing the configuration menu. I think that to avoid issues like @AlexD717 brought up with the camera config button being missing, and with the reality that we will not be making icons for or fitting every configuration subpanel into the top bar/mobile hud, we should simply have a "See more" configuration button that opens the ConfigurationPanel, rather than forcing people to click one of the specific buttons and then click the back arrow
| <Stack direction="row" textAlign="center" gap={1} key={elementKey}> | ||
| {hasParent && <Spacer width={10} />} | ||
| <Tooltip | ||
| title={hasParent ? "Following Joint " + behavior.parentJointIndex : selectable ? "Set as parent" : ""} |
There was a problem hiding this comment.
so this still references the index (which is now not shown anywhere). I personally don't mind making everything use the index and type but it should be consistent.
SYNTH-63
Task
SYNTH-142
Implementing the newly designed UI with a Topbar replacing the current MainHUD implementation. In doing so, we aim to elevate the user experience, decrease friction during configuration workflows, and make Synthesis more intuitive.
The mockup design looks as such:
A screenshot of the newly implemented topbar looks like:
Mobile
SYNTH-26
The redesign also supports mobile:
Joints
Merging the joint panels
SYNTH-205
Before merging, ensure the following criteria are met: