-
Notifications
You must be signed in to change notification settings - Fork 67
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
Customization: Add instructions for changing search button color #606
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this, can be merged after adjustments.
To change the orange search button color, in your instance create `assets/less/site/elements/button.variables` file instead of the globals folder and add the following line to it: | ||
|
||
```less | ||
@searchButtonColor: /* your desired color */; | ||
``` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting that this is still needed. I have a note in my codebase saying that having to use button.variables
could be deleted in v11. Maybe it has to do with something else on our side...
In any case, let's move this below the next paragraph with a little connection so as to get:
---start---
You can override any styling variable in your site.variables
file. The available styling variables are typically found in the .variables
files of the various Invenio modules installed (see note below).
For instance, to change the search button color, create the assets/less/site/elements/button.variables
file (notice the use of the elements
folder instead of the globals
one) and add the following line to it:
@searchButtonColor: /* your desired color */;
---end---
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change is implemented thanks for review @fenekku
I also explored the issue at InvenioRDM #1642 without success, yet the current approach seems effective. Let's proceed with this solution.
b0aa16c
to
299b891
Compare
❤️ Thank you for your contribution!
Description
Please describe briefly your pull request.
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:
Third-party code
If you've added third-party code (copy/pasted or new dependencies), please reach out to an architect.
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that: