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

TW-1135: preserve users linebreaks in message content #1277

Merged
merged 1 commit into from
Jan 12, 2024

Conversation

imGok
Copy link
Contributor

@imGok imGok commented Jan 4, 2024

linagora/flutter_matrix_html#12
linagora/matrix-dart-sdk#42

Demo

Twake.Chat.-.Google.Chrome.2024-01-09.17-17-04.mp4

Linebreaks proofs

image
image

#1135

Copy link

github-actions bot commented Jan 4, 2024

This PR has been deployed to https://linagora.github.io/twake-on-matrix/1277

pubspec.lock Outdated Show resolved Hide resolved
sherlockvn
sherlockvn previously approved these changes Jan 5, 2024
@hoangdat hoangdat force-pushed the TW-1135-linebreaks-pubspeclock branch from e65179d to 077915f Compare January 5, 2024 04:08
@hoangdat hoangdat force-pushed the TW-1135-linebreaks-pubspeclock branch from 077915f to 644086c Compare January 5, 2024 04:09
@hoangdat
Copy link
Member

hoangdat commented Jan 5, 2024

I tested, it not work well, please show the demo.

@imGok imGok changed the title TW-1135: update pubspec lock WIP TW-1135: update pubspec lock Jan 5, 2024
@imGok imGok force-pushed the TW-1135-linebreaks-pubspeclock branch from 644086c to 44b715b Compare January 9, 2024 15:45
@imGok imGok changed the base branch from main to upgrade_3.16 January 9, 2024 15:45
@imGok imGok changed the title WIP TW-1135: update pubspec lock TW-1135: preserve users linebreaks in message content Jan 9, 2024
@imGok imGok force-pushed the TW-1135-linebreaks-pubspeclock branch from 44b715b to b5507b6 Compare January 10, 2024 13:51
@imGok
Copy link
Contributor Author

imGok commented Jan 10, 2024

@hoangdat I confirm it's working on my side after cleaning all libs etc. the deployment is not updated

@sherlockvn
Copy link
Collaborator

Work from my side
image

@sherlockvn
Copy link
Collaborator

Error when i try:
"""
adsasd
adsasdd
"""
result:
image
expected:
"""
adsasd
adsasdd
"""

@imGok
Copy link
Contributor Author

imGok commented Jan 11, 2024

Error when i try:

"""

adsasd

adsasdd

"""

result:

image

expected:

"""

adsasd

adsasdd

"""

It's because we trim the string, it's intentional. @hoangdat do we remove it ?

@imGok
Copy link
Contributor Author

imGok commented Jan 11, 2024

@sherlockvn
Please check again on your local version (pr deployment is not updated)

Twake.Chat.-.Google.Chrome.2024-01-11.15-13-47.mp4

@sherlockvn
Copy link
Collaborator

Okay i can confirm it's work from my side too
image

@imGok imGok force-pushed the TW-1135-linebreaks-pubspeclock branch from 9e82783 to 4ec9b23 Compare January 12, 2024 11:09
@imGok imGok changed the base branch from upgrade_3.16 to main January 12, 2024 11:09
@imGok imGok dismissed sherlockvn’s stale review January 12, 2024 11:09

The base branch was changed.

@hoangdat hoangdat merged commit 360b950 into main Jan 12, 2024
@hoangdat hoangdat deleted the TW-1135-linebreaks-pubspeclock branch January 12, 2024 11:42
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.

3 participants