Commit b7c849f3 authored by Michael Gratton's avatar Michael Gratton 🤞 Committed by Michael Gratton

Fix composer being hidden when autoselect disabled and mail appended

When autoselect is disabled, we need to ensure the conversation viewer's
spinner needs to be hidden at some point, but we don't want to attempt
to do that if a composer is visible, since there clearly won't be any
spinner visible.

Fixes #251
parent f2a11edd
Pipeline #61358 passed with stages
in 42 minutes and 21 seconds
......@@ -1217,10 +1217,13 @@ public class GearyController : Geary.BaseObject {
enable_message_buttons(false);
} else {
// When not doing autoselect, we never get
// conversations_selected firing from the convo list, so
// we need to stop the loading spinner here
// conversations_selected firing from the convo list,
// so we need to stop the loading spinner here. Only
// do so if there isn't already a selection or a
// composer to avoid interrupting those.
if (!this.application.config.autoselect &&
list.get_selection().count_selected_rows() == 0) {
list.get_selection().count_selected_rows() == 0 &&
!viewer.is_composer_visible) {
viewer.show_none_selected();
enable_message_buttons(false);
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment