Skip to content
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

Polish navigation #215

Merged
merged 2 commits into from
Jan 22, 2019
Merged

Polish navigation #215

merged 2 commits into from
Jan 22, 2019

Conversation

damassi
Copy link
Member

@damassi damassi commented Jan 22, 2019

@zephraph @orta - Ok Palette is basically done! 🎉Just need to add content styles as we start adding content. I dropped a message in #design-channel with an update.

nav

Remaining items:

  • Design polish with design team
  • Code cleanup
  • Get apollo-cli running to generate typescript graphql types
  • Do a prod build test run
  • Update CI
  • Organize the content better. Some of the items seem a bit unintuitive from a dev stand point.

Other than ^ and likely some things i'm forgetting its pretty much a wrap.

@damassi damassi self-assigned this Jan 22, 2019
@damassi damassi force-pushed the more-polish branch 2 times, most recently from 1d26cbd to 695de78 Compare January 22, 2019 03:37
@@ -79,6 +80,7 @@
"@babel/core": "^7.2.2",
"@mdx-js/mdx": "^0.16.6",
"@mdx-js/tag": "^0.16.6",
"@reach/router": "^1.2.1",
Copy link
Member Author

Choose a reason for hiding this comment

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

Gatsby uses @reach/router under the hood so we can tap into all of its features if need be.

@artsyit
Copy link
Contributor

artsyit commented Jan 22, 2019

Deploy preview for artsy-palette ready!

Built with commit 97d381a

https://deploy-preview-215--artsy-palette.netlify.com

const NavLink = styled(NavLinkWrapper)`
cursor: pointer;

&&.isActive {
Copy link
Member Author

Choose a reason for hiding this comment

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

&& in styled-components forces higher specificity.

@artsyit
Copy link
Contributor

artsyit commented Jan 22, 2019

Deploy preview for artsy-palette ready!

Built with commit 695de78

https://deploy-preview-215--artsy-palette.netlify.com

Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor
incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis
nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat.

Copy link
Member Author

@damassi damassi Jan 22, 2019

Choose a reason for hiding this comment

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

In terms of MDX, note that i'm interpolating markdown inside of of a JSX <Tab> component; however, the markdown has to be left justified for it to behave properly.

Being able to mix and match like this is so totally 👌 -- JSX as more sane version of short-codes.

Good idea, mdx.

@artsyit
Copy link
Contributor

artsyit commented Jan 22, 2019

Deploy preview for artsy-palette ready!

Built with commit a5f3622

https://deploy-preview-215--artsy-palette.netlify.com

@@ -1,8 +1,8 @@
---
name: Utilities
order: 5
order: 9
Copy link
Member Author

Choose a reason for hiding this comment

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

I can't think of a more intelligent way to page order than this. Anyone have any suggestions? This is pretty simple / straightforward but seems like there might be a better way.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, this is how I always end up doing it TBH

@orta
Copy link
Contributor

orta commented Jan 22, 2019

looking way slicker 👍

@damassi
Copy link
Member Author

damassi commented Jan 22, 2019

Thinking we can probably hook something like https://getgrav.org up for editing the docs. There are so many options these days....

@damassi damassi merged commit 9922e0f into artsy:master Jan 22, 2019
@damassi damassi deleted the more-polish branch January 22, 2019 04:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants