Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

Add brand colors and xlarge padding token and increase logo clearspace #791

Merged
merged 1 commit into from
Jun 16, 2020

Conversation

LukasHirt
Copy link
Contributor

@LukasHirt LukasHirt commented Jun 15, 2020

Bring oC CI into the default theme, improve buttons design and improve the size of the logo in the sidebar. This PR contains quite a few changes but they're quite small in size so I merged them into one PR.

Related issues:

@LukasHirt LukasHirt self-assigned this Jun 15, 2020
@LukasHirt LukasHirt marked this pull request as ready for review June 15, 2020 09:45
@LukasHirt LukasHirt requested a review from kulmann June 15, 2020 09:45
@LukasHirt LukasHirt force-pushed the oc-branding branch 2 times, most recently from 03ad9a7 to 967b886 Compare June 15, 2020 09:54
@LukasHirt LukasHirt mentioned this pull request Jun 15, 2020
kulmann
kulmann previously approved these changes Jun 16, 2020
Copy link
Member

@kulmann kulmann left a comment

Choose a reason for hiding this comment

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

I have two things that I don't fully agree with - but that should be part of further discussion and/or user testing, not of this PR. Will post them separately.
This PR is pure gold. I linked phoenix to this ODS PR and compared the new look with the live demo instance. I love it - most of all the overhauled buttons. ❤️

@kulmann
Copy link
Member

kulmann commented Jun 16, 2020

Turns out that one of my two concerns is actually a small bug: #796
Please decide if you want to fix that in a separate PR

This is my other concern: #795
Should be evaluated through user feedback.

Added brand colors and xlarge padding token and increase clearspace around logo.
Use Source Sans Pro as a global font and redesign buttons.
@LukasHirt
Copy link
Contributor Author

LukasHirt commented Jun 16, 2020

Different buttons size fixed by using the transparent border in primary, secondary and danger button (falling back to default in UIkit).

@LukasHirt LukasHirt merged commit 98e4933 into master Jun 16, 2020
@LukasHirt LukasHirt deleted the oc-branding branch June 16, 2020 18:29
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.

Primary button is one pixel smaller in height Use oC CI font Improve buttons design Button padding
3 participants