🐛 fix(dialog): remove hardcoded max-width for dialog variants

[viniciuscigma]

What was done? 📝

Removida a classe sm:max-w-[425px] hardcoded do dialogVariants (no arquivo .ts e na documentação). Isso evita o travamento da largura em telas maiores e permite que o tamanho do modal seja sobrescrito e customizado livremente via class externa.

Screenshots or GIFs 📸

|-----Figma-----|
|-----Implementation-----|
| N/A | N/A |

Link to Issue 🔗

Closes #491

Type of change 🏗

• New feature (non-breaking change that adds functionality)
• Bug fix (non-breaking change that fixes an issue)
• Refactor (non-breaking change that improves the code or technical debt)
• Chore (none of the above, such as upgrading libraries)

Breaking change 🚨

Nenhuma.

Checklist 🧐

• Tested on Chrome
• Tested on Safari
• Tested on Firefox
• No errors in the console

Summary by CodeRabbit

• Style
  • Modified dialog width behavior on smaller screens and above for improved layout consistency.

[coderabbitai]

📝 Walkthrough

Walkthrough

Removed the sm:max-w-[425px] responsive max-width constraint from the dialog component's Tailwind styling in the dialogVariants CVA configuration. This eliminates the fixed width limitation on small screens and above, allowing the dialog layout to respond more flexibly to its container.

Changes

Cohort / File(s)	Summary
Dialog Variants Styling libs/zard/src/lib/shared/components/dialog/dialog.variants.ts	Removed sm:max-w-[425px] responsive max-width constraint from the CVA-based dialogVariants, allowing dialog width to be determined by other factors or overrides.

Possibly related issues

• [Bug] Dialog width hardcoded #491 — Directly addresses the removal of the hardcoded sm:max-w-[425px] width constraint that was preventing dialogs from responding appropriately to layout requirements.

Poem

🎭 A dialog unbound, no longer confined,
The constraint removed, freedom redefined,
Widths now adapt with grace and ease,
Layout's horizons expand to please! 📦✨

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately reflects the main change: removing hardcoded max-width styling from dialog variants, enabling flexible customization.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check	✅ Passed	PR description covers all essential template sections with appropriate detail: objective is clear, issue link provided, change type selected, and testing checklist completed.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[mikij]

LGTM