Skip to content

Conversation

@ryangwilson
Copy link

@ryangwilson ryangwilson commented Apr 18, 2019

Work in Progress
I'm passing this off to @chasenlehara to:

Chasen tasks:

  • Current version needs to be shown next to the dropdown near the logo
  • Header: links need to be wired up to add/remove class "current" to the correct active page
  • Hide #left at tablet/desktop sizes on pages that don’t have sidebar content (homepage & community)
  • Dismiss the search results popover when a result is clicked
  • Clicking outside the search results should dismiss them
  • Show the search results if the search field is given focus while it has text in it
  • Search results shouldn’t overlap with the survey ad

Ryan tasks:

  • left sidebar: just show subpages related to the page you are on
  • class="current" is hard coded in nav-links.mustache atm
  • Fix spacing around the version selector dropdown
  • Fix search results on mobile

Ryan Wilson and others added 25 commits April 18, 2019 09:12
…ontent

This is unfortunately a hack for now. The homepage and community page are hard-coded to not show the sidebar.
- Dismiss the search results popover when a result is clicked
- Dismiss the search results when clicking outside them
- Show the search results if the search field is given focus while it has text in it
- Make sure the search results don’t overlap with the survey ad
- Remove the brand’s unnecessary white background that was visible in Safari
- Fix the background of the search results in Safari
- Tone down the drop shadow on the search results
They broke after updating the docMap.
Node 12 fails with the error “ReferenceError: primordials is not defined”
@chasenlehara chasenlehara merged commit 9456eb4 into master Apr 23, 2019
@chasenlehara chasenlehara deleted the i4744 branch April 23, 2019 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants