-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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: Modal Opener #4186
base: master
Are you sure you want to change the base?
Add plugin: Modal Opener #4186
Conversation
Thank you for your submission, an automated scan of your plugin code's revealed the following issues: Required[1][2][3][4][5][6]:You should not cast this, instead use a [1][2][3][4][5][6][7][8][9][10][11][12][13][14][15][16][17][18][19][20][21][22]: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]:Adding [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 consider limiting the number of Optional[1][2][3][4][5][6][7][8][9][10][11][12][13][14][15]:Casting to Do NOT open a new PR for re-validation. |
/skip [3] |
Changes requested by bot have been made, assigning human for additional review. |
Hello!I found the following issues in your plugin submission Errors: ❌ Could not parse 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: ❌ Could not parse 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. |
.modal { this.addCommand({ private debounceTimeout: NodeJS.Timeout | null = null; const response = await fetch(url, { method: 'HEAD', mode: 'no-cors' }); "Open in Modal Window": "Open in Modal Window",, 'Current File':'Current File',, 'Enable Animation and Blur':'Enable Animation and Blur',, 'Show File View Header':'Show File View Header',, 'Disable external click Close':'Disable external click Close',, 'Hide Tab Header':'Hide Tab Header',, 'Hide Tab Header':'Hide Tab Header',, 'Custom Commands':'Custom Commands',, 'Description':'Enter File Path or Web Link', |
Hi joethei, I've made the requested changes. Could you please review the updates at your convenience? Let me know if there's anything else I need to address.
Thank you! |
this.app.keymap.popScope(this.scope);, this.app.keymap.pushScope(this.scope); this.scope.register([], 'Escape', (evt: KeyboardEvent) => { if (savedHotkeys && savedHotkeys.length > 0) { You can use this.scope = new Scope(this.app.scope); // Allow app commands to work inside modal To allow commands to be ran inside of your modal. |
Hi joethei,
Could you please review my plugin again? |
I am submitting a new Community Plugin
Repo URL
Link to my plugin: https://github.com/likemuuxi/obsidian-modal-opener
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
.