Fix title inconsistencies #229
|
@ -3,9 +3,9 @@
|
||||||
margin-bottom: calc(60rem / 16);
|
margin-bottom: calc(60rem / 16);
|
||||||
}
|
}
|
||||||
|
|
||||||
.main > h2 {
|
.main > h1, .main > section > h1 {
|
||||||
padding-bottom: 1rem;
|
padding-bottom: calc(16rem / 16);
|
||||||
border-bottom: 1px solid var(--primary-heading);
|
border-bottom: calc(1rem / 16) solid var(--primary-heading);
|
||||||
}
|
}
|
||||||
|
|
||||||
.header {
|
.header {
|
||||||
|
|
|
@ -67,8 +67,8 @@ export default function Term(props: Props) {
|
||||||
<Link href="/events/archive">Archive</Link>
|
<Link href="/events/archive">Archive</Link>
|
||||||
</div>
|
</div>
|
||||||
{hasFutureEvents && (
|
{hasFutureEvents && (
|
||||||
|
|||||||
<>
|
<section>
|
||||||
<h2>Upcoming Events</h2>
|
<h1>Upcoming Events</h1>
|
||||||
<div className={styles.miniEventCards}>
|
<div className={styles.miniEventCards}>
|
||||||
{props.futureEvents.map(({ content, metadata }) => (
|
{props.futureEvents.map(({ content, metadata }) => (
|
||||||
<EventCard
|
<EventCard
|
||||||
|
@ -81,20 +81,34 @@ export default function Term(props: Props) {
|
||||||
</EventCard>
|
</EventCard>
|
||||||
))}
|
))}
|
||||||
</div>
|
</div>
|
||||||
</>
|
</section>
|
||||||
|
)}
|
||||||
|
{hasPastEvents && props.isCurrentTerm && (
|
||||||
|
<section>
|
||||||
|
<h1>Past Events</h1>
|
||||||
|
<div className={styles.miniEventCards}>
|
||||||
|
{props.pastEvents.map(({ content, metadata }) => (
|
||||||
|
<MiniEventCard
|
||||||
|
{...metadata}
|
||||||
|
date={new Date(metadata.date)}
|
||||||
|
description={<MDXRemote {...content} />}
|
||||||
|
key={metadata.name + metadata.date.toString()}
|
||||||
|
/>
|
||||||
|
))}
|
||||||
|
</div>
|
||||||
|
</section>
|
||||||
)}
|
)}
|
||||||
{hasPastEvents && props.isCurrentTerm && <h2>Past Events</h2>}
|
|
||||||
{hasPastEvents && !props.isCurrentTerm && (
|
{hasPastEvents && !props.isCurrentTerm && (
|
||||||
<h2>
|
<h1>
|
||||||
Events Archive:
|
Events Archive:
|
||||||
<span className={styles.blue}>
|
<span className={styles.blue}>
|
||||||
{` ${capitalize(props.term)} ${props.year}`}
|
{` ${capitalize(props.term)} ${props.year}`}
|
||||||
</span>
|
</span>
|
||||||
</h2>
|
</h1>
|
||||||
)}
|
)}
|
||||||
{!hasFutureEvents && !hasPastEvents && (
|
{!hasFutureEvents && !hasPastEvents && (
|
||||||
<>
|
<>
|
||||||
<h2>Events</h2>
|
<h1>Events</h1>
|
||||||
There are no upcoming or past events for the{" "}
|
There are no upcoming or past events for the{" "}
|
||||||
{`${capitalize(props.term)} ${props.year}`} term. Please check back
|
{`${capitalize(props.term)} ${props.year}`} term. Please check back
|
||||||
later!
|
later!
|
||||||
|
|
|
@ -2,9 +2,9 @@
|
||||||
margin-top: calc(60rem / 16);
|
margin-top: calc(60rem / 16);
|
||||||
}
|
}
|
||||||
|
|
||||||
.main > h2 {
|
.main > h1 {
|
||||||
padding-bottom: calc(1rem / 16);
|
padding-bottom: calc(16rem / 16);
|
||||||
border-bottom: 1px solid var(--primary-heading);
|
border-bottom: calc(1rem / 16) solid var(--primary-heading);
|
||||||
}
|
}
|
||||||
|
|
||||||
.blue {
|
.blue {
|
||||||
|
|
|
@ -2,6 +2,11 @@
|
||||||
padding-bottom: calc(30rem / 16);
|
padding-bottom: calc(30rem / 16);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
.page > h1 {
|
||||||
|
padding-bottom: calc(16rem / 16);
|
||||||
|
border-bottom: calc(1rem / 16) solid var(--primary-heading);
|
||||||
j285he marked this conversation as resolved
Outdated
a3thakra
commented
calc(1rem / 16) solid ... calc(1rem / 16) solid ...
|
|||||||
|
}
|
||||||
|
|
||||||
.term {
|
.term {
|
||||||
color: var(--primary-accent);
|
color: var(--primary-accent);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue
actually can you put this in a section tag? technically, there are two h1's on the page. But a page cannot have 2 h1s. If we look closely, we have two sections on this page, each with it's own h1. I think that would make this nicer. (in terms of semantics)
I currently have Past Events as an h2. Should I make it an h1?
Yes, semantically speaking, each page should have exactly one h1, and then 0 or more h2's
when i say h1, i mean the h1 tag. you can style it as you wish.
Ah okay. Also for the original comment, I don't see two h1's on the same page. It's just either
Upcoming Events
orEvents Archive:
. Am I going crazy?"Upcoming events" and "Past events" should both be
h1
s enclosed within asection
. So the final markup would look something like this:Ahhhhhh sorry I see what you mean, thanks!
Although what are your thoughts on the styling? I think having h2 on Past Events makes the user more drawn towards current upcoming events, instead of having two blocky h1's competing for attention, but idk
As I said earlier, they should be h1 tags, you can style them like h2s to match figma 😉
OK, thank you! Since Figma's title sizes differ a bit from ours, let me know if I should change the size of Past Events or not