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

UX improvements: CA-381442, XSI-1476, CA-375740 #3198

Merged
merged 5 commits into from
Aug 18, 2023
Merged

Conversation

kc284
Copy link
Contributor

@kc284 kc284 commented Aug 14, 2023

No description provided.

…t causes confusion to the user.

Signed-off-by: Konstantina Chremmou <[email protected]>
Also, prevent OnPageUpdated being called twice.

Signed-off-by: Konstantina Chremmou <[email protected]>
@kc284 kc284 added the ITE PR should be reviewed within this iteration label Aug 15, 2023
@kc284 kc284 self-assigned this Aug 15, 2023
Also, allow for a longer field label before ellipsization takes over.

Signed-off-by: Konstantina Chremmou <[email protected]>
@kc284 kc284 changed the title UX improvements: CA-381442 and XSI-1476 UX improvements: CA-381442, XSI-1476, CA-375740 Aug 15, 2023
@danilo-delbusso danilo-delbusso added ASAP PR should be reviewed as soon as possible and removed ITE PR should be reviewed within this iteration labels Aug 15, 2023
@danilo-delbusso danilo-delbusso added the 1 approval PR has been approved by one reviewer label Aug 16, 2023
@CitrixChris CitrixChris added 2 approvals PR has been approved by two reviewers and removed 1 approval PR has been approved by one reviewer labels Aug 18, 2023
@kc284 kc284 merged commit e4b2b67 into xenserver:master Aug 18, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2 approvals PR has been approved by two reviewers ASAP PR should be reviewed as soon as possible
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants