Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…with coming soon styling Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…onflicts Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
Agrega una sección de logos de clientes (marquee) al Hero y ajusta estilos/layout en secciones de landing, además de actualizar textos de precios y sumar assets SVG/PNG en public/.
Changes:
- Nuevo
ClientsMarqueerenderizado bajo el modelo 3D delHero. - Ajustes de alineación/altura en
HardwareAlternativesy estilos “coming soon” enStack. - Actualización de descripciones de precios en navegación y agregado de nuevos assets públicos (logos + íconos).
Reviewed changes
Copilot reviewed 6 out of 16 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/sections/landing/ClientsMarquee.tsx | Nuevo marquee animado con logos de clientes y overlays de fade lateral |
| src/sections/landing/Hero.tsx | Inserta ClientsMarquee debajo de la escena 3D |
| src/sections/landing/HardwareAlternatives.tsx | Ajusta layout para igualar alturas/alineación de tarjetas y bloques internos |
| src/sections/landing/Stack.tsx | Estiliza tarjeta “Próximamente” y chips adicionales; cambia colores/hover |
| src/data/landing.tsx | Marca el item “Próximamente” con coming: true |
| src/data/layout.tsx | Actualiza descripciones de precios (“Desde X UF/mes”) |
| public/* | Agrega íconos y logos (SVG/PNG) usados por el marquee y header |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ...((tech as any).coming | ||
| ? { | ||
| borderColor: alpha(theme.palette.divider, 0.5), | ||
| borderStyle: 'dashed', | ||
| opacity: 0.75, | ||
| } | ||
| : { |
There was a problem hiding this comment.
Avoid using (tech as any) for the new coming flag. Since TECH_STACK now includes coming, tech.coming should be typed directly (or define a TechStackItem type with coming?: boolean) so you keep type-safety and don’t lose autocomplete/refactor support.
| }), | ||
| '& .tech-logo': { transition: 'color 0.3s ease' }, | ||
| '&:hover .tech-logo': { color: 'primary.main' }, |
There was a problem hiding this comment.
&:hover .tech-logo applies even when tech.coming is true, which means the “Próximamente” icon can still turn primary on hover despite being styled as disabled. Consider gating this hover rule behind !tech.coming (or overriding it in the coming branch) to keep the disabled/coming visual state consistent.
| }), | |
| '& .tech-logo': { transition: 'color 0.3s ease' }, | |
| '&:hover .tech-logo': { color: 'primary.main' }, | |
| '&:hover .tech-logo': { color: 'primary.main' }, | |
| }), | |
| '& .tech-logo': { transition: 'color 0.3s ease' }, |
| {tech.name} | ||
| </Typography> | ||
| <Typography variant='body2' color='text.secondary'> | ||
| <Typography variant='body2' color='text.disabled'> | ||
| {tech.description} |
There was a problem hiding this comment.
Typography for tech.description is now always color='text.disabled', which will dim descriptions for all tech cards, not just the “Próximamente” one. If the intent is to mute only the coming-soon card, make this conditional (e.g., tech.coming ? 'text.disabled' : 'text.secondary').
| 0% { transform: translateX(-25%); } | ||
| 100% { transform: translateX(0); } | ||
| `; | ||
|
|
||
| export function ClientsMarquee() { | ||
| const items = [...CLIENTS, ...CLIENTS, ...CLIENTS, ...CLIENTS]; |
There was a problem hiding this comment.
items is built by duplicating CLIENTS 4 times, which renders 4× the DOM nodes (and 4× <img> elements) permanently. Consider the typical marquee pattern of rendering just 2 copies (one aria-hidden) and translating by 50%, or using a CSS background/mask approach, to reduce DOM weight while keeping the loop seamless.
| 0% { transform: translateX(-25%); } | |
| 100% { transform: translateX(0); } | |
| `; | |
| export function ClientsMarquee() { | |
| const items = [...CLIENTS, ...CLIENTS, ...CLIENTS, ...CLIENTS]; | |
| 0% { transform: translateX(0); } | |
| 100% { transform: translateX(-50%); } | |
| `; | |
| export function ClientsMarquee() { | |
| const items = [...CLIENTS, ...CLIENTS]; |
| width: 'auto', | ||
| mx: 6, | ||
| opacity: 0.35, | ||
| filter: 'brightness(0) invert(1)', |
There was a problem hiding this comment.
The logo styling forces filter: 'brightness(0) invert(1)' (white logos). Since the theme supports both light and dark color schemes, this will make the marquee nearly invisible on light backgrounds. Consider switching the filter based on the active color scheme (or use currentColor via SVGs / theme.palette.text.primary-aligned styling) so logos have appropriate contrast in both modes.
| filter: 'brightness(0) invert(1)', | |
| filter: (theme) => | |
| theme.palette.mode === 'dark' ? 'brightness(0) invert(1)' : 'none', |
Resumen
Agrega una sección de marquee de logos de clientes debajo del modelo 3D en el Hero, corrige el
alineamiento de las tarjetas de hardware, mejora la tarjeta "Próximamente" en el stack técnico, y
actualiza precios reales en el submenú de navegación.
Cambios principales:
ClientsMarqueecon logos de clientes animados (scroll infinito hacia la derecha) yfade lateral con overlays opacos
HardwareAlternatives(feature items conminHeight, "Casode uso" con
flex: 1y centrado)Stackcon chipsx86_64 / ???, borde punteado, animación pulse y estilomisterioso
Desde 15 UF/mes(Edge) yDesde 8 UF/mes(Cloud)main(github, instagram, linkedin, world) sobredevelopsinconflictos
developcombinando estructura dechips[]con lógicacoming: trueTipo de Cambio
Checklist del Desarrollador
src/components/Screenshots / Demo
Notas Adicionales
ClientsMarqueecorresponden a los SVGsMesa_de_trabajo_3–6.svg—pendiente confirmar nombre real de cada empresa para actualizar el array
CLIENTSdevelop(PR Fix/UI improvements #55) con resolución de conflictos enlanding.tsxy`Stack.tsx