-
Notifications
You must be signed in to change notification settings - Fork 280
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
Settings item in Tools menu #1020
Comments
I have no motivation to add such a menu item, sorry. See the comment: #940 (comment) |
Is there a function to open Settings dialog like below? coomanPlus.openCookieEditor();
s3menuwizard.open_options_window(); So users can add the item themselves |
No such function. TST's configuration dialog is only shown by Firefox's addon manager - I entrust anything around the configuration dialog to Firefox itself, so I really don't know details how to do it manually. And I believe that it is not a task of addon authors. |
In other words, I don't want to treat how to open configuration dialogs by myself because the most well-mannered way to open configuration UI of addons can change on future versions of Firefox. I hope to spend more time to fix other critical bugs instead of maintaining such unnecessary codes. |
And, as my comments on the referred issue #940, if you are annoyed by something and you have to open TST's configuration dialog again and again, then the problem should be solved with any other method better than adding menu item to open the configuration dialog. |
Thank you for sharing your thoughts. I found a way to open Settings dialog using S3 Menu Wizard StepsCreate an item and put this code in onCommand event openDialog('chrome://treestyletab/content/config.xul', '', 'chrome,titlebar,toolbar,centerscreen,modal'); and use this CSS to change the icon of the menu item:
ResultHope this helps people with the same request :) |
Thanks! |
Optional Settings menu item in Tools menu would be really handy oppose to searching for Options button in Add-ons page.
The text was updated successfully, but these errors were encountered: