Johannes Theiner
Johannes Theiner
[main.js](https://github.com/gitcpy/mermaid-zoom-drag/blob/ba576401f91a090730d11dfab9890c0fa70e12cf/.gitignore#L2) Since all your code is in that file it should not be in the `.gitignore`. (Sorry, the bot sometimes flags this even if it's incorrect) ["description": "A plugin to...
["id": "printsidian",](https://github.com/gitatmax/printsidian/blob/de738d8221b00ed79f62a773b043acb30c78dc3c/manifest.json#L2-L3) Please rename your plugin, the name should not include abreviations of "Obsidian" ["description": "Print from Obsidian",](https://github.com/gitatmax/printsidian/blob/de738d8221b00ed79f62a773b043acb30c78dc3c/manifest.json#L6) The description should also not include "Obsidian" We have published a guide...
Your README is missing a lot of information on how to use the plugin. Some screenshots/GIFs would also be nice. [### For Users](https://github.com/Yohh/obsidian-simple-sketch/blob/8dc054e062d9f66fe37ece26c5f880e791623810/README.md#L38-L44) And these are not the correct installation...
[async loadParticlesJS() {](https://github.com/chasebank87/mesh-ai/blob/6410257b456d49021da5c48ca4f663aa40c438b2/main.ts#L39-L46) This should be bundled into the plugin, not loaded externally. [workspace.detachLeavesOfType(MESH_VIEW_TYPE);](https://github.com/chasebank87/mesh-ai/blob/6410257b456d49021da5c48ca4f663aa40c438b2/main.ts#L59) Do not detach leaves with your custom view here, this is an antipattern, see:
[containerEl.createEl('h2').setText('Command Tracker - Settings');](https://github.com/namikaze-40p/obsidian-command-tracker/blob/6169e326e128b865017fb7fb4d4dd5373df19bf7/src/settings.ts#L63) [Don't add a top-level heading in the settings tab, such as "General", "Settings", or the name of your plugin.](https://docs.obsidian.md/Plugins/Releasing/Plugin+guidelines#Only+use+headings+under+settings+if+you+have+more+than+one+section.) [div.createEl('h6', { text: 'Command Tracker View' });](https://github.com/namikaze-40p/obsidian-command-tracker/blob/6169e326e128b865017fb7fb4d4dd5373df19bf7/src/view.ts#L123)...
[name: 'Generate Banner Image (Flux AI)',](https://github.com/kkingsbe/obsidian-ai-banners/blob/8de41273600621e01198b13a2b1ada554d4d7f95/main.ts#L22), [.setName('OpenAI API Key')](https://github.com/kkingsbe/obsidian-ai-banners/blob/8de41273600621e01198b13a2b1ada554d4d7f95/lib/settingstab.ts#L18), [.setName('fal.ai API Key')](https://github.com/kkingsbe/obsidian-ai-banners/blob/8de41273600621e01198b13a2b1ada554d4d7f95/lib/settingstab.ts#L28) [Use sentence case in UI](https://docs.obsidian.md/Plugins/Releasing/Plugin+guidelines#Use+sentence+case+in+UI) [const response = await fetch(imageUrl);](https://github.com/kkingsbe/obsidian-ai-banners/blob/8de41273600621e01198b13a2b1ada554d4d7f95/main.ts#L89) Use the `requestUrl` function from the Obsidian API...
Don't include "Plugin" in the description.
Please remove the main.js file from the repo and add it to your .gitignore. Compiled release files shouldn't get mixed with source code in your commits and should go to...
[``](https://github.com/jamiefdhurst/obsidian-auto-periodic-notes/blob/341edc782567da682975450b4cd71e6aa404ceb4/src/settings/pages/SettingsTab.svelte#L37-L40) For section headings in settings use: ```ts new Setting(containerEl).setName('name here').setHeading(); ``` [`No Periodic Notes Enabled`](https://github.com/jamiefdhurst/obsidian-auto-periodic-notes/blob/341edc782567da682975450b4cd71e6aa404ceb4/src/settings/components/PeriodicNotesUnavailableBanner.svelte#L5), [Open and Pin New {capitalise(periodicity)} Notes](https://github.com/jamiefdhurst/obsidian-auto-periodic-notes/blob/341edc782567da682975450b4cd71e6aa404ceb4/src/settings/components/OpenAndPinSetting.svelte#L11) [Use sentence case in UI](https://docs.obsidian.md/Plugins/Releasing/Plugin+guidelines#Use+sentence+case+in+UI) [import moment, {...
[linen](https://github.com/pkhr/obsidian-inline-translate/blob/b8da66be692e883f44465f6dba5bf725d194f240/main.ts#L28) Typo [editorCallback: async (editor: Editor, view: MarkdownView) => {](https://github.com/pkhr/obsidian-inline-translate/blob/b8da66be692e883f44465f6dba5bf725d194f240/main.ts#L59-L68), [editorCallback: async (editor: Editor, view: MarkdownView) => {](https://github.com/pkhr/obsidian-inline-translate/blob/b8da66be692e883f44465f6dba5bf725d194f240/main.ts#L75-L84) This should be `editorCheckCallback`, then the command will only show up if...