SCM - Action button polish

[lszomoru]

• Left alignment is incorrect as the input field has a larger left margin
• Sync icon should appear before the text to be consistent with the Publish button
• Sync and Publish icon should not have a left margin so that the content is centred
• Rows that contain the input field and button should not have a hover feedback

//cc @misolori

[lszomoru]

[miguelsolorio]

Beautiful!

[kimadeline]

When using VS Code 1.63.0-insider commit 0cc0904 I noticed the following:

• the "Sync" button isn't as wide as the message text box
• when resizing the panel to be at its minimum size the buttons overflow:

• when clicking on the rows there is a selection outline; clicking on the message row focuses on the text box, but clicking on the button row doesn't select the button. Is it expected?

scm_button.mp4

[lramos15]

Reopening as verification found based on @kimadeline's comments. Feel free to close and have those be separate polish items

[lszomoru]

Thanks @lramos15. I will push this out to the next milestone.

[heartacker]

hi, @lszomoru could you please add an option to git sync button

1. sync by default
2. sync with rebase
   like:
   Improving the default SCM user experience #135992 (comment)
   

and if

"git.rebaseWhenSync": true

then the default button action will be sync change with rebase

[lszomoru]

Closing this issue as it is scoped to the sync button overflow functionality. We have settled on the following:

Full label when there is enough space:

Trimmed label where there is not enough space:

[LinqLover]

Very nice, thank you!