New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/future header #78
Conversation
be9e4fe
to
25e6a54
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, awesome work
@@ -0,0 +1,140 @@ | |||
import React, { Fragment, useEffect, useState } from 'react' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file was mostly just moved from react
to react/legacy
, please refrain from delving too deeply into it for this PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job 🚀
react/components/Row.js
Outdated
|
||
return ( | ||
<div | ||
className={`w-100 top-0 ${sticky ? 'z-999' : '' } ${inverted ? 'bg-base--inverted c-on-base--inverted' : 'bg-base c-on-base'}`} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could use classNames
here
Co-Authored-By: lbebber <lucasbbebber@gmail.com>
What is the purpose of this pull request?
First, unstable implementation of the new composable header
Also adds stubs for some components which will be further developed in the future.
Also adds support for the
unstable--menu
block on the now legacy header.What problem is this solving?
How should this be manually tested?
Screenshots or example usage
Types of changes