Skip to content

fix(docs-framework): let examples shrink to fit window width#5075

Merged
thatblindgeye merged 1 commit into
patternfly:mainfrom
mcoker:issue-5074
Jul 15, 2026
Merged

fix(docs-framework): let examples shrink to fit window width#5075
thatblindgeye merged 1 commit into
patternfly:mainfrom
mcoker:issue-5074

Conversation

@mcoker

@mcoker mcoker commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

fixes #5074

In #4802, the example area was given the change below, which added min-width: 825px. That's keeping the examples from shrinking below 825px

Screenshot 2026-07-14 at 6 01 11 PM

I'm pretty sure that was to fix a bug on example pages that have narrow example content and no text that would make the example area fill the content width like this. AFAIK it only showed up on one example page, but I don't remember which example it was.

Screenshot 2026-07-14 at 6 06 33 PM

That said, flex-grow: 1 fixes that specific bug ^, so we can revert back to the old max-width and fix the bug caused by the new min-width.

@mcoker
mcoker requested review from jcmill and thatblindgeye July 14, 2026 23:14
@patternfly-build

Copy link
Copy Markdown
Collaborator

Preview: https://pf-org--pr-5075-site.surge.sh

@thatblindgeye thatblindgeye left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming it would have still been at exactly 1450px width, the original ToC bug doesnt seem to happen with this update (ToC goes from the collapsible view to the always-visible-on-the-right view when going from 1449px to 1450px width)

@jcmill jcmill left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👾

@thatblindgeye
thatblindgeye merged commit 5bbd5ef into patternfly:main Jul 15, 2026
5 of 7 checks passed
@patternfly-build

Copy link
Copy Markdown
Collaborator

Your changes have been released in:

  • @patternfly/ast-helpers@1.4.0-alpha.393
  • @patternfly/documentation-framework@6.49.1

Thanks for your contribution! 🎉

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 this pull request may close these issues.

Bug - docs framework - examples do not shrink below 825px

4 participants