Amp-wp: Dragging not working as expected when CTA block present

Created on 21 Oct 2019  ·  7Comments  ·  Source: ampproject/amp-wp

Bug Description


If there's a CTA block present then adding a Text block and trying to drag it results in the Text block being positioned incorrectly.

This happens only if the block is in the initial position (positionTop=0 due to the CTA block), the blocks that have been positioned before already, don't have this issue.

The reason might be more related to the positionTop value 0 and not specifically with the CTA block.

Expected Behaviour


Dragging should work as expected.

Steps to reproduce

  1. Add two Pages
  2. Add CTA block
  3. Add a Text block
  4. Try dragging the Text block
  5. See the dragging not working as expected.

Screenshots


dragging

Additional context

  • WordPress version:
  • Plugin version: develop
  • Gutenberg plugin version (if applicable):
  • AMP plugin template mode:
  • PHP version:
  • OS:
  • Browser: [e.g. chrome, safari]
  • Device: [e.g. iPhone6]

_Do not alter or remove anything below. The following sections will be managed by moderators only._

Acceptance criteria

Implementation brief

QA testing instructions

Demo

Changelog entry

AMP Stories (obsolete) Bug

Most helpful comment

If you cut it and paste it in the same place on the same page (or another page), there's no issue. So it's probably not the position. It's somehow not initialised correctly.

But I guess this needs priority and AC before we investigate further.

All 7 comments

I can confirm seeing the same, but only once. When dropping the text block and dragging it again, it works as expected. Very odd.

I can also see it only once. It might have something to do with positionTop being 0.

If you cut it and paste it in the same place on the same page (or another page), there's no issue. So it's probably not the position. It's somehow not initialised correctly.

But I guess this needs priority and AC before we investigate further.

I am not seeing the same Ubuntu in Chrome and Firefox.

I can't replicate this

Looks like something fixed it meanwhile, perhaps #3584.

Also can't replicate this anymore.

Closing this for now in that case :-)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

westonruter picture westonruter  ·  5Comments

GitaStreet picture GitaStreet  ·  4Comments

swissspidy picture swissspidy  ·  5Comments

schlessera picture schlessera  ·  5Comments

miina picture miina  ·  5Comments