You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
River.Content does not accept a Heading with as="h2". River automatically applies as="h3" by default.
Using River and FAQ automatically uses h3 headings. Semantically, that doesn't always make sense, e.g. on https://accelerator.github.com the heading structure now goes from h1 to h3. Would be great to be able to customize the heading to h2 etc. when needed.
Actually, I would expect any component that uses Heading to be able to accept any parameters that Heading accepts.
Yes, for some context the reason we default to h3 is to help enforce content order, whereby River should only ever appear beneath a SectionIntro, which uses a h2. Appreciate in hindsight how limiting this can be, so I would propose that we resolve this by maintaining the default, but forwarding an override.
Thanks for the context. The way I see it we should enforce content order through the templates we build, not the components themselves, otherwise we create dependencies that don't always make sense. Example, SectionIntro doesn't exist yet and might not be required in each context.
To think about it differently, we can offer different "levels" of components with increased control, e.g
Heading and Text basic building blocks to do anything.
River and SectionIntro components consisting of the above, but free to use independently.
RiverSection, a "section component" (words are hard 😄) consisting of the above and enforcing a certain order.
Think Brick (Primitives) -> House (Components) -> Neighborhood (Sections) -> City (Page template).
Using
River
andFAQ
automatically usesh3
headings. Semantically, that doesn't always make sense, e.g. on https://accelerator.github.com the heading structure now goes fromh1
toh3
. Would be great to be able to customize the heading toh2
etc. when needed.Actually, I would expect any component that uses
Heading
to be able to accept any parameters thatHeading
accepts.cc @aguevara23
The text was updated successfully, but these errors were encountered: