Skip to content

Commit

Permalink
Merge pull request #5312 from tk0miya/5294_quickstart_colorscheme_for…
Browse files Browse the repository at this point in the history
…_windows

Fix #5294: sphinx-quickstart blank prompts in PowerShell
  • Loading branch information
tk0miya committed Aug 19, 2018
2 parents 6e06ec2 + a22f9df commit b50b73e
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGES
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ Bugs fixed
* #5211: autodoc: No docs generated for functools.partial functions
* #5306: autodoc: ``getargspec()`` raises NameError for invalid typehints
* #5298: imgmath: math_number_all causes equations to have two numbers in html
* #5294: sphinx-quickstart blank prompts in PowerShell

Testing
--------
Expand Down
8 changes: 7 additions & 1 deletion sphinx/cmd/quickstart.py
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,12 @@

PROMPT_PREFIX = '> '

if sys.platform == 'win32':
# On Windows, show questions as bold because of color scheme of PowerShell (refs: #5294).
COLOR_QUESTION = 'bold'
else:
COLOR_QUESTION = 'purple'


# function to get input from terminal -- overridden by the test suite
def term_input(prompt):
Expand Down Expand Up @@ -189,7 +195,7 @@ def do_prompt(text, default=None, validator=nonempty):
prompt = prompt.encode('utf-8')
except UnicodeEncodeError:
prompt = prompt.encode('latin1')
prompt = colorize('purple', prompt, input_mode=True)
prompt = colorize(COLOR_QUESTION, prompt, input_mode=True)
x = term_input(prompt).strip()
if default and not x:
x = default
Expand Down

0 comments on commit b50b73e

Please sign in to comment.