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

Turn on WordWrap in license view #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Pawe1
Copy link
Contributor

@Pawe1 Pawe1 commented Jul 10, 2017

Turning on WordWrap makes license much more readable in extreme cases 😁 (there is only 1 scrollbar enabled)

Before this it was possible to get some 2-liiiiiiiiiiiiiiiine horror...

@Pawe1
Copy link
Contributor Author

Pawe1 commented Aug 30, 2017

Any comment?...

@Memnarch
Copy link
Owner

2 line horrow? An example? Haven't seen a file without unix/windows linebreaks yet(and it should convert them properly)
(Sorry for responding this late, quite busy :( )

@Pawe1
Copy link
Contributor Author

Pawe1 commented Aug 31, 2017

hmm...
I have investigated this case and it looks like bug... I just figured out that it happens after installation of package
CR/LF I guess?
It seems that this pull request is only workaround - You can ignore it 😛

Screenshot

BTW - Embarcadero also quickly got interested in Grijjy packages and incorporated some code into GetIt.
I have downloaded it to test and inside readme file... there is a note from GitHub readme about Delphinus xD
This was wise strategy xD xD xD

@Memnarch
Copy link
Owner

ahh ok the bug is, it does not replace the unix breaks with windows breaks for the installed version!

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.

2 participants