-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Desktop: fixes long lines warpped #2447
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
Conversation
@PackElend @tessus please review this updated PR |
I'm on mobile but I see only a single line added:question: |
@PackElend adding the |
OK
But Travis CI failed???
sent from a fair mobile
…On Wed, 5 Feb 2020, 19:35 Amit singh, ***@***.***> wrote:
@PackElend <https://github.com/PackElend> adding the word-wrap:
break-word; fixes the issue of long lines wrap
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2447?email_source=notifications&email_token=AA3AYD3GPOU2LFD7UYWACT3RBMBFPA5CNFSM4KQPOPDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK4P5KI#issuecomment-582549161>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA3AYD7QRRMS32VBFCNS2A3RBMBFPANCNFSM4KQPOPDA>
.
|
@PackElend I don't why it happened, do you think is there something wrong? |
I cannot check I'm on the move.
I use only FastHub to check the PR history and that does not tell me that
much, Uou have to check it yourself
sent from a fair mobile
…On Wed, 5 Feb 2020, 19:42 Amit singh, ***@***.***> wrote:
@PackElend <https://github.com/PackElend> I don't why it happened, due
think is there something wrong?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2447?email_source=notifications&email_token=AA3AYD6GHBV7VCGBSWAMMNTRBMCDBA5CNFSM4KQPOPDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK4QWZY#issuecomment-582552423>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA3AYD4N77YTHFMLNXSMR2DRBMCDBANCNFSM4KQPOPDA>
.
|
The Travis CI error has nothing to do with your PR:
@laurent22 how is this possible? shouldn't the linter only check the staged files? |
I don't know, but it looks good so let's merge. Thanks @amitsin6h! |
Thanks 👍 |
@PackElend label me, please |
"Desktop" for the Windows/macOS/Linux app (Electron app)