-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Add plugin: Folder Property Manager #6241
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
base: master
Are you sure you want to change the base?
Conversation
Hello!I found the following issues in your plugin submission Errors: ❌ Plugin ID mismatch, the ID in this PR ( This check was done automatically. Do NOT open a new PR for re-validation. Instead, to trigger this check again, make a change to your PR and wait a few minutes, or close and re-open it. |
Hello!I found the following issues in your plugin submission Errors: ❌ Plugin ID mismatch, the ID in this PR ( This check was done automatically. Do NOT open a new PR for re-validation. Instead, to trigger this check again, make a change to your PR and wait a few minutes, or close and re-open it. |
Hello!I found the following issues in your plugin submission Errors: ❌ Plugin ID mismatch, the ID in this PR ( This check was done automatically. Do NOT open a new PR for re-validation. Instead, to trigger this check again, make a change to your PR and wait a few minutes, or close and re-open it. |
Thank you for your submission, an automated scan of your plugin code's revealed the following issues: Required[1][2]:Obsidian's configuration directory isn't necessarily [1][2][3][4][5][6][7][8][9][10][11][12][13][14][15][16][17][18][19][20][21][22][23][24][25][26][27][28][29][30] and more :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. Optional[1][2][3][4][5][6][7][8]:Casting to Do NOT open a new PR for re-validation. |
/skip The use of style.display = "none" is intentional in this context, as it is used to dynamically show or hide table rows based on user interaction. This behavior is not easily replicable using only CSS without introducing complexity or performance overhead. Since themes generally do not affect visibility logic, we believe this usage is acceptable here. /skip first error are in test file (jest, wdio, etc...) . |
I am submitting a new Community Plugin
Repo URL
Link to my plugin: https://github.com/elekis-git/obsidian-properties-view
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
.