The padding prop seems to have no effect with PageLayout.Sidebar
Los mantenedores suelen responder en 2 días
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- react, typescript
- Área
- frontend
Línea de trabajo
Comienza con los puntos de entrada PageLayout.Sidebar y PageLayout.Header y reproduce el ejemplo de TypeScript proporcionado. Compara cómo aplica cada componente su prop padding cuando el componente padre usa padding="none"; se considera terminado cuando Sidebar padding="condensed" tiene efecto visible, igual que Header padding.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
It seems like padding="condensed" on PageLayout.Sidebar does not have an effect when used within PageLayout with padding="none". Other sub-components do seem to have an effect. For example:
import {PageLayout} from '@primer/react'
export function BenchmarksClientPage({children}: {children: React.ReactNode}) {
return (
<PageLayout padding="none">
<PageLayout.Header padding="condensed">
header
</PageLayout.Header>
<PageLayout.Sidebar divider="line" sticky padding="condensed">
sidebar
</PageLayout.Sidebar>
<PageLayout.Content>content</PageLayout.Content>
</PageLayout>
)
}
In this snippet, PageLayout.Sidebar would have no padding even though padding="condensed" is set where-as PageLayout.Header does have padding set
- Lenguaje dominante
- TypeScript
- Estrellas
- 3.9k
- Forks
- 697
- Merge medio
- 2 d 17 h
- PR fusionados (30 d)
- 37
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de primer/react
-
ActionMenu: add preventFocusOnClosePosiblemente ocupada @tay1orjones la tomó hace 7 días. Abiertocomponent: ActionMenu
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
primer/react#8488 · 1 comentario · 1 reacción · 1 asignado ·
Los mantenedores suelen responder en 2 días
-
FormControl renders duplicate non-choice inputs when layout="horizontal"Posiblemente ocupada @namangoyal176-10 la tomó hace 7 días. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 2 días
-
PageHeader.Title does not seem to accept id propPosiblemente ocupada @minwookshin la tomó hace 12 días. Abiertocomponent: PageHeader react staff
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
primer/react#8474 · 1 comentario · 1 reacción · 1 asignado ·
Los mantenedores suelen responder en 2 días
-
bug component: Spinner react
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
primer/react#8423 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Low contrast in SelectAbiertowaiting for author response
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
primer/react#8172 · 1 comentario ·
Los mantenedores suelen responder en 2 días
Todos los issues de primer/react
Issues similares
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 70/100
siyuan-note/siyuan#20517 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
NaturalIntelligence/fast-xml-parser#890 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
zsviczian/obsidian-excalidraw-plugin#3000 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
pnpm/get.pnpm.io#68 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
nuxt-modules/better-auth#578 · 2 comentarios ·
Los mantenedores suelen responder en 1 día