Child breadcrumb with empty path is ignored and replaces parent

[chad-smith]

I have found some odd behaviour when defining empty paths, it's quite fiddly to explain in text, so I have altered the demo site to reproduce my findings.

If you apply the changes in chad-smith/xng-breadcrumb@master...demo-data-bug you will find that neither of the breadcrumbs being set are used.

After a bit of poking around, I found the following:

1. Removing the root data: { breadcrumb: ...} makes no difference
2. Removing the module data: { breadcrumb: ...} resolves the issue
3. Removing the breadcrumb label but keeping the data: {}_ in the module causes the issue!

Scenario 3 is why the project I'm using breadcrumbs on has been seeing the problem (we use the data attributes frequently for permissions).

NB: If you deviate from an empty path, you will see both breadcrumbs as expected.

I thought this might be related to #25 because using the 'hack' that was described there also seems to work.

[chad-smith]

To make this a little more visual, with the code changes referenced, I see this:

And I am expecting to see this:

Whilst prepping those screenshots, I noticed that whilst I could get it to display Mentee (Root), I couldn't get it to display Mentee (Module) (the above screenshot is faked).

Either value would probably work for us, but it seems logical that both should (with the child taking precedence)

[chad-smith]

Also, sorry for keep finding problems 😄

If it's any consolation, I'm invested in flushing them all out because it's a library I intend to use in multiple projects and recommend to others 👍

[udayvunnam]

@chad-smith I really appreciate you for finding and reporting these subtle bugs - I have fixed this and also covered it in a test so that it won't break again. Fix available with version 6.2.0 🎉

https://github.com/udayvunnam/xng-breadcrumb/releases/tag/v6.2.0

[nativeforest]

i having a issue, when a wrapp xng-breadcrumb inside another component(app-custom-breadcrumb) and and this componet is used with a ngIf sentence(to know device resolution), its seems got a empty route

[udayvunnam]

@nativeforest your issue is fixed with xng-breadcrumb@6.8.1