Skip to content
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

Rework notice boxes #455

Closed
wants to merge 4 commits into from
Closed

Conversation

McShelby
Copy link

This patch includes the following:

After rework, it is now easier for theme users to overwrite just the colors and icons of the notice boxes in their own CSS.

- translatable labels (#334, #128)
- multiple blocks in one box (#402, #152)
- usage of distinct icons for each type
- switching colors for info and note
for users who may want to put the borders to the left of the box
without redefining the colors
@dahlsailrunner
Copy link

Is there a timing on getting this change merged? I got here by trying to figure out how to to more than one paragraph in a notice and it seems like this merge will be the answer. Thanks in advance. :)

@m-czernek
Copy link

+1, notice shortcode is not usable for multiple lines.

@dahlsailrunner
Copy link

For anyone else looking to create note content that is more than one line (can include bulleted lists, etc), just take the changes in layouts/shortcodes/notice.html and static/css/theme.css and apply them to the corresponding files in your local project. Worked like a champ for me.

Good work and thank you for the change @McShelby ! :)

fcooper8472 added a commit to SABS-R3/ScientificComputingInMatlab that referenced this pull request Dec 1, 2020
@McShelby McShelby closed this Jul 16, 2021
@McShelby McShelby deleted the McShelby/issue334 branch July 16, 2021 22:15
maxatome pushed a commit to maxatome/hugo-theme-learn that referenced this pull request Jan 23, 2024
maxatome pushed a commit to maxatome/hugo-theme-learn that referenced this pull request Jan 23, 2024
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.

None yet

3 participants