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

Future of Admon Plugin #102

Open
KyleKing opened this issue Jan 16, 2024 · 1 comment
Open

Future of Admon Plugin #102

KyleKing opened this issue Jan 16, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@KyleKing
Copy link
Contributor

KyleKing commented Jan 16, 2024

Context

Since the admon plugin was initially ported to mdit-py-plugins, there have been requests for improvements (supporting collapsible MKDocs-style admonitions, MKDocs content tabs, etc.)

In response, I've refactored the logic to make it more extensible and published new versions of mdformat-mkdocs (for the mkdocs-related code) and mdformat-admon with the extensible version of the python-markdown !!! syntax (https://github.com/KyleKing/mdformat-admon/blob/d48ca09eb3160dbfbb2c8219319cba7e63ff8e4a/mdformat_admon/mdit_plugins/_python_markdown_admon.py)

Proposal

Because there is currently duplicate implementations of the admon logic (here and in mdformat-admon), we should try to reduce the duplication. I think there are a couple of options:

  1. Remove the admon plugin from mdit-py-plugins and point users to mdformat-admon if they want that functionality (we could have mdit-py-plugins raise an exception if someone tries to import with a better error or just remove entirely to reduce the maintenance burden
  2. Move a subset of the logic from mdformat-admon into mdit-py-plugins (the extensible factories and updated plugin) to replace what is currently here

Option 2 has the benefit of being easier to support because any issues with admonitions will be on a repo I maintain and all bug issues will go directly to me. Option 1 would be better because of backward compatibility and the usage of the the factory code without the dependency on mdformat.

What is your preference?

Tasks and updates

No response

@KyleKing KyleKing added the enhancement New feature or request label Jan 16, 2024
Copy link

welcome bot commented Jan 16, 2024

Thanks for opening your first issue here! Engagement like this is essential for open source projects! 🤗

If you haven't done so already, check out EBP's Code of Conduct. Also, please try to follow the issue template as it helps other community members to contribute more effectively.

If your issue is a feature request, others may react to it, to raise its prominence (see Feature Voting).

Welcome to the EBP community! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant