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

Render correctly linebreaks #108

Merged
merged 4 commits into from
Aug 14, 2020

Conversation

larkox
Copy link
Contributor

@larkox larkox commented Jul 27, 2020

Summary

I copied most of the relevant styles on the posts on the webapp to render all the elements as they are in posts, solving the problem with line breaks.

The problem with line breaks is that the text:

something
some other line

was rendered as:
something some other line

Ticket Link

None

@larkox larkox added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester labels Jul 27, 2020
@larkox larkox added this to the v0.4.0 milestone Jul 27, 2020
@larkox larkox requested review from jfrerich and hanzei July 27, 2020 11:04
@larkox larkox self-assigned this Jul 27, 2020
@codecov-commenter
Copy link

codecov-commenter commented Jul 27, 2020

Codecov Report

Merging #108 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #108   +/-   ##
======================================
  Coverage    3.97%   3.97%           
======================================
  Files           9       9           
  Lines         930     930           
======================================
  Hits           37      37           
  Misses        893     893           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba724ab...c29ffc2. Read the comment docs.

@hanzei hanzei requested review from asaadmahmood and removed request for hanzei July 27, 2020 11:13
@hanzei hanzei added the 1: UX Review Requires review by a UX Designer label Jul 27, 2020
@hanzei
Copy link
Contributor

hanzei commented Jul 27, 2020

I would like to hear feedback from UX on this. Also, @asaadmahmood is much more suited to review CSS code.

Copy link
Contributor

@asaadmahmood asaadmahmood left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Copy link
Contributor

@jfrerich jfrerich left a comment

Choose a reason for hiding this comment

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

LGTM! I also tested an verified the change locally

@larkox larkox requested a review from DHaussermann July 27, 2020 15:19
@larkox larkox removed 1: UX Review Requires review by a UX Designer 2: Dev Review Requires review by a core committer labels Jul 27, 2020
Copy link

@DHaussermann DHaussermann left a comment

Choose a reason for hiding this comment

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

Tested and passed

  • Line breaks work as expected
  • Regression tested line wrapping
  • Regression tested other text formatting
    No issues found.
    LGTM!

@DHaussermann DHaussermann added 4: Reviews Complete All reviewers have approved the pull request and removed 3: QA Review Requires review by a QA tester labels Aug 13, 2020
@larkox larkox merged commit 76cd654 into mattermost-community:master Aug 14, 2020
@larkox larkox mentioned this pull request Sep 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants