search

buildo / react-components

Collection of general React components used in buildo projects.
http://react-components.buildo.io/
MIT License
157 stars 21 forks source link

Panel guidelines #1224

Closed FrancescoCioria closed 6 years ago

alessandravilla commented 6 years ago

Perchè non stiamo seguendo la IA che avevamo definito per queste guidelines? https://paper.dropbox.com/doc/Panels-58BRPZBY0ORaPArJRYwCs screen shot 2018-04-06 at 11 35 16 le sezioni Floating,Docked,Collapsible,Tabbed Panel se considererei come sottosezioni di "Panel attributes", aggiungerei anche la sezione "with header" (dark e light sono attributi di with header). Anche "tabbed panel" è un attributo di header

FrancescoCioria commented 6 years ago

@alessandravilla

Perchè non stiamo seguendo la IA che avevamo definito per queste guidelines?

perchè è diversa dagli altri componenti e non ne vedo una ragione: sia in button che in dropdown parliamo di types/varianti, perchè qui no? (visto che la prop effettivamente si chiama type?) Inoltre non vedo una ragione nell'elencare tutti i casi di docked e collapsible: sono tutti la stessa cosa in direzione diversa, mi sembra chiaro dalle guidelines, no?

Anche "tabbed panel" è un attributo di header

lo è? nella nostra implementazione è addirittura un altro componente

alessandravilla commented 6 years ago

Non dobbiamo specificare tutti i casi, però per come è formattato il testo sembra che il pannello abbia due types (floating,docked), mentre non è chiaro cosa siano Dark,Collapsible,Tabbed Panel. Sono dei tipi di pannello o sono delle varianti?

nemobot commented 6 years ago