-
Notifications
You must be signed in to change notification settings - Fork 1
add: snippet which adds course search box to top on startpage #3
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.
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
Conversation
…dleMootDACH and amended by the Boost Union maintainers afterwards
8c3511f
to
483a5e8
Compare
Hi @Menrath , many thanks for your contribution. Regardless of the fact that I have just restructured this repo's directory structure before releasing it to the public which means that your PR should be rebased, I have had a look at your snippet in 6c4d17f. And I am wondering about the necessity of this snippet. Are you aware that you can influence this order on /admin/settings.php?section=frontpagesettings&theme=boost_union? With this setting, I can create an order like this: Isn't this what your snippet is doing as well? Or did I get it wrong? |
@abias Try and add some normal course content to the site 0 course! It will show above all the site home items. |
Finally rebased it correctly... |
@Menrath I got it. Thank you for the clarification. I see your goal now. I just reviewed the snippet and:
I will merge the PR now. |
Could be a snippet, could be a feature in boost_union :)