Skip to content

web: go back to wrapping-by-default and add section dividers#2690

Merged
nicks merged 1 commit into
masterfrom
nicks/wrap
Dec 18, 2019
Merged

web: go back to wrapping-by-default and add section dividers#2690
nicks merged 1 commit into
masterfrom
nicks/wrap

Conversation

@nicks

@nicks nicks commented Dec 18, 2019

Copy link
Copy Markdown
Contributor

Hello @hyu,

Please review the following commits I made in branch nicks/wrap:

c2a234a (2019-12-18 13:39:24 -0500)
web: go back to wrapping-by-default and add section dividers

@nicks
nicks requested a review from hyu December 18, 2019 18:39
@nicks

nicks commented Dec 18, 2019

Copy link
Copy Markdown
Contributor Author

Screenshot from 2019-12-18 13-39-56

@hyu hyu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is much better overall! Glad to be rid of the horizontal scroll.

Comment thread web/src/LogPane.tsx Outdated
@nicks

nicks commented Dec 18, 2019

Copy link
Copy Markdown
Contributor Author

here's what it looks like now:

Screenshot from 2019-12-18 14-21-44

@nicks
nicks merged commit e22eb8e into master Dec 18, 2019
@nicks
nicks deleted the nicks/wrap branch December 18, 2019 21:03
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