search

kamranayub / CaliburnBindableAppBar

Bindable App Bar support for Caliburn.Micro and WP7.1+
MIT License
16 stars 10 forks source link

Set deferLoad to false on active items to load them now #18

Closed samuelens closed 10 years ago

samuelens commented 10 years ago

When using multiple pivots, when accessing the internal (not the main), the active item doesn't load it appbar since it is deferred from the first time. Then the active item should be updated.

kamranayub commented 10 years ago

Thanks, I think I see the change you made, the rest seems to be formatting braces. Would it be possible to undo the formatting, or separating it from the actual bug fix for a cleaner merge?

samuelens commented 10 years ago

Thanks for your quickly response. I have pushed and updated the pull request.

2014-04-07 17:59 GMT+03:00 Kamran Ayub notifications@github.com:

Thanks, I think I see the change you made, the rest seems to be formatting braces. Would it be possible to undo the formatting, or separating it from the actual bug fix for a cleaner merge?

Reply to this email directly or view it on GitHubhttps://github.com/kamranayub/CaliburnBindableAppBar/pull/18#issuecomment-39740789 .

kamranayub commented 10 years ago

Excellent thank you.

samuelens commented 10 years ago

Thanks !!

2014-04-07 18:19 GMT+03:00 Kamran Ayub notifications@github.com:

Merged #18 https://github.com/kamranayub/CaliburnBindableAppBar/pull/18.

Reply to this email directly or view it on GitHubhttps://github.com/kamranayub/CaliburnBindableAppBar/pull/18 .