Skip to content
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

Reload/refresh button should have a tooltip #207

Closed
gnprice opened this issue Jul 12, 2017 · 2 comments
Closed

Reload/refresh button should have a tooltip #207

gnprice opened this issue Jul 12, 2017 · 2 comments

Comments

@gnprice
Copy link
Member

gnprice commented Jul 12, 2017

In the lower left, just above a gear icon, there's a circle-arrow "reload" icon. It's not obvious just what that's supposed to do; if nothing else, it should have a short tooltip.

On clicking it, it appears to reload the webview. That seems like something the user shouldn't have to do -- if they do that and it helps them, whatever issue it was resolving for them was a bug. And at least in a browser I think it's currently pretty rare for us to have a bug like that -- I think the only times I've benefitted from reloading a Zulip tab have been when it didn't handle a network change, due to the issue mentioned in zulip/zulip#5598 . So I'm curious how often people who've been using the app feel like they're doing that.

@akashnimare
Copy link
Member

Zhongyi - The reload button is still useful I think. In some cases, e.g. when app reloads on network reconnected, but somehow before DNS resolution is ready, the page will be blank and users need to reload the page manually.

Adding to this I think that reload button is necessary for now. We don't want a situation where a white blank page shows up (due to network issues etc) and the user has no idea what to do at that moment. Mostly a quick reload fixes everything.

Ideally, the app should be able to auto-reload itself. We're trying to figure out those cases when the white page shows up. We'll remove this reload button in future. Just to make it clear I have added a tooltip.

awesomescreenshot-2017-07-18t17-58-50-413z

@akashnimare
Copy link
Member

@gnprice thanks for the suggestions, closing this now.

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

No branches or pull requests

2 participants