Skip to content

Conversation

@alexgrozav
Copy link
Contributor

This pull request introduces a new interactive demo component for fluid responsive design and integrates it into multiple documentation pages to illustrate how typography and layout scale fluidly across different viewport sizes. The main focus is on enhancing the documentation experience by providing a live, visual example of fluid design principles in action.

Key changes:

New Component Implementation

  • Added a new Vue component, FluidResponsiveDesignDemo.vue, which demonstrates fluid scaling of typography and card elements based on viewport width. The component uses mathematical calculations to interpolate font sizes and scale ratios, allowing users to interactively adjust the viewport and observe real-time changes.

Documentation Enhancements

  • Embedded the FluidResponsiveDesignDemo component into the fluid design documentation page (01.index.md) to provide readers with a hands-on example of fluid responsive design. [1] [2]
  • Added the demo to the main documentation index page (index.md) to showcase fluid design as a core feature, and made a minor section border adjustment for improved layout. [1] [2]
  • Included the demo in the Styleframe Pro overview page (pro.md) to highlight fluid responsive design as a foundational feature of the premium offering.

@alexgrozav alexgrozav requested a review from Copilot November 22, 2025 12:16
@alexgrozav alexgrozav self-assigned this Nov 22, 2025
@changeset-bot
Copy link

changeset-bot bot commented Nov 22, 2025

⚠️ No Changeset found

Latest commit: 90a9e0b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Nov 22, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
app-styleframe-dev-production Ready Ready Preview Comment Nov 22, 2025 0:43am
styleframe-dev-production Ready Ready Preview Comment Nov 22, 2025 0:43am

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds an interactive fluid responsive design demo component to enhance documentation by providing a visual, hands-on example of how typography and layouts scale smoothly across viewport sizes using mathematical interpolation.

Key Changes:

  • Introduced FluidResponsiveDesignDemo.vue component with interactive viewport slider and real-time scaling preview
  • Integrated the demo into fluid design documentation, Pro overview, and main index pages
  • Removed redundant border class from index.md page section

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 8 comments.

File Description
apps/docs/app/components/content/demo/FluidResponsiveDesignDemo.vue New interactive demo component showcasing fluid typography scaling with viewport adjustment controls
apps/docs/content/docs/04.design-tokens/06.fluid-design/01.index.md Embedded fluid design demo into the fluid design documentation section
apps/docs/content/pro.md Added demo to Pro overview page to highlight fluid design as a premium feature
apps/docs/content/index.md Integrated demo into main index page and simplified border styling on page section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

</p>
<div
class="cards flex gap-4"
:class="screenWidth < 768 ? 'flex-col' : 'flex-row'"
Copy link

Copilot AI Nov 22, 2025

Choose a reason for hiding this comment

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

Inconsistent breakpoint value. The template uses 768px as the threshold for switching card layout, but this value doesn't match any of the breakpoints used in viewportLabel (576, 992, 1200). Consider using a consistent breakpoint value or extracting all breakpoint values as named constants.

Copilot uses AI. Check for mistakes.
Comment on lines 78 to 79
const h1Min = baseFontSizeMin * getScalePower(2, true);
const h1Max = baseFontSizeMax * getScalePower(2, false);
Copy link

Copilot AI Nov 22, 2025

Choose a reason for hiding this comment

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

Inconsistent scale power calculation for h1 size. In fluidFontSizes computed (lines 46-48), h1 uses getScalePower(4, ...), but here in actualSizes it uses getScalePower(2, ...). This will cause the displayed size values to differ from the actual rendered sizes, leading to incorrect information shown to users.

Suggested change
const h1Min = baseFontSizeMin * getScalePower(2, true);
const h1Max = baseFontSizeMax * getScalePower(2, false);
const h1Min = baseFontSizeMin * getScalePower(4, true);
const h1Max = baseFontSizeMax * getScalePower(4, false);

Copilot uses AI. Check for mistakes.
alexgrozav and others added 6 commits November 22, 2025 14:32
…o.vue

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…o.vue

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…o.vue

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…o.vue

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…o.vue

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Nov 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@alexgrozav alexgrozav merged commit 5518c58 into main Nov 22, 2025
1 of 3 checks passed
@alexgrozav alexgrozav deleted the fluid-design-demo branch November 22, 2025 12:38
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.

2 participants