diff options
author | Daniel Calviño Sánchez <danxuliu@gmail.com> | 2018-01-22 11:57:53 +0300 |
---|---|---|
committer | Morris Jobke <hey@morrisjobke.de> | 2018-01-22 19:19:13 +0300 |
commit | 3aeda32b3b26629491873e1f2c5d2d7aa76ffe5f (patch) | |
tree | e03b560cf06af8df8f6191426708627a5e046acb | |
parent | 837826324d555bce463bbdb9959f1898d6e21a64 (diff) |
Fix adding shares to wrong HTML elementv12.0.5
"_addShareWith" adds a share to the "#shareWithList" element, which is a
descendant of the share dropdown. However, the "Sharing" tab of the
Files app sidebar contains its own "#shareWithList" element, so when the
share dropdown is shown on top of the Files app (by clicking on an image
and then sharing it from the slideshow) and the "Sharing" tab is open in
the sidebar the share element ends added to the wrong element. To
prevent this the matched element is now limited to descendants of the
share dropdown.
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
-rw-r--r-- | js/vendor/nextcloud/share.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/js/vendor/nextcloud/share.js b/js/vendor/nextcloud/share.js index 029637d6..f01479ac 100644 --- a/js/vendor/nextcloud/share.js +++ b/js/vendor/nextcloud/share.js @@ -867,7 +867,7 @@ } html += '</div>'; html += '</li>'; - html = $(html).appendTo('#shareWithList'); + html = $(html).appendTo('#dropdown #shareWithList'); if (oc_config.enable_avatars === true) { if (shareType === this.SHARE_TYPE_USER) { html.find('.avatar').avatar(escapeHTML(shareWith), 32); |