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

Shadow are cut outside the iframe on the embed #1954

Closed
2 tasks done
metmarkosaric opened this issue Jun 10, 2022 · 1 comment · Fixed by #2744
Closed
2 tasks done

Shadow are cut outside the iframe on the embed #1954

metmarkosaric opened this issue Jun 10, 2022 · 1 comment · Fixed by #2744

Comments

@metmarkosaric
Copy link
Contributor

Past Issues Searched

  • I have searched open and closed issues to make sure that the bug has not yet been reported

Issue is a Bug Report

  • This is a bug report and not a feature request, nor asking for self-hosted support

Using official Plausible Cloud hosting or self-hosting?

Plausible Cloud from plausible.io

Describe the bug

"I've been playing with embeding an iframe into a cms and shadow are cut outside the iframe which creates some nasty effect on some inputs and blocks"

Expected behavior

"From what I can see (I'm a webdev), there a two ways to fix this problem :

  • Adding a 25px (larger shadow offset) padding to the left and the right of the content. For exemple, adding the style "padding: 0px 25px;" on the body fixes the problem. Of course that should be done on embedded views only.

  • Replacing shadow with borders on borders. That would of course change the style a little but since borders don't overflow, there would be no problem."

Screenshots

No response

Environment

- OS:
- Browser:
- Browser Version:
@stevenart
Copy link

Hi.

Some more information (I reported the bug to @metmarkosaric):

  • An example wan be found here : https://jsfiddle.net/hvzonamu/
  • Screenshots attached.
  • As far as I can tell, the "bug" is browser-agnostic and os-agnostic since it is really the "normal" behavior of an iframe.

Screenshot from 2022-06-09 19-14-33
Screenshot from 2022-06-09 19-14-42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants