Skip to content
This repository was archived by the owner on Nov 1, 2020. It is now read-only.

Conversation

@danxuliu
Copy link
Member

Follow-up to #415

If the tooltip is not explicitly hidden it would still be shown once the autocompletion dropdown is shown (behind it while it is open, but fully visible when it is closed).

This unifies its behaviour with the one used in the server, but it does not address its problems (for example, if a search is started and while it is being performed the input field is cleared the tooltip would be still shown once the search response is received, even if the input field is now empty).

How to test:

  • Open an album in the Gallery app
  • Show the Share dialog for the album
  • In the search field type the name of a user that exists, and then add any letter to it; the No results found tooltip is shown.
  • Remove the extra letter; the autocompletion dropdown is shown
  • Click on an empty space of the share dialog to close the autocompletion dropdown

Expected result:
No tooltip is visible.

Actual result:
The No results found tooltip is still visible.

If the tooltip is not explicitly hidden it would still be shown once the
autocompletion dropdown is shown (behind it while it is open, but fully
visible when it is closed).

This unifies its behaviour with the one used in the server, but it does
not address its problems (for example, if a search is started and while
it is being performed the input field is cleared the tooltip would be
still shown once the search response is received, even if the input
field is now empty).

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
@danxuliu danxuliu added bug Something isn't working backport-request Pending backport by the backport-bot feature:sharing 3. to review Waiting for reviews labels Mar 26, 2018
@danxuliu danxuliu added this to the Nextcloud 14 milestone Mar 26, 2018
@danxuliu danxuliu requested review from rullzer and skjnldsv March 26, 2018 12:19
Copy link
Member

@MorrisJobke MorrisJobke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

@MorrisJobke MorrisJobke merged commit bd99aac into master Apr 3, 2018
@MorrisJobke MorrisJobke deleted the hide-no-results-found-tooltip-on-autocompletion branch April 3, 2018 15:41
@danxuliu danxuliu removed the backport-request Pending backport by the backport-bot label Apr 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

3. to review Waiting for reviews bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants