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

Made example contents working as expected and visible #575

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

rabinadk1
Copy link
Contributor

@rabinadk1 rabinadk1 commented May 12, 2021

Prerequisites

Put an x into the box(es) that apply:

  • This pull request fixes a bug.
  • This pull request adds a feature.
  • This pull request introduces breaking change.

Description

  • 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
  • Remove not working short codes
  • Comment out CSP from config.toml

Issues Resolved

  • Adds visibility to codes rendered using pygments in dark mode
  • The HTML codes inside of the markdowns are now rendered correctly
  • Katex codes are rendered properly
  • Removal of CSP from config.toml helps in checking the code during pull requests

Checklist

Put an x into the box(es) that apply:

General

  • Describe what changes are being made
  • Explain why and how the changes were necessary and implemented respectively
  • Reference issue with #<ISSUE_NO> if applicable

Resources

  • If you have changed any SCSS code, run make release to regenerate all CSS files

Contributors

  • Add yourself to CONTRIBUTORS.md if you aren't on it already

rabinadk1 added 3 commits May 12, 2021 12:12
* 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]
Copy link
Owner

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?

Copy link
Contributor Author

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.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which one?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

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.

Copy link
Owner

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
@rabinadk1
Copy link
Contributor Author

Now, the following posts also support inline CSS.

  • emoji-support
  • placeholder-text

Copy link
Contributor

@ccviolett ccviolett left a 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.

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