-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[Enterprise Search] Add solution-level side navigation #74705
Merged
Merged
Commits on Aug 11, 2020
-
Add basic layout/sidebar blocking
- note: we should *not* set left: 0 / top: 0 etc., as this can interfere with Kibana's existing UI (e.g. docked navigation, telemetry callout)
Configuration menu - View commit details
-
Copy full SHA for 4f6cb95 - Browse repository at this point
Copy the full SHA 4f6cb95View commit details -
Configuration menu - View commit details
-
Copy full SHA for b9b38ad - Browse repository at this point
Copy the full SHA b9b38adView commit details -
Refactor SideNav to be a reusable component
- So that Workplace Search can reuse the same layout but pass in their own custom nav + Refactor AppSearch to use Layout in router
Configuration menu - View commit details
-
Copy full SHA for cee4072 - Browse repository at this point
Copy the full SHA cee4072View commit details -
Configuration menu - View commit details
-
Copy full SHA for dd5ec40 - Browse repository at this point
Copy the full SHA dd5ec40View commit details -
Fix root redirects not working as expected
- If enterpriseSearchUrl hasn't been set, all pages should redirect to SetupGuide, not just root - The engines redirect simply wasn't working at all - it would always show a blank page when '/' was clicked in the Kibana breadcrumbs. Not sure if this is a Kibana issue - had to change to a component load to fix + Simplify index.test.tsx (probably unreasonable and not super helpful to add assertions for each new route)
Configuration menu - View commit details
-
Copy full SHA for 4743611 - Browse repository at this point
Copy the full SHA 4743611View commit details -
Configuration menu - View commit details
-
Copy full SHA for f1206c2 - Browse repository at this point
Copy the full SHA f1206c2View commit details -
- By adding a new useLocation mock + add SideNavLink active class test TODO: I should probably rename react_router_history.mock to just react_router.mock
Configuration menu - View commit details
-
Copy full SHA for c3e6cc1 - Browse repository at this point
Copy the full SHA c3e6cc1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3a5e7fb - Browse repository at this point
Copy the full SHA 3a5e7fbView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9f371ad - Browse repository at this point
Copy the full SHA 9f371adView commit details -
[Feedback] Update mobile UX to close menu on link click/navigation
- This requires updating our EUI/React Router components to accept and run custom onClick events - Also requires adding a new ReactContext to pass down closeNavigation, but that's not too onerous thanks to useContext
Configuration menu - View commit details
-
Copy full SHA for e63d2b2 - Browse repository at this point
Copy the full SHA e63d2b2View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.