-
Notifications
You must be signed in to change notification settings - Fork 0
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
consider removing the "Go!" button #160
Comments
The "Go!" button was actually specifically requested and thoughtfully designed towards the user experience. |
ok... I'm predicting that it will conflict with resolving these issues |
Interesting!! Thank you for the heads up |
Indeed the 'Go!' button was considered necessary as we want to make sure that the user is aware that they can choose an asset class and sector to view. There was a worry that if we directly show results for one asset class and one sector some users might now realize they can see results for other sectors/asset classes as well. Apparently that was the case for the interactive report. I believe what is cumbersome right now is that the user needs to make a selection each time they change a tab which is a different problem and it is captured in #144. If we can solve that issue then I don't think that #145 will be conflicting with the existence of the button. We could add a state to URL (if it is at all possible) indicating if the first selection was made. |
It seems awkward to require clicking the "Go!" button to see the results, especially when the sector and investment type selectors have already defaulted to some value. Could we have it "go" automatically to whatever the default values for the selectors have?
The text was updated successfully, but these errors were encountered: