-
Notifications
You must be signed in to change notification settings - Fork 67
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
Port all work after 3.2.0 into refactor #80
Comments
@bradfrost we'll have to methodically port this stuff over. I can take point on it, since it's mostly JS and not design related work |
bmuenzenmeyer
added a commit
that referenced
this issue
Oct 9, 2017
one more change to go |
bmuenzenmeyer
added a commit
that referenced
this issue
Oct 9, 2017
bmuenzenmeyer
added a commit
that referenced
this issue
Oct 9, 2017
What tag would fit this issue? |
bmuenzenmeyer
added a commit
that referenced
this issue
Oct 10, 2017
@bradfrost what? oh, label. um not needed. it's done |
closed with c1c8b01 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Building off of #72 (comment)
make sure tab is hiddenbrad has changed this logic so I think its okupdate copy of missing descriptionlikely not needed per Remove or Revise "There is no description for this pattern." Default #29fix checks for when view in new window menu item is hiddendo not see this codepath anymoreadd a conditional around "All" button in t op nav so it respect the JSON option to hide itshoulld be done proper with Request: Finer control over "view all" links #84The text was updated successfully, but these errors were encountered: