-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Add plugin: Dynamic Outline #4760
Conversation
Thank you for your submission, an automated scan of your plugin code's revealed the following issues: Required[1]:The command name should not include the plugin name. [1][2]:You should avoid assigning styles via JavaScript or in HTML and instead move all these styles into CSS so that they are more easily adaptable by themes and snippets. [1][2]:Using Do NOT open a new PR for re-validation. |
Changes requested by bot have been made, assigning human for additional review. |
Sorry for the delay in reviewing this. that does not require the core "Outline" plugin name: "Toggle for Current File",, Dynamic Outline callback: () => { All in all, a good implementation, good job. |
Hi, @joethei, thanks for the review! I've made the changes to the code and bumped the version. |
I am submitting a new Community Plugin
Repo URL
Link to my plugin: https://github.com/theopavlove/obsidian-dynamic-outline
Release Checklist
main.js
manifest.json
styles.css
(optional)v
)id
in mymanifest.json
matches theid
in thecommunity-plugins.json
file.I have given proper attribution to these other projects in my
README.md
.