Mirror
/
zulip
mirror of https://github.com/zulip/zulip.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Remove duplicate connection-error message. 

There's really no need for a separate user-visible error message for
get_messages vs. get_events.

Fixes #4703.
This commit is contained in:
Rohitt Vashishtha 2017-05-08 18:48:43 +05:30 committed by Tim Abbott
parent 94b7058743
commit 5be7494ddf
2 changed files with 2 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
static/js/message_fetch.js
View File

@ -14,7 +14,7 @@ exports.reset_load_more_status = function reset_load_more_status() {
};
function process_result(messages, opts) {
$('#get_old_messages_error').removeClass("show");
$('#connection-error').removeClass("show");
if ((messages.length === 0) && (current_msg_list === message_list.narrowed) &&
message_list.narrowed.empty()) {
@ -120,7 +120,7 @@ exports.load_old_messages = function load_old_messages(opts) {
}
// We might want to be more clever here
$('#get_old_messages_error').addClass("show");
$('#connection-error').addClass("show");
setTimeout(function () {
exports.load_old_messages(opts);
}, 5000);

5
templates/zerver/index.html
View File

@ -80,11 +80,6 @@ var page_params = {{ page_params }};
{% trans %}<strong class="message">Unable to connect to
Zulip.</strong> Updates may be delayed.{% endtrans %} {{ _('Retrying soon...') }} <a class="restart_get_events_button">{{ _('Try now.') }}</a>
</div>
<div class="alert alert_sidebar alert-error home-error-bar" id="get_old_messages_error">
<div class="exit"></div>
{% trans %}<strong class="message">Unable to connect to
Zulip.</strong> Could not fetch messages.{% endtrans %} {{ _('Retrying soon...') }}
</div>
<div class="alert alert_sidebar alert-error home-error-bar" id="zephyr-mirror-error">
<div class="exit"></div>
{# The below isn't tagged for translation