Skip to content

Bento: components fail to hide placeholder content once loaded #34616

@westonruter

Description

@westonruter

Description

When using amp-twitter 1.0 (Bento) with placeholder content as follows:

<amp-twitter width="600" height="480" layout="responsive" data-tweetid="1397995435386679302" class="twitter-tweet" data-width="550" data-dnt="true">
  <blockquote class="twitter-tweet" data-width="550" data-dnt="true" placeholder="">
    <p lang="en" dir="ltr">
      ⛺🔥
      Fun fact: cAMPfire stories are the best stories.<br>
      <a href="https://twitter.com/CrystalOnScript?ref_src=twsrc%5Etfw">@CrystalOnScript</a>,
      <a href="https://twitter.com/nainar92?ref_src=twsrc%5Etfw">@nainar92</a>, and Caroline Liu gather around to
      unbox the Bento developer preview, which allows you to use AMP ⚡
      components everywhere!<br> <br>
      📺
      Tune in → <a href="https://t.co/RViEFP4AR9">https://t.co/RViEFP4AR9</a> <a href="https://t.co/Ni8B5VNZVz">pic.twitter.com/Ni8B5VNZVz</a>
    </p>
    — AMP Project (@AMPhtml) <a href="https://twitter.com/AMPhtml/status/1397995435386679302?ref_src=twsrc%5Etfw">May
    27, 2021</a>
  </blockquote>
</amp-twitter>

The placeholder content fails to be hidden when the component initializes, causing overlapping text:

image

See example on playground.

Reproduction Steps

Add amp-twitter 1.0 with placeholder content.

Relevant Logs

No response

Browser(s) Affected

No response

OS(s) Affected

No response

Device(s) Affected

No response

AMP Version Affected

2105150310000

Metadata

Metadata

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions