Skip to content

#109255 - Add encoding description to settings "File: Encoding" drop-down#109551

Merged
roblourens merged 3 commits intomicrosoft:masterfrom
maldahleh:109255-support-detail-text-settings-dropdown
Nov 10, 2020
Merged

#109255 - Add encoding description to settings "File: Encoding" drop-down#109551
roblourens merged 3 commits intomicrosoft:masterfrom
maldahleh:109255-support-detail-text-settings-dropdown

Conversation

@maldahleh
Copy link
Contributor

@maldahleh maldahleh commented Oct 28, 2020

This PR fixes #109255 by adding the encoding key in a new detail field under the "File: Encoding" dropdown of Settings, this is similar to how encodings are displayed in the "Select file encoding to save with" action.

Questions

  • Is it okay to introduce new colours like I did? I assume this might have implications on themes?
  • What are your thoughts on the shade of grey? I tried finding something that matched the background of the dropdown. I wasn't able to re-use the colour from the "Select file encoding to save with" dropdown since that dropdown has a different background colour.

Screenshots
Screen Shot 2020-10-27 at 9 13 54 PM

@ghost
Copy link

ghost commented Oct 28, 2020

CLA assistant check
All CLA requirements met.

@bpasero bpasero assigned roblourens and unassigned bpasero Oct 28, 2020
@roblourens roblourens added this to the November 2020 milestone Oct 29, 2020
@roblourens
Copy link
Member

This looks like a good start, we are stabilizing a release this week and I will review it in depth next week.

@roblourens
Copy link
Member

Made a small change to remove the new registered color, and match the styling of the editor encoding quick pick. Thanks for the PR!

@roblourens roblourens merged commit 038835d into microsoft:master Nov 10, 2020
@maldahleh maldahleh deleted the 109255-support-detail-text-settings-dropdown branch November 10, 2020 19:10
@github-actions github-actions bot locked and limited conversation to collaborators Dec 25, 2020
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.

Support detail text in settings editor dropdown

3 participants