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

installer: shorten text wrap for terminal captions #69

Merged
merged 1 commit into from Aug 20, 2015
Merged

installer: shorten text wrap for terminal captions #69

merged 1 commit into from Aug 20, 2015

Conversation

charego
Copy link

@charego charego commented Aug 19, 2015

The wizard page for choosing your terminal emulator truncates some words. See examples in the screenshot below (I am using Windows 8.1, 1920x1080, not sure it matters):

gitinstallerterminalemulator

Would this work? I haven't tried building a new installer and running it. Will give it a try if needed.

@nalla
Copy link
Contributor

nalla commented Aug 20, 2015

This at least partly addresses the same issue as #70.

@charego
Copy link
Author

charego commented Aug 20, 2015

Yep. How detailed should the Windows version be? "Windows 8.1" or more detailed?

@dscho
Copy link
Member

dscho commented Aug 20, 2015

@charlesrgould detailed enough so I can quickly start the correct VM and test for myself... ;-)

@dscho
Copy link
Member

dscho commented Aug 20, 2015

But actually, I can reproduce the issue on Windows Server 2012 R2, so all is good. Testing your patch now (still, it would be nice if you could update the commit message as indicated).

The wizard page for choosing your terminal emulator for Git truncated
some words, e.g. "window" became "wind" and "as" was hidden completely.

This was seen on Windows 8.1 and Windows Server 2012 R2.

Signed-off-by: Charles Gould <charles.r.gould@gmail.com>
@charego
Copy link
Author

charego commented Aug 20, 2015

@dscho - Done, rebased as well, just... because.

dscho added a commit that referenced this pull request Aug 20, 2015
…mulator-captions

installer: shorten text wrap for terminal captions
@dscho dscho merged commit 886cc28 into git-for-windows:master Aug 20, 2015
@dscho
Copy link
Member

dscho commented Aug 20, 2015

Perfect, thank you very much!

@dscho dscho mentioned this pull request Aug 20, 2015
@charego charego deleted the hotfix/installer-terminal-emulator-captions branch August 20, 2015 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants