Skip to content

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

Merged
merged 10 commits into from
Jul 21, 2025

Conversation

Menrath
Copy link
Collaborator

@Menrath Menrath commented Jul 1, 2025

Could be a snippet, could be a feature in boost_union :)

…dleMootDACH and amended by the Boost Union maintainers afterwards
@abias abias force-pushed the main branch 2 times, most recently from 8c3511f to 483a5e8 Compare July 3, 2025 10:50
@abias
Copy link
Member

abias commented Jul 3, 2025

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.
If I understand it correctly, this is about Moodle's site home and the order of the elements which you see there.

Are you aware that you can influence this order on /admin/settings.php?section=frontpagesettings&theme=boost_union?
grafik

With this setting, I can create an order like this:
grafik

Isn't this what your snippet is doing as well? Or did I get it wrong?

@Menrath
Copy link
Collaborator Author

Menrath commented Jul 3, 2025

@abias Try and add some normal course content to the site 0 course! It will show above all the site home items.

@Menrath
Copy link
Collaborator Author

Menrath commented Jul 4, 2025

Finally rebased it correctly...

@abias
Copy link
Member

abias commented Jul 21, 2025

@Menrath I got it. Thank you for the clarification. I see your goal now.

I just reviewed the snippet and:

  • introduced a new "site home" scope to Boost Union as this snippet is not really targeted to the dashboard
  • Added a screenshot
  • Modified the quite complicated (and also somehow fragile) selector for the course search box to a selector containing :has(.simplesearchform)

I will merge the PR now.

@abias abias merged commit 44f323d into moodle-an-hochschulen:main Jul 21, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants