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

ncyBreadcrumbLabel can be build by text or function #136

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

ncyBreadcrumbLabel can be build by text or function #136

wants to merge 1 commit into from

Conversation

Krolock1
Copy link

@Krolock1 Krolock1 commented Mar 3, 2016

No description provided.

@Krolock1
Copy link
Author

Krolock1 commented Mar 4, 2016

with this extention it is possible to define the label of the breadcrumb as string or as callback function that is called with with viewScope as parameter.
The behaviour is similar to the 'parent' parameter

@ncuillery
Copy link
Owner

Hi, many thanks for the PR.

At first glance, it looks good to me. I can't merge it for the moment: I need to fix tests (#135) before.

Then I'll test this stuff.

@ravikiranr26
Copy link

Hi, Is this updated to the plugin ? can we use label function like parent ?

@ncuillery
Copy link
Owner

Hi @Krolock1, can you rebase your PR please ? Tests should be working.

@Krolock1
Copy link
Author

It's my first time that I will do a rebase, so I have to look how to do.
Since the week is very busy I will do it next week

@stramel
Copy link

stramel commented Sep 2, 2016

Seems related to #105 #146

@javoire
Copy link

javoire commented Sep 21, 2016

I like this, it looks good to me too. Would be awesome if it got merged.

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.

None yet

5 participants