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

[mobile ux] Copy-paste in shopfront message display strange white strips #1860

Closed
myriamboure opened this issue Sep 26, 2017 · 9 comments · Fixed by #4816
Closed

[mobile ux] Copy-paste in shopfront message display strange white strips #1860

myriamboure opened this issue Sep 26, 2017 · 9 comments · Fixed by #4816
Assignees
Labels
bug-s5 We can live with it... Few users are impacted. good first issue hackathon Issues for upcoming hackathons

Comments

@myriamboure
Copy link
Contributor

myriamboure commented Sep 26, 2017

In a feature branch or master?

This can be merged straight to master 🎉

Description

When copy-pasting from another shopfront or any text file, into the "message when shopfront open/close" (in menu enterprise setting > shop preferences), here is what happens. Some shops have done that apparently and then their shopfront is not looking good.

bug

@daniellemoorhead
Copy link
Contributor

Hey @myriamboure I think this could be something a newbie could do, so would you be able to add a little bit more information about what you did / where you did it and where this message displays? I'm not sure if it would be obvious for a new person :)

@myriamboure
Copy link
Contributor Author

Sure ! In the enterprise menu, the user write in "shopfront preference" a welcome message for his hub 1.
He creates another hub and wants to use the same welcome message. So he copies/pastes the welcome message from hub 1 to hub 2. He saves and go on the hub 2 shopfront, he sees the image above so the white strips below the text.
He is surprised so he goes back to the hub 2 menu and delete the message and retype it, but it doesn't change anything, still the white strips.
Hope it's clear enough, if any question come back to me :-)

@mllocs
Copy link
Collaborator

mllocs commented Oct 18, 2017

Pretty easy to reproduce, just paste a multi-paragraph text like:

paragraph1

paragraph2

and the WYSIWYG will generate a markup with a white background paragraph...

@lin-d-hop lin-d-hop added bug-s5 We can live with it... Few users are impacted. and removed p - low labels Dec 19, 2017
@mjal
Copy link
Contributor

mjal commented Jan 29, 2018

This seems to be fixed in the latest version of the library, copying and pasting from the editor don't add white strips anymore.

However the editor also paste the style of the text, so the example above will still highlight the text because it's inside a code segment (highlighted with a grey background)

@Matt-Yorkley
Copy link
Contributor

99.9% sure this is closed by #4655. Feel free to reopen if that's not the case.

@daniellemoorhead
Copy link
Contributor

Screen Shot 2020-02-07 at 2 12 01 pm

@Matt-Yorkley this bug is not fixed as was expected by changing to the home tab.

I think we need to strip any background colour formatting that is copied across to the fields.

The shop where this is happening is https://openfoodnetwork.org.au/local-produce-ladies/shop#/home

@mkllnk mkllnk self-assigned this Feb 7, 2020
@mkllnk
Copy link
Member

mkllnk commented Feb 7, 2020

@daniellemoorhead I think this is already solved. Can you check?

@daniellemoorhead
Copy link
Contributor

@mkllnk here's another example of background colour displaying. https://staging.katuma.org/fredo-s-farm-hub/shop#/home

I copied text from a google doc that I had highlighted in green and that background highlight colour shows up on the tab.

@mkllnk
Copy link
Member

mkllnk commented Feb 20, 2020

Ah, great. I was able to copy that example. When I copy from a document locally it doesn't transfer the background colour. 🤷‍♀️ Maybe my Linux is just not doing that.

@daniellemoorhead daniellemoorhead changed the title Copy-paste in shopfront message display strange white strips [mobile ux] Copy-paste in shopfront message display strange white strips Feb 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-s5 We can live with it... Few users are impacted. good first issue hackathon Issues for upcoming hackathons
Projects
None yet
Development

Successfully merging a pull request may close this issue.