Skip to content
This repository has been archived by the owner. It is now read-only.

Refactor header #38

Merged
merged 5 commits into from Sep 12, 2017
Merged

Refactor header #38

merged 5 commits into from Sep 12, 2017

Conversation

@c-w
Copy link
Contributor

@c-w c-w commented Sep 12, 2017

Just a bit of refactoring to make the header easier to work with:

  • Make spacing consistent
  • Extract some methods
  • Split into multiple components

No functional change, dashboard still loads as before:

image

c-w added 2 commits Sep 12, 2017
- Make spacing consistent
- Expose high-level structure by extracting methods
@c-w c-w requested a review from Smarker Sep 12, 2017
Copy link
Contributor

@Smarker Smarker left a comment

LGTM

@c-w c-w merged commit bcaefc9 into master Sep 12, 2017
@c-w c-w deleted the refactor-header branch Sep 12, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants