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.

Sign up for GitHub

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

Jump to bottom

Fixed issue with considering breadcrumbs on failing pages - revisited #32

Open

JPTouron wants to merge 5 commits into thelarz:master from JPTouron:master

JPTouron commented Jan 11, 2018

Overridden OnResultExecuted on BreadCrumbAttribute class to remove the crumb when the action result fails
also modified BreadCrumb class DisplayRaw method to display the breadcrumbs with the same logic as Display method (setting the non-current crumbs as links) and also allowed the concatenator ( '>' ) to be parameterized, defaulting to '>'

JPTouron and others added 5 commits

June 30, 2017 09:45


          - Overridden OnResultExecuted on BreadCrumbAttribute class to remove …

…the crumb when the action result fails

- Cleaned up code


          Merge branch 'master' into master

4386a59


          merged with master from TheLarz/MvcBreadCrumbs

4db7bd0


          re-implemented changes and resolved all conflicts on PR:

c3e71e4

Fixed issue with considering breadcrumbs on failing pages
thelarz#28


          cleaned up all code using codemaid

3186d04

Author

JPTouron commented Jan 11, 2018

Note: this is a revisit for accidentally closed PR:
#28

please, don't close this one! @thelarz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet