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

NCD-768: Fix VCOM component UX #77

Merged
merged 2 commits into from
Mar 11, 2024
Merged

NCD-768: Fix VCOM component UX #77

merged 2 commits into from
Mar 11, 2024

Conversation

cybic
Copy link
Contributor

@cybic cybic commented Mar 8, 2024

  • Disable and disconnect HWFC when VCOM is disconnected
  • Remove enable-label
  • Use Material icon to indicate tooltip

UI Changes:
image

image

image

Here the UI is shown with one disconnected and one connected VCOM. The HWFC toggle of the disconnected VCOM is also disconnected in addition to the toggle being disabled.
image

cybic added 2 commits March 7, 2024 16:04
 * Disable and disconnect HWFC when VCOM is disconnected
 * Remove enable-label
 * Use Material icon
@cybic cybic added doc required This PR includes user-facing changes that need to be documented. ui required All PRs either need "ui required" or "ui not required". labels Mar 8, 2024
@cybic cybic requested a review from boundlesscalm March 8, 2024 14:27
@cybic cybic merged commit b8da885 into main Mar 11, 2024
5 checks passed
@cybic cybic deleted the NCD-768_fix_vcom_component_ux branch March 11, 2024 07:32
@greg-fer
Copy link
Contributor

Tooltips and edits look good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc required This PR includes user-facing changes that need to be documented. ui required All PRs either need "ui required" or "ui not required".
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants