diff options
| author | sillylaird <sillyfanboy@gmail.com> | 2026-09-03 00:33:59 +0000 |
|---|---|---|
| committer | sillylaird <sillyfanboy@gmail.com> | 2026-09-03 00:33:59 +0000 |
| commit | 898b52edcb47bcb3e9d6106e74ca73e74ea01e70 (patch) | |
| tree | 85c6ee5ad58b860144551184d4cf86b560c62b91 /.agents/skills/web-design-reviewer/references/framework-fixes.md | |
| download | www-898b52edcb47bcb3e9d6106e74ca73e74ea01e70.tar.gz www-898b52edcb47bcb3e9d6106e74ca73e74ea01e70.zip | |
Diffstat (limited to '')
| -rw-r--r-- | .agents/skills/web-design-reviewer/references/framework-fixes.md | 475 |
1 files changed, 475 insertions, 0 deletions
diff --git a/.agents/skills/web-design-reviewer/references/framework-fixes.md b/.agents/skills/web-design-reviewer/references/framework-fixes.md new file mode 100644 index 0000000..d400ca1 --- /dev/null +++ b/.agents/skills/web-design-reviewer/references/framework-fixes.md @@ -0,0 +1,475 @@ +# Framework-specific Fix Guide + +This document explains specific fix techniques for each framework and styling method. + +--- + +## Pure CSS / SCSS + +### Fixing Layout Overflow + +```css +/* Before: Overflow occurs */ +.container { + width: 100%; +} + +/* After: Control overflow */ +.container { + width: 100%; + max-width: 100%; + overflow-x: hidden; +} +``` + +### Text Clipping Prevention + +```css +/* Single line truncation */ +.text-truncate { + overflow: hidden; + text-overflow: ellipsis; + white-space: nowrap; +} + +/* Multi-line truncation */ +.text-clamp { + display: -webkit-box; + -webkit-line-clamp: 3; + -webkit-box-orient: vertical; + overflow: hidden; +} + +/* Word wrapping */ +.text-wrap { + word-wrap: break-word; + overflow-wrap: break-word; + hyphens: auto; +} +``` + +### Spacing Unification + +```css +/* Unify spacing with CSS custom properties */ +:root { + --spacing-xs: 0.25rem; + --spacing-sm: 0.5rem; + --spacing-md: 1rem; + --spacing-lg: 1.5rem; + --spacing-xl: 2rem; +} + +.card { + padding: var(--spacing-md); + margin-bottom: var(--spacing-lg); +} +``` + +### Improving Contrast + +```css +/* Before: Insufficient contrast */ +.text { + color: #999999; + background-color: #ffffff; +} + +/* After: Meets WCAG AA standards */ +.text { + color: #595959; /* Contrast ratio 7:1 */ + background-color: #ffffff; +} +``` + +--- + +## Tailwind CSS + +### Layout Fixes + +```jsx +{/* Before: Overflow */} +<div className="w-full"> + <img src="..." /> +</div> + +{/* After: Overflow control */} +<div className="w-full max-w-full overflow-hidden"> + <img src="..." className="w-full h-auto object-contain" /> +</div> +``` + +### Text Clipping Prevention + +```jsx +{/* Single line truncation */} +<p className="truncate">Long text...</p> + +{/* Multi-line truncation */} +<p className="line-clamp-3">Long text...</p> + +{/* Allow wrapping */} +<p className="break-words">Long text...</p> +``` + +### Responsive Support + +```jsx +{/* Mobile-first responsive */} +<div className=" + flex flex-col gap-4 + md:flex-row md:gap-6 + lg:gap-8 +"> + <div className="w-full md:w-1/2 lg:w-1/3"> + Content + </div> +</div> +``` + +### Spacing Unification (Tailwind Config) + +```javascript +// tailwind.config.js +module.exports = { + theme: { + extend: { + spacing: { + '18': '4.5rem', + '22': '5.5rem', + }, + }, + }, +} +``` + +### Accessibility Improvements + +```jsx +{/* Add focus state */} +<button className=" + bg-blue-500 text-white + hover:bg-blue-600 + focus:outline-none focus:ring-2 focus:ring-blue-500 focus:ring-offset-2 +"> + Button +</button> + +{/* Improve contrast */} +<p className="text-gray-700 bg-white"> {/* Changed from text-gray-500 */} + Readable text +</p> +``` + +--- + +## React + CSS Modules + +### Fixes in Module Scope + +```css +/* Component.module.css */ + +/* Before */ +.container { + display: flex; +} + +/* After: Add overflow control */ +.container { + display: flex; + flex-wrap: wrap; + overflow: hidden; + max-width: 100%; +} +``` + +### Component-side Fixes + +```jsx +// Component.jsx +import styles from './Component.module.css'; + +// Before +<div className={styles.container}> + +// After: Add conditional class +<div className={`${styles.container} ${isOverflow ? styles.overflow : ''}`}> +``` + +--- + +## styled-components / Emotion + +### Style Fixes + +```jsx +// Before +const Container = styled.div` + width: 100%; +`; + +// After +const Container = styled.div` + width: 100%; + max-width: 100%; + overflow-x: hidden; + + @media (max-width: 768px) { + padding: 1rem; + } +`; +``` + +### Responsive Support + +```jsx +const Card = styled.div` + display: grid; + grid-template-columns: repeat(3, 1fr); + gap: 1.5rem; + + @media (max-width: 1024px) { + grid-template-columns: repeat(2, 1fr); + } + + @media (max-width: 640px) { + grid-template-columns: 1fr; + gap: 1rem; + } +`; +``` + +### Consistency with Theme + +```jsx +// theme.js +export const theme = { + colors: { + primary: '#2563eb', + text: '#1f2937', + textLight: '#4b5563', // Improved contrast + }, + spacing: { + sm: '0.5rem', + md: '1rem', + lg: '1.5rem', + }, +}; + +// Usage +const Text = styled.p` + color: ${({ theme }) => theme.colors.text}; + margin-bottom: ${({ theme }) => theme.spacing.md}; +`; +``` + +--- + +## Vue (Scoped Styles) + +### Fixing Scoped Styles + +```vue +<template> + <div class="container"> + <p class="text">Content</p> + </div> +</template> + +<style scoped> +/* Applied only to this component */ +.container { + max-width: 100%; + overflow: hidden; +} + +.text { + /* Fix: Improve contrast */ + color: #374151; /* Was: #9ca3af */ +} + +/* Responsive */ +@media (max-width: 768px) { + .container { + padding: 1rem; + } +} +</style> +``` + +### Deep Selectors (Affecting Child Components) + +```vue +<style scoped> +/* Override child component styles (use cautiously) */ +:deep(.child-class) { + margin-bottom: 1rem; +} +</style> +``` + +--- + +## Next.js / App Router + +### Global Style Fixes + +```css +/* app/globals.css */ +:root { + --foreground: #171717; + --background: #ffffff; +} + +/* Prevent layout overflow */ +html, body { + max-width: 100vw; + overflow-x: hidden; +} + +/* Prevent image overflow */ +img { + max-width: 100%; + height: auto; +} +``` + +### Fixes in Layout Components + +```tsx +// app/layout.tsx +export default function RootLayout({ children }) { + return ( + <html lang="en"> + <body className="min-h-screen flex flex-col"> + <header className="sticky top-0 z-50"> + {/* Header */} + </header> + <main className="flex-1 container mx-auto px-4 py-8"> + {children} + </main> + <footer> + {/* Footer */} + </footer> + </body> + </html> + ); +} +``` + +--- + +## Common Patterns + +### Fixing Flexbox Layout Issues + +```css +/* Before: Items overflow */ +.flex-container { + display: flex; + gap: 1rem; +} + +/* After: Wrap and size control */ +.flex-container { + display: flex; + flex-wrap: wrap; + gap: 1rem; +} + +.flex-item { + flex: 1 1 300px; /* grow, shrink, basis */ + min-width: 0; /* Prevent flexbox overflow issues */ +} +``` + +### Fixing Grid Layout Issues + +```css +/* Before: Fixed column count */ +.grid-container { + display: grid; + grid-template-columns: repeat(4, 1fr); +} + +/* After: Auto-adjust */ +.grid-container { + display: grid; + grid-template-columns: repeat(auto-fit, minmax(250px, 1fr)); + gap: 1.5rem; +} +``` + +### Organizing z-index + +```css +/* Systematize z-index */ +:root { + --z-dropdown: 100; + --z-sticky: 200; + --z-modal-backdrop: 300; + --z-modal: 400; + --z-tooltip: 500; +} + +.modal { + z-index: var(--z-modal); +} +``` + +### Adding Focus States + +```css +/* Add focus state to all interactive elements */ +button:focus-visible, +a:focus-visible, +input:focus-visible, +select:focus-visible, +textarea:focus-visible { + outline: 2px solid #2563eb; + outline-offset: 2px; +} + +/* Customize focus ring */ +.custom-focus:focus-visible { + outline: none; + box-shadow: 0 0 0 3px rgba(37, 99, 235, 0.5); +} +``` + +--- + +## Debugging Techniques + +### Visualizing Element Boundaries + +```css +/* Use only during development */ +* { + outline: 1px solid red !important; +} +``` + +### Detecting Overflow + +```javascript +// Run in console to detect overflow elements +document.querySelectorAll('*').forEach(el => { + if (el.scrollWidth > el.clientWidth) { + console.log('Horizontal overflow:', el); + } + if (el.scrollHeight > el.clientHeight) { + console.log('Vertical overflow:', el); + } +}); +``` + +### Checking Contrast Ratio + +```javascript +// Use Chrome DevTools Lighthouse or axe DevTools +// Or check at the following site: +// https://webaim.org/resources/contrastchecker/ +``` |
