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

WEB-883 update ja copy #8886

Merged
merged 2 commits into from
Oct 23, 2020
Merged

WEB-883 update ja copy #8886

merged 2 commits into from
Oct 23, 2020

Conversation

zbayoff
Copy link
Contributor

@zbayoff zbayoff commented Oct 22, 2020

@zbayoff zbayoff requested review from a team as code owners October 22, 2020 15:28
@github-actions github-actions bot added the Architecture Everything related to the Doc backend label Oct 22, 2020
@zbayoff zbayoff added Do Not Merge Just do not merge this PR :) and removed Architecture Everything related to the Doc backend labels Oct 22, 2020
@nsollecito
Copy link
Collaborator

@zbayoff Looks like the Solutions drop-down is blank.. can you take a look?

@zbayoff
Copy link
Contributor Author

zbayoff commented Oct 22, 2020

@nsollecito sorry, can you post a screen video? I don't see the issue.

@nsollecito
Copy link
Collaborator

@nsollecito
Copy link
Collaborator

this is also happening - https://share.getcloudapp.com/4guJKgb7#

@github-actions github-actions bot added the Architecture Everything related to the Doc backend label Oct 22, 2020
@zbayoff
Copy link
Contributor Author

zbayoff commented Oct 22, 2020

@nsollecito I've fixed the solutions menu. I dont see the issue with the white space under the footer though.

@nsollecito
Copy link
Collaborator

@zbayoff the CTA is still pushing to a second line in the nav and overlapping the body but it's like that on live currently too

@zbayoff
Copy link
Contributor Author

zbayoff commented Oct 23, 2020

@nsollecito yeah this is a known issue, a limitation due to our design. I can try and address in a separate card and get guidance from Brand Design.

@zbayoff zbayoff removed the Do Not Merge Just do not merge this PR :) label Oct 23, 2020
@apigirl apigirl merged commit eb8c57d into master Oct 23, 2020
@apigirl apigirl deleted the zach/update-footer-copy branch October 23, 2020 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Architecture Everything related to the Doc backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants