Adding filter to define which conditions to enable primary sidebar on.

[pixleight]

function display_sidebar() in app/helpers.php referred to a filter that did not exist.

[QWp6t]

Thanks for taking the time to submit a PR, but this was deliberately removed.

#1760

[pixleight]

@QWp6t was an equivalent replacement added that I've overlooked?

[QWp6t]

Nah. We just make the filter available for developers to do with as they please.

[rutger1140]

The code sample did help me out. I had to scroll back in the repo history to see how to use the filter. Maybe it's useful to include/document it somewhere?

[QWp6t]

It's in the Sage 9 docs.

https://github.com/roots/docs/blob/sage-9/sage/theme-sidebar.md#displaying-the-sidebar

[rutger1140]

👌 thanks!

[mikedoubintchik]

What if I want to show the sidebar on all pages using the default page template, but hide it on a custom page template? @QWp6t

[retlehs]

@allurewebsolutions you use use the filter — you can do whatever you want with it. please don't use github for personal support requests 👉 https://discourse.roots.io/