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

"Select a source" message appears when source is already selected #1290

Closed
eloquence opened this issue Sep 1, 2021 · 6 comments · Fixed by #1329
Closed

"Select a source" message appears when source is already selected #1290

eloquence opened this issue Sep 1, 2021 · 6 comments · Fixed by #1329
Labels
bug Something isn't working release QA
Milestone

Comments

@eloquence
Copy link
Member

Reported by @creviera in #1275 (comment)

Steps to reproduce

Select a source message appears when a source is already selected.

  1. Select a source
  2. Delete files and messages
  3. Log out
  4. Log back in

Expected behavior

You continue to see the tear pattern in the conversation view of the selected source

Actual behavior

The "Select a source" message is displayed on top of the tear pattern until the next sync

@eloquence eloquence added the bug Something isn't working label Sep 1, 2021
@conorsch conorsch added this to the 0.5.0 milestone Sep 1, 2021
@conorsch
Copy link
Contributor

Was able to reproduce this one today. Didn't grab a screenshot, because the sync clears up the artifact, as noted above.

@gonzalo-bulnes
Copy link
Contributor

I was able to see a flash of content, I suppose that counts as a reproduction. 👍
@creviera I'm interested in learning to slow down the sync rate, so that we can reproduce this more reliably. 🙂 Not a blocker for now, though.

@sssoleileraaa
Copy link
Contributor

Tip for debugging: this happens if you log in while a source who's files and messages have been deleted is already selected. So no need to keep deleting files and messages. You can do it just once.

@gonzalo-bulnes
Copy link
Contributor

Note: the sync doesn't seem to be involved.

@sssoleileraaa
Copy link
Contributor

sssoleileraaa commented Nov 4, 2021

I think the issue might be between these two calls clobbering each other:

self.update_sources()
self.gui.refresh_current_source_conversation()

Here is where we decided whether or not to display the "Select a source" message:

self.empty_conversation_view.show_no_source_selected_message()
and the update_deletion_markers is where we decide whether or not to display the "Files and messages deleted" message.

These are the two messages showing up on top of each other.

@sssoleileraaa
Copy link
Contributor

In case it helps...
Screenshot from 2021-11-03 18-13-56

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working release QA
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants