Skip to content

Added max-width to videos to prevent them escaping the page content area - #4204

Merged
ssddanbrown merged 3 commits into
BookStackApp:developmentfrom
snowc0de:development
Apr 27, 2023
Merged

ssddanbrown merged 3 commits into
BookStackApp:developmentfrom
snowc0de:development

Conversation

@snowc0de

Copy link
Copy Markdown
Contributor

This is to prevent that videos included in pages don't exceed the page border. This is to avoid the following issue:
image

Which now becomes this:

image

I didn't build the project and only added the CSS code into my instance's "head config" but as far as I know this fix should work.

This is to prevent that videos included in pages don't exceed the page border
@ssddanbrown

Copy link
Copy Markdown
Member

Thanks for offering. Setting a width could cause further issues though (videos with pre-defined sizes, videos smaller than the page width). Instead, a max-width may be a better approach, within our page content style block so its clear this rule relates to video in page content (Rather than in the CSS reset).

@snowc0de

Copy link
Copy Markdown
Contributor Author

Thanks for the feedback @ssddanbrown, I just made the change :)

@ssddanbrown

Copy link
Copy Markdown
Member

Thanks for updating @chopin2712. Will merge for next release.


Weakly related to #1760

@ssddanbrown
ssddanbrown merged commit a9d9525 into BookStackApp:development Apr 27, 2023
@ssddanbrown ssddanbrown changed the title Adding a video { width: 100%; } Added max-width to videos to prevent them escaping the page content area Apr 27, 2023
@JNR8

JNR8 commented Jun 18, 2024

Copy link
Copy Markdown

Did this get implemented? My videos till break page width on mobile devices.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants