Skip to content

Hide the tooltip if the list row is rerendered - #18252

Merged
rullzer merged 1 commit into
stable17from
17/bugfix/noid/tooltip-hide-date
Dec 5, 2019
Merged

Hide the tooltip if the list row is rerendered#18252
rullzer merged 1 commit into
stable17from
17/bugfix/noid/tooltip-hide-date

Conversation

@juliusknorr

Copy link
Copy Markdown
Member

Steps to reproduce:

  • Click the share icon for a file in the files list
  • Click the date on another file (tooltip shows)
  • Let the cursor stay on the date until the highlighted row changes
  • Move the cursor -> the tooltip stays

Could not reproduce on master, therefore just 17/16

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

/backport to stable16

Comment threadapps/files/js/filelist.js Outdated
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr
juliusknorrforce-pushed the 17/bugfix/noid/tooltip-hide-date branch from e2a77d1 to 7b6eb80CompareDecember 5, 2019 13:46

@GretaDGretaD left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@rullzer
rullzer merged commit a6f9a08 into stable17Dec 5, 2019
@rullzer
rullzer deleted the 17/bugfix/noid/tooltip-hide-date branch December 5, 2019 19:47
@rullzerrullzer added this to the Nextcloud 18 milestone Dec 5, 2019
@rullzer

Copy link
Copy Markdown
Member

/backport to stable17

@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable17 failed. Please do this backport manually.

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable16 in #18260

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

This is stable 17 only as i could not reproduce it on master

@rullzerrullzer mentioned this pull request Dec 11, 2019
1 task
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@juliusknorr@rullzer@GretaD@skjnldsv