Add events archive #187

Merged
a3thakra merged 2 commits from adi-events-archive into main 2021-08-28 15:52:46 -04:00
Owner

Closes #117

Closes #117
a3thakra requested review from j285he 2021-08-27 20:03:05 -04:00
j285he approved these changes 2021-08-27 22:01:30 -04:00
j285he left a comment
Member

Looks great!

Looks great!
@ -0,0 +1,47 @@
import React from "react";
Member

After the title, there's a missing border-bottom as per Figma

After the title, there's a missing border-bottom as per Figma
Member

Also inspect element says the title is h1, should be h2

Also inspect element says the title is h1, should be h2
Author
Owner

It should be h1 because it's the only heading of the page, so semantically it makes sense for it to be h1

It should be h1 because it's the only heading of the page, so semantically it makes sense for it to be h1
a3thakra marked this conversation as resolved
@ -0,0 +1,47 @@
import React from "react";
Member

We have been using 60px for margin-bottom in [term], but I see 40px for this component. Maybe we change one or the other for consistency?

We have been using 60px for margin-bottom in [term], but I see 40px for this component. Maybe we change one or the other for consistency?
Author
Owner

The line height is a lot more (3 instead of 1.5ish) than other sections, so imo 40px actually looks a lot like 60px. I tried to make it 60px and that was a lot.

The line height is a lot more (3 instead of 1.5ish) than other sections, so imo 40px actually looks a lot like 60px. I tried to make it 60px and that was a lot.
a3thakra marked this conversation as resolved
@ -0,0 +42,4 @@
: defaultGetShapesConfig(width, height);
}) as GetShapesConfig;
function capitalize(str: string) {
Member

We're using this function a lot. Maybe it's worth putting it into lib?

We're using this function a lot. Maybe it's worth putting it into lib?
a3thakra marked this conversation as resolved
a3thakra force-pushed adi-events-archive from 94c44b112d to 98717fafbc 2021-08-28 15:42:00 -04:00 Compare
a3thakra merged commit c31571f745 into main 2021-08-28 15:52:46 -04:00
a3thakra referenced this issue from a commit 2021-08-28 15:52:46 -04:00
a3thakra deleted branch adi-events-archive 2021-08-28 15:52:54 -04:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: www/www-new#187
No description provided.