8000 feat(mj-font): add fonts option by christophehenry · Pull Request #90 · jolimail/mrml-core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Jul 25, 2023. It is now read-only.

feat(mj-font): add fonts option #90

Merged
merged 5 commits into from
Jun 21, 2023

Conversation

christophehenry
Copy link
Contributor
@christophehenry christophehenry commented Jun 7, 2023

Also add a documentation on what options are available.

Fixes #88

8000
| Name | Comment | Default value |
|----------------------|------------------------------------------------------|------------------------------------------------------------------------------------------------------|
| `disable_comments` | Strip comments out of rendered HTML | `false` |
| `social_icon_origin` | Custom URL for fetching social icons | `None` |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe the format of the ressource at the URL should be specified here. Also I noticed that the default URL returns an error. This probably should be checked.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for spotting this, I created an issue here to track this 😉

christophehenry added a commit to christophehenry/mrml-core that referenced this pull request Jun 7, 2023
@christophehenry
Copy link
Contributor Author

I am sorry, I don't get what is the problem with the semantic PR check…

@christophehenry christophehenry changed the title Add fonts option feat(options): add fonts option Jun 7, 2023
@christophehenry
Copy link
Contributor Author
christophehenry commented Jun 20, 2023

@jdrouet Sorry to mention you but we need this feature for a project and this PR should be easy to review.

jdrouet added 2 commits June 21, 2023 08:00
Signed-off-by: Jérémie Drouet <jeremie.drouet@gmail.com>
@jdrouet jdrouet changed the title feat(options): add fonts option feat(mj-font): add fonts option Jun 21, 2023
jdrouet added 2 commits June 21, 2023 08:31
Signed-off-by: Jérémie Drouet <jeremie.drouet@gmail.com>
Signed-off-by: Jérémie Drouet <jeremie.drouet@gmail.com>
@jdrouet jdrouet merged commit 94be0ee into jolimail:main Jun 21, 2023
@christophehenry
Copy link
Contributor Author

@jdrouet Thank you very much. Would you agree to push a release?

@christophehenry christophehenry deleted the add-fonts-option branch June 21, 2023 09:14
@jdrouet
Copy link
Contributor
jdrouet commented Jun 21, 2023

👍 I'll push that tonight or tomorrow 😉

@jdrouet
Copy link
Contributor
jdrouet commented Jun 27, 2023

@christophehenry with a bit of delay, I published a new version 😉

@christophehenry
Copy link
Contributor Author

Awesome! Thx!

@christophehenry
Copy link
Contributor Author
christophehenry commented Jun 27, 2023

I'm sorry, I don't see 2.0.0-rc3 available on crates.io. How should I use it?
Edit: oh, NVM, it compiles. Thx.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document or support render options
2 participants
0