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

Improve visibility of hyphen/dash in the PowerShell ISE theme when typing in the editor #862

Merged
merged 1 commit into from
Jun 12, 2017

Conversation

stefanstranger
Copy link
Contributor

@stefanstranger stefanstranger commented Jun 12, 2017

Solves issue with not seeing dashes when editor line is selected.

Fixes #864.

The lineHighlighBackground settings has the same color as dashes being used when typing interactively. 

Changed setting from Changed editor.lineHighlightBackground setting from #94c6f7 to #add8e6

More info: PowerShell/PowerShellEditorServices#511
@msftclas
Copy link

@stefanstranger,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.

It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@daviwil
Copy link
Contributor

daviwil commented Jun 12, 2017

You shouldn't have to sign a CLA for a one-line change, I think the automation bot is getting confused by all the other commits. Can you check the "Allow edits from maintainers" box here:

image

Once that's checked I can fix it for you.

@rkeithhill
Copy link
Contributor

I wonder if this should be rebased against upstream/master?

@daviwil
Copy link
Contributor

daviwil commented Jun 12, 2017

Yeah, that's the fix, I just didn't want him to have to learn how to rebase for his first PR ;)

@daviwil
Copy link
Contributor

daviwil commented Jun 12, 2017

Strange, I tried rebasing his branch and it appears to be up to date. Not sure what's going on there...

@rkeithhill
Copy link
Contributor

rkeithhill commented Jun 12, 2017

The PR is trying to merge into develop. Thought we were going to drop develop and just use master? I'm guessing at this point, develop is way behind master.

@daviwil daviwil changed the base branch from develop to master June 12, 2017 17:29
@daviwil
Copy link
Contributor

daviwil commented Jun 12, 2017

Nice catch! That was the issue.

@daviwil
Copy link
Contributor

daviwil commented Jun 12, 2017

I'm going to close/reopen to see if it gets CLA bot to not require CLA.

@daviwil daviwil closed this Jun 12, 2017
@daviwil daviwil reopened this Jun 12, 2017
@msftclas
Copy link

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@rkeithhill
Copy link
Contributor

It's only taken about a year and a half but I'm finally starting to grok Git with much help/coaching from you - so thanks!

@daviwil daviwil added this to the 1.3.2 milestone Jun 12, 2017
@daviwil
Copy link
Contributor

daviwil commented Jun 12, 2017

Haha, you give me too much credit.

Thanks for the PR Stefan, merging it!

@daviwil daviwil merged commit 2c59773 into PowerShell:master Jun 12, 2017
@daviwil daviwil changed the title Patch 1 Improve visibility of hyphen/dash in the PowerShell ISE theme when typing in the editor Jun 12, 2017
@mattmcnabb
Copy link
Contributor

Was this just an update to the editor highlight color? I'm not certain what dashes were not visible - @stefanstranger do you have a screenshot?

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