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

Realign slash command content widget in panel chat #190053

Merged
merged 3 commits into from
Aug 9, 2023

Conversation

joyceerhl
Copy link
Contributor

@joyceerhl joyceerhl commented Aug 9, 2023

Fix #190039

image
image

@joyceerhl joyceerhl self-assigned this Aug 9, 2023
@joyceerhl joyceerhl enabled auto-merge (squash) August 9, 2023 14:59
@VSCodeTriageBot VSCodeTriageBot added this to the August 2023 milestone Aug 9, 2023
@@ -4,7 +4,8 @@
*--------------------------------------------------------------------------------------------*/

.chat-slash-command-content-widget {
padding: 0 3px 1px 3px;
padding: 1px 2px 0px 1px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Possible to do this without changing the internal cell padding?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry I don't follow, can you elaborate?

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry, wrong word choice above. This changes the padding of the pill itself which might make it look lopsided the top has visibly more padding than the bottom or vice versa. Note the extra top space in the pill pictured in the issue for createWorkspace.

Ideally the pill, pill text, and input text can all be vertically dead center and then we don't have to worry about lopsided padding.

IIRC you can change the input editor text position with some editor API? Happy to checkout and take a look if you'd like.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think the issue is that the input text outside the pill is itself 1px below dead center, which is noticeable if you enter some text and hit ctrl+a (shown below), so if we want the text inside the pill to be vertically centered, it will be misaligned with the other text in the input. IMO it's more important that the text inside the pill is aligned with text outside the pill, I pushed a change which tries to do this:

image

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks great. Made one change to align everything vertically as a group.

CleanShot 2023-08-09 at 11 48 02@2x

@joyceerhl joyceerhl merged commit 01f748f into main Aug 9, 2023
6 checks passed
@joyceerhl joyceerhl deleted the dev/joyceerhl/brilliant-asp branch August 9, 2023 19:36
@github-actions github-actions bot locked and limited conversation to collaborators Sep 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Chat pill is misalign wrt text
4 participants