The preview is showing redundant controls which should be shown in a new window instead

I'll be looking at this one this weekend. My rush to get the dev release out may have excluded some work we have to make sure this doesn't happen.

@rkoller I attempted to author some testing steps that seem to describe what you did to create this situation. If they're wrong, can you please edit them?

Good point. thanks. i forgot to make the steps more explicit since i did more or less nothing aside requiring and installing the module. but made a few adjustments to the steps you've outlined. (and by the way the duplicated view mode drop down is still 100% reproducible on my end - happened on another page i quickly installed the module on to recheck the steps. version used is from 14 Mar 2023 at 22:10 UTC)

Here's the testing steps I've done:

1. Created a new site from scratch
2. Use composer to get the dev version of same_page_preview
3. Install site using demo_umami installation profile.
4. Edit an existing piece of content.

Here's what I see:

I can't reproduce the bug. @rkoller can you?

I actually posted this on https://www.drupal.org/project/same_page_preview/issues/3348315#comment-... 🐛 Installation on already existing sites broken with no real preview shown Fixed but looking at this again, this actually feels like the one that is more like a cache-adjacent issue. We're doing some altering to remove the controls from the preview embedded in the iframe. This feels like the alter hasn't registered yet somehow.

I'm still not able to duplicate this locally though.

I suppose I've figured out the root cause. the redundant controls were due to the fact that i've used drupal 10.1.x for testing purposes instead of 10.0.x. Realized that in a conversation with @cosmicdreams today when i was trying to get a drupalpod instance running. there i've learned that 10.1.x uses a different render element and something has to change in same page preview for that. installed now three different instances (one drupalpod and two local one) and it turns out in none of them the redundant controls have showed up. closing this issue as works as designed.