Skip to content

Add files of first version of plugin Bg Remover#436

Open
joselmm wants to merge 1 commit intoONLYOFFICE:masterfrom
joselmm:bg-remover-plugin
Open

Add files of first version of plugin Bg Remover#436
joselmm wants to merge 1 commit intoONLYOFFICE:masterfrom
joselmm:bg-remover-plugin

Conversation

@joselmm
Copy link

@joselmm joselmm commented Aug 27, 2025

3

The original code (vite + react ) is in this repo https://github.com/joselmm/bg-remove
baseUrl (just to test): https://joselmm.github.io/bg-remove/dist/

@joselmm joselmm changed the title Add files of version of plugin Bg Remover Add files of first version of plugin Bg Remover Aug 27, 2025
@trofim24
Copy link
Contributor

Hi @joselmm, thanks for submitting the BG Remover plugin!

I've reviewed the plugin against our guidelines and found a couple of issues that need to be addressed before we can merge.

Blocked Issues
Plugin not registered in store/config.json

Please add the plugin entry to store/config.json:

{ "name": "bg-remover", "discussion": "" }
Missing offered field in config.json

The provider field should be renamed to offered to follow the plugin configuration standard:

"offered": "joselmm",

Suggestions (Optional)
Consider adding separate dark/light toolbar icons in resources/dark/ and resources/light/ directories with scaled versions (@1.25x, @1.5x, @1.75x, @2x) for better theme support.
Consider using the icons2 array format in config.json for full DPI scaling support.
Please make these changes and update the PR. Let me know if you have any questions!

@trofim24 trofim24 added the waiting feedback Issues that we waiting to be answered from author of issue label Mar 26, 2026
@joselmm
Copy link
Author

joselmm commented Mar 26, 2026

Thanks for the suggestions, I'll let you know as soon as I make the necessary changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting feedback Issues that we waiting to be answered from author of issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants