Skip to content

Top navigation bar should expand with content.#7850

Merged
thomasneirynck merged 1 commit intoelastic:masterfrom
thomasneirynck:fix/7808
Jul 26, 2016
Merged

Top navigation bar should expand with content.#7850
thomasneirynck merged 1 commit intoelastic:masterfrom
thomasneirynck:fix/7808

Conversation

@thomasneirynck
Copy link
Contributor

@thomasneirynck thomasneirynck commented Jul 26, 2016

Fix for #7808.

This change preserves the current behavior: the navigation bar continues to line up with the Kibana-logo, which was the original reason for the 70px height.

Note that the current highlight-style - which underlines the nav-links - does not look that great when the breadcrumbs stack vertically. This is however a larger styling issues that can be addressed separately.

When the breadcrumbs stack vertically the header should expand vertically
as well. Otherwise, the breadcrumbs float outside the header. This
rearrangement happens when the screen is minimized horizontally, and the breadcrumbs change from
horizontal to vertical layout.
@jbudz
Copy link
Contributor

jbudz commented Jul 26, 2016

LGTM

@cjcenizal
Copy link
Contributor

LGTM!

@thomasneirynck thomasneirynck merged commit ebf196b into elastic:master Jul 26, 2016
@thomasneirynck
Copy link
Contributor Author

this also addresses #7810.

@thomasneirynck thomasneirynck deleted the fix/7808 branch July 27, 2016 15:15
simianhacker pushed a commit to simianhacker/kibana that referenced this pull request Aug 23, 2016
simianhacker pushed a commit to simianhacker/kibana that referenced this pull request Aug 23, 2016
simianhacker pushed a commit to simianhacker/kibana that referenced this pull request Aug 23, 2016
Added percentage option to significant_terms agg	
Significant terms: add scriptable significance heuristic elastic#7850
Add `format` support for date range filter and queries elastic#7821
Add `min_score` parameter to function score query to only match docs
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
Top navigation bar should expand with content.

Former-commit-id: ebf196b
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.

4 participants