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

Code mode block vanish in the landing page builder #8325

Closed
florentpetitjean opened this issue Jan 13, 2020 · 10 comments · Fixed by #8340
Closed

Code mode block vanish in the landing page builder #8325

florentpetitjean opened this issue Jan 13, 2020 · 10 comments · Fixed by #8340
Labels
bug Issues or PR's relating to bugs builder-legacy Anything related to the legacy email or landing page builders ready-to-test PR's that are ready to test
Milestone

Comments

@florentpetitjean
Copy link

Bug Description

When adding a code mode block in the landing page builder, it can vanish if you don't let any "text" but let markup code in it.
the block is still there (visible in the browser console) but it no displaying anymore in the builder so it's hard to find it or move it again.

Q A
Mautic version 2.16.0
PHP version
Browser Chrome

Steps to reproduce

  1. Create a new landing page

  2. Go in the builder

  3. Add a "code mode block"
    image

  4. remove the place holder text in the block but let the markup in the block
    image

  5. See that the block "vanish" in your page.

Log errors

@npracht npracht added the bug Issues or PR's relating to bugs label Jan 13, 2020
@kuzmany
Copy link
Member

kuzmany commented Jan 13, 2020

But this PR should fixed it, doesn't it?

#6810

PS: I think that PR resolve just email builder, not landing page builder. Right? Can you confirm my consideration

@florentpetitjean
Copy link
Author

I think you're right, don't have this bug in Email builder so the PR have fixed email but not landing page.

@kuzmany
Copy link
Member

kuzmany commented Jan 14, 2020

@florentpetitjean
I did test on landing page and placeholder works for me. Can you re-test?

image

@florentpetitjean
Copy link
Author

I got the bug when I left code markup in the field, (like

or
) with the field totally empty it works like in your screenshot.

@kuzmany
Copy link
Member

kuzmany commented Jan 20, 2020

Let's test: #8340

Don't forget call php app/console mautic:assets:generate
And refresh cache in browser

@npracht
Copy link
Member

npracht commented Jan 24, 2020

Still not working !

@RCheesley
Copy link
Sponsor Member

Let's get this tested and merged in 3.1 - needs to be tested by 14th August.

@florentpetitjean can you take a look at the #8340 PR and give it a test?

@RCheesley RCheesley added this to the 3.1.0 milestone Jul 29, 2020
@RCheesley RCheesley added the builder-legacy Anything related to the legacy email or landing page builders label Jul 29, 2020
@florentpetitjean
Copy link
Author

Yes already review this PR, and working on production for us for a while.

@RCheesley
Copy link
Sponsor Member

Great, but your earlier review was before Mautic 3.x and rebasing which dismisses all previous reviews - can you just do a quick follow up test on M3 and report that on the PR?

@mautibot
Copy link

This issue has been mentioned on Mautic Community Forums. There might be relevant details there:

https://forum.mautic.org/t/mautic-3-1-new-beginnings/15787/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues or PR's relating to bugs builder-legacy Anything related to the legacy email or landing page builders ready-to-test PR's that are ready to test
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants