Skip to content

Update Configure Tabs Indicator on Content Size Change [SYNTH-258] - #1428

Open
AlexD717 wants to merge 2 commits into
devfrom
alexey/258/selected-tab-indicator-misaligned
Open

Update Configure Tabs Indicator on Content Size Change [SYNTH-258]#1428
AlexD717 wants to merge 2 commits into
devfrom
alexey/258/selected-tab-indicator-misaligned

Conversation

@AlexD717

@AlexD717 AlexD717 commented Jul 20, 2026

Copy link
Copy Markdown
Member

Task

SYNTH-258

Symptom

Navigating to configure -> inputs -> Ernie (or any other specific input) causes the tab indicator to be misaligned

image

This also happens if you initially enter a specific input option and then you navigate to the root inputs option menu

Solution

Refresh the tab indicator when the content changes size

Verification

  • Going from the root input configuration panel to a specific input configuration causes the tab indicator to update positions to match the new panel size

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.

@AlexD717 AlexD717 self-assigned this Jul 20, 2026
@AlexD717 AlexD717 added bug Fixes an issue ui/ux Relating to user interface, or in general, user experience labels Jul 20, 2026
@AlexD717
AlexD717 marked this pull request as ready for review July 20, 2026 21:19
@AlexD717
AlexD717 requested review from a team as code owners July 20, 2026 21:19

@rutmanz rutmanz 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

@0xda157 0xda157 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.

LGTM

@AlexD717

Copy link
Copy Markdown
Member Author

PR #1460 should be merged first to be able to properly verify that the changes still work

@BrandonPacewic BrandonPacewic added the blocked Blocked by another task label Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked Blocked by another task bug Fixes an issue ui/ux Relating to user interface, or in general, user experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants