-
Notifications
You must be signed in to change notification settings - Fork 71
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
Move settings to options (background) page #39
Comments
True, there's a lot of stuff going on, but maybe it could be weird to hide them behind a curtain. The background page is hard to access (you either have to do it through a button in your bar, which I guess nobody would like, or through the chrome extension settings. I think it'd be a bit cumbersome and it'd remove the flexibility we had so far in adding new things and letting the user know about it. I think what can be done for now is that some options are merged into other options (not sure which though), and then we can try a new design on the settings menu that solves the problem. There could be two solutions to this off the top of my mind. Either we implement option grouping, so that we have only two entries in the settings (styling and functionality), and each setting goes under one of the two, or we make the settings list scrollable and hide some of the settings in a way that you can see them if you scroll. The latter might not work well, because some user might never discover the additional options unless it's made very clear. |
Scrollable list (with an indicator) would be a good idea. |
Following options can be mandatory and removed from option list: Infinite scrolling |
I'm not 100% sure on this. My problem is, if these become mandatory, we might lose users who don't want to have them as mandatory (especially with infinite scrolling I'd say). Also, if we remove these it means we'll have to remove more once new features come along, so it's probably not a viable way to proceed. I think the best way is to find a solution to the display problem by somehow grouping or paginating. |
Why we can't open background page when user clicks on gear icon? |
I'm not 100% sure it's possible, but I also think it would be a downgrade from the current version. |
That might be a good idea, nice one. It could either slide down or cover the links area. |
I can try to whip something up in a separate branch this weekend. |
@gabrielecirulli It's kind of starting to get messy now when you click on gear icon. There 11 options, should be 5 max IMHO. We can make some options mandatory like user tooltips, but anyway we should think about moving on... Your thoughts Gabe?
The text was updated successfully, but these errors were encountered: