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

Revert clearing of search when clicking on conversation / contact #1042

Merged
merged 1 commit into from
Mar 5, 2021

Conversation

threema-danilo
Copy link
Contributor

It seems that when searching through a lot of conversations (I have over
230 conversations), after clicking on a conversation, the search clears
almost instantly, but it sometimes takes multiple seconds until the
conversation has opened.

This has a lot to do with the inefficient architecture of AngularJS,
where every UI update can trigger a lot of processing code, which in
turn may cause yet another UI update.

For now, we need to disable the clearing/hiding of the search when
clicking on a search result.

Refs #1026.

It seems that when searching through a lot of conversations (I have over
230 conversations), after clicking on a conversation, the search clears
almost instantly, but it sometimes takes multiple seconds until the
conversation has opened.

This has a lot to do with the inefficient architecture of AngularJS,
where every UI update can trigger a lot of processing code, which in
turn may cause yet another UI update.

For now, we need to disable the clearing/hiding of the search when
clicking on a search result.
@threema-danilo threema-danilo added the ui User interface related label Mar 5, 2021
@threema-danilo threema-danilo self-assigned this Mar 5, 2021
@threema-danilo threema-danilo merged commit 768d693 into master Mar 5, 2021
@threema-danilo threema-danilo deleted the revert-search-clearing-on-click branch March 5, 2021 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui User interface related
Development

Successfully merging this pull request may close these issues.

1 participant