Skip to content

Fix UI for PR draft status check entry#4370

Merged
alexr00 merged 8 commits intomicrosoft:mainfrom
Thomas1664:ready-for-review
Jan 5, 2023
Merged

Fix UI for PR draft status check entry#4370
alexr00 merged 8 commits intomicrosoft:mainfrom
Thomas1664:ready-for-review

Conversation

@Thomas1664
Copy link
Contributor

@Thomas1664 Thomas1664 commented Dec 26, 2022

Before:

image

After:

image

Without icon:

image

This screenshot was taken before I figured out that the icon in the first row of status checks was misaligned.

@Thomas1664 Thomas1664 marked this pull request as draft December 26, 2022 22:37
@Thomas1664 Thomas1664 marked this pull request as ready for review December 26, 2022 22:37
.ready-for-review-text-wrapper {
display: flex;
gap: 8px;
align-items: center;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also fixes icon on first row of status checks (see 'before' screenshot) is misaligned.

@alexr00 alexr00 self-assigned this Dec 27, 2022
@alexr00 alexr00 added this to the January 2023 milestone Dec 27, 2022
Copy link
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

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

The editor webview looks good, but this adversely affects the sidebar webview
image

@Thomas1664 Thomas1664 requested a review from alexr00 December 27, 2022 10:55
Copy link
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

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

The sidebar webview now has the "Ready for review" button before the "This pr is a draft" message. It should be after.

image

Otherwise it looks good.

@Thomas1664 Thomas1664 requested a review from alexr00 January 5, 2023 12:22
Copy link
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

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

Looks good, thank you!

@alexr00 alexr00 merged commit 697a3cf into microsoft:main Jan 5, 2023
@Thomas1664 Thomas1664 deleted the ready-for-review branch January 5, 2023 15:31
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.

5 participants