-
Notifications
You must be signed in to change notification settings - Fork 7
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
Fix options page and remove extension info option #252
Conversation
Signed-off-by: Suhas Hariharan <hariharan774531@sas.edu.sg>
Signed-off-by: Suhas Hariharan <hariharan774531@sas.edu.sg>
Signed-off-by: Suhas Hariharan <hariharan774531@sas.edu.sg>
Signed-off-by: Suhas Hariharan <hariharan774531@sas.edu.sg>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm 👍
Oh, what was the removal of the extension info option for? |
I just realised it wasn't an option in the extension option page so I thought I'd just remove it for now, if you'd prefer I can add it back and make it a proper selectable option in the page. |
It's used to control whether or not the extension info page is open or closed: saspes/src/js/components/ExtensionInfo.vue Lines 87 to 90 in f48f55a
|
ah, sorry I completely missed that functionality, I assumed it was yet to be implemented on the options page. I can re-add that in another PR. |
closes #239 |
Fix options page and remove extension info option
closes #239