-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Made example contents working as expected and visible #575
base: main
Are you sure you want to change the base?
Conversation
* Change pygmentsStyle from bw to b2 to make it more readable in dark mode * Add ability to insert html inside of markdowns * `math: true` uses mathjax. So use `katex:true` instead
@@ -190,3 +190,9 @@ url = "projects/" | |||
name = "Contato" | |||
weight = 5 | |||
url = "contact/" | |||
|
|||
# Enable html support inside markdown | |||
[markup] |
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.
Everything looks fine. I'm only against this addition. This is not a mandatory configuration for this theme.
Please, could you remove it?
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.
Removing it will not render the blog posts provided correctly.
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.
Which one?
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.
- This section doesn't work as intended. https://hugo-coder.netlify.app/posts/markdown-syntax/#other-elements--abbr-sub-sup-kbd-mark
- This section doesn't show the photo. https://hugo-coder.netlify.app/posts/placeholder-text/#vagus-elidunt
- And, this doesn't show emoji at all. https://hugo-coder.netlify.app/posts/emoji-support/
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.
Yes, I met the same problem thought it is not a big deal.
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.
I forgot to come back here.
- About the 'abbr, sub, sup, kbd, mark' no styling, this should be just missing CSS rules.
- The SVG file should be added with a shortcode. I don't know if there is a default one with Hugo, if not I can build one later.
- Emoji should be working. Maybe we are using an old version of Hugo to build the preview. I'm going to take a look on that issue.
Make following posts work as intended. - emoji-support - placeholder-text
Now, the following posts also support inline CSS.
|
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.
It is more safer to implement CSP when put in use. It just need some introduction about it.
Prerequisites
Put an
x
into the box(es) that apply:Description
pygmentsStyle
from bw to b2 to make it more readable in darkmode
math: true
uses mathjax. So usekatex:true
insteadconfig.toml
Issues Resolved
config.toml
helps in checking the code during pull requestsChecklist
Put an
x
into the box(es) that apply:General
#<ISSUE_NO>
if applicableResources
make release
to regenerate all CSS filesContributors
CONTRIBUTORS.md
if you aren't on it already