Skip to content

Improve styling of joint sequencing panel [SYNTH-188] - #1366

Merged
PepperLola merged 2 commits into
devfrom
zachr/188/joints-interface-cleanup
Jul 1, 2026
Merged

Improve styling of joint sequencing panel [SYNTH-188]#1366
PepperLola merged 2 commits into
devfrom
zachr/188/joints-interface-cleanup

Conversation

@rutmanz

@rutmanz rutmanz commented Jun 30, 2026

Copy link
Copy Markdown
Member

Task

SYNTH-188

Symptom

image

Solution

image

Added tooltips and improved button placement, sizing, color, and labeling to improve clarity.

Verification

Spawn the 2471 robot from 2018 (in the assetpack) or any other robot with lots of joints and go to Configure > Sequence Joints. This new interface makes sense and feels clear to me, but my goal is to make this somewhat confusing process intuitive, so if it doesn't feel natural to you please let me know.


Before merging, ensure the following criteria are met:

  • All acceptance criteria outlined in the ticket are met.
  • Necessary test cases have been added and updated.
  • A feature toggle or safe disable path has been added (if applicable).
  • User-facing polish:
    • Ask: "Is this ready-looking?"
  • Cross-linking between Jira and GitHub:
    • PR links to the relevant Jira issue.
    • Jira ticket has a comment referencing this PR.

@rutmanz
rutmanz requested review from a team as code owners June 30, 2026 00:44

@RoushilS RoushilS left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a really good reworking to the UI. I liked your side by side approach.

UI aside though, I still think that this overall selection style is a bit unintuitive for a first time user. My idea was to replace the follow button of each joint with a dropdown named "Follows" that consists of (ex. for Joint 1) [None, Joint 2, Joint 3]. I think that it could make internal logic a bit more complicated with grandchild blocking behavior, but overall easier/faster for the user.

Lmk what you think @rutmanz

@AlexD717 AlexD717 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. Issues happen when you press cancel and the behavior gets messed up, but the issue still occurs on dev (separate PR).

@rutmanz

rutmanz commented Jun 30, 2026

Copy link
Copy Markdown
Member Author

lgtm. Issues happen when you press cancel and the behavior gets messed up, but the issue still occurs on dev (separate PR).

@AlexD717 I've noticed a few things like that, I'm tracking in SYNTH-201

@RoushilS
RoushilS self-requested a review June 30, 2026 23:05

@RoushilS RoushilS left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-request review

@PepperLola PepperLola left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good; I do think the UX could be improved but this is a good style improvement on what we alread had.

@PepperLola
PepperLola requested a review from RoushilS July 1, 2026 17:07
@PepperLola
PepperLola dismissed RoushilS’s stale review July 1, 2026 17:08

we decided the UI is fine

@PepperLola
PepperLola enabled auto-merge (squash) July 1, 2026 17:08
@PepperLola
PepperLola merged commit 6369ea1 into dev Jul 1, 2026
16 checks passed
@PepperLola
PepperLola deleted the zachr/188/joints-interface-cleanup branch July 1, 2026 17:10
BrandonPacewic added a commit that referenced this pull request Jul 1, 2026
* 'dev' of github.com:Autodesk/synthesis: (64 commits)
  Improve styling of joint sequencing panel `[SYNTH-188]` (#1366)
  Snap Field Placement `[SYNTH-94]` (#1368)
  fix: inverted joystick control scheme
  add aria label to analytics decline button (#1367)
  Remove Electron Icons `[SYNTH-203]` (#1369)
  Fix Gamepad custom selection `[SYNTH-152]` (#1361)
  Increase Minimum Sphericity to 0.99
  Sphere Game Piece Unit Tests
  Update ConfigureShotTrajectoryInterface.tsx
  Style Fixes
  style: fix
  fix: provide backup values to prevent crashing
  fix: remove move handle from modal headers
  feat: support transaltion + rotation; needed feedforward logic
  Style Fixes
  Disable Focus Selection on "None" and Field
  Style Fix
  Prevent Changing Mode while Focus is on a Field
  Biome Style Fixes
  Minor tootlip text improvement
  ...
BrandonPacewic added a commit that referenced this pull request Jul 1, 2026
* 'dev' of github.com:Autodesk/synthesis: (86 commits)
  Improve styling of joint sequencing panel `[SYNTH-188]` (#1366)
  Snap Field Placement `[SYNTH-94]` (#1368)
  fix: inverted joystick control scheme
  add aria label to analytics decline button (#1367)
  Remove Electron Icons `[SYNTH-203]` (#1369)
  Fix Gamepad custom selection `[SYNTH-152]` (#1361)
  Increase Minimum Sphericity to 0.99
  Sphere Game Piece Unit Tests
  Update ConfigureShotTrajectoryInterface.tsx
  Style Fixes
  style: fix
  fix: provide backup values to prevent crashing
  fix: remove move handle from modal headers
  feat: support transaltion + rotation; needed feedforward logic
  Style Fixes
  Disable Focus Selection on "None" and Field
  Style Fix
  Prevent Changing Mode while Focus is on a Field
  Biome Style Fixes
  Minor tootlip text improvement
  ...
@PepperLola PepperLola mentioned this pull request Jul 16, 2026
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.

4 participants