Fix Preview URL: Suppress breadcrumbs#12111
Conversation
qqmyers
left a comment
There was a problem hiding this comment.
Didn't test but I see that the header responds to this param and it was not set for the dataset page and was for the file page.
I'll approve - only question is whether it should have a one line release note.
|
@qqmyers I can add a release note, but I'm going to advocate for a patch since it's just one line in an xhtml file and a user noticed it. |
I'm happy to both test/merge, and to patch it in our own prod. But I agree with @qqmyers that it would be worth mentioning in a one-sentence release note as well, for everybody else. The bug seems important enough. |
|
@sekmiller |
What this PR does / why we need it: Fixes an bug introduced after preview url update
Which issue(s) this PR closes:
Special notes for your reviewer:
Suggestions on how to test this: remember, that if you are logged in with privileges the breadcrumbs will show.
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
Is there a release notes update needed for this change?:
Additional documentation: