Remove contextual links from same_page'ed preview

should we be displaying the version of the content that the ADMINISTRATOR sees, or should we be displaying the content that AUTHENTICATED users see?

Perhaps it is somewhere in-between? Showing what the current authenticated user would see, but with UI that isn't relevant to this task (previewing content) disabled.

We're already taking some conceptually similar steps when embedding the preview. In same_page_preview_preprocess_html we remove the admin toolbar. Seems reasonable to me that we might also disable things like quickedit.

Yes, I see a path there. We could pick and choose pieces to manually remove. The list of things to possibly tweak / exclude is not infinite. But a more complete solution would be to show a preview as non-admin sees it.

Since proposing that, I came to think of all the edge cases that might cause.

• Previewing a secure page
• Pages that have different contexts applied (analytics driven displays, language
• lots of stuff I'm probably not thinking of

I think the purpose of the same page preview is to not overwhelm the user with choices. Just the preview please. The full preview could the place to get all specific with choices / personalization / formats. There certainly would be space for all that.

With that in mind, I'm all for doing what we have to do to get this happy path handled.

cosmicdreams → closed merge request !12

I have a MR that removes contextual links from the preview

This looks good from my perspective.