Skip to content

Add breadcrumb-page-title to management-app - #12765

Merged
w33ble merged 2 commits into
elastic:masterfrom
BigFunger:add-breadcrumb-page-title
Jul 11, 2017
Merged

w33ble merged 2 commits into
elastic:masterfrom
BigFunger:add-breadcrumb-page-title

Conversation

@BigFunger

@BigFunger BigFunger commented Jul 11, 2017 •

Copy link
Copy Markdown
Contributor

Summary: The bread-crumbs component allows for the consumer to pass in a page-title value which is appended as the final breadcrumb in the chain. This PR adds a scope argument to the management-app to take advantage of it.

Changes: Adds a breadcrumbPageTitle scope argument to the kbn-management-app directive with two-way binding, and passes the value along to the bread-crumbs directive.

sectionName: '@section',
omitPages: '@omitBreadcrumbPages'
omitPages: '@omitBreadcrumbPages',
breadcrumbPageTitle: '='

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd probably just call this page-title, since the consumer shouldn't care where the title is used, and we could move that title out of the breadcrumb in the future.

@w33ble w33ble left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small comment, but this LGTM

@chrisronline chrisronline left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@w33ble
w33ble merged commit e951369 into elastic:master Jul 11, 2017
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
* Adds breadcrumb-page-title to management-app

* Changes argument name to page-title
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