Skip to content
This repository was archived by the owner on Jul 9, 2020. It is now read-only.

[REVIEW] Add PageTitle component & upgrade theme to have title font styles#17

Open
monzey wants to merge 2 commits intorgsystemes:masterfrom
monzey:pagetitle
Open

[REVIEW] Add PageTitle component & upgrade theme to have title font styles#17
monzey wants to merge 2 commits intorgsystemes:masterfrom
monzey:pagetitle

Conversation

@monzey
Copy link
Contributor

@monzey monzey commented Nov 20, 2019

No description provided.

const lineHeights = [0.25, 1, 1.1, 1.25, 1.75]
lineHeights.heading = lineHeights[3]
lineHeights.body = lineHeights[4]
lineHeights.title = lineHeights[2]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
lineHeights.title = lineHeights[2]
lineHeights.title = lineHeights[2]

}

export const pageTitle = () => (
<PageTitle>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe an example with an icon ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants