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

Revert "Add a   after timestamp" #5944

Merged
merged 1 commit into from
Jan 8, 2018
Merged

Revert "Add a   after timestamp" #5944

merged 1 commit into from
Jan 8, 2018

Conversation

dbkr
Copy link
Member

@dbkr dbkr commented Jan 8, 2018

Reverts #3046

This is adding a visible ' '...

@dbkr dbkr merged commit 1405859 into develop Jan 8, 2018
@rubo77
Copy link
Contributor

rubo77 commented Jan 9, 2018

What do you mean?

The idea was to add a space, so if you copy paste a conversation the time does not stick to the first word any more

@t3chguy
Copy link
Member

t3chguy commented Jan 9, 2018

@rubo77
image

@rubo77
Copy link
Contributor

rubo77 commented Jan 9, 2018

By the time I created the pr this behaviour was different.

What about now, if you copy paste a conversation do you have a space after the timestamp or not?

And maybe it would correctly be changed here if you just add a space here, not a non breaking space.

@t3chguy
Copy link
Member

t3chguy commented Jan 9, 2018

12:22PM
Matthew: Ah, sorry I caught you in a bad time. Couled I ask you to delete my account (this one) when you get the chance? I forgot my password so I can't do it by myself.

right now when I copy and paste in the Desktop app I have a new line between the timestamp and the content

@rubo77
Copy link
Contributor

rubo77 commented Jan 9, 2018

And in Firefox? I think the problem was only there

@rubo77
Copy link
Contributor

rubo77 commented Jan 9, 2018

And I didn't use the 12h format

@t3chguy t3chguy deleted the revert-3046-patch-1 branch May 12, 2022 09:02
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