diff options
| author | Daniel Calviño Sánchez <danxuliu@gmail.com> | 2019-11-07 23:30:56 +0300 |
|---|---|---|
| committer | Daniel Calviño Sánchez <danxuliu@gmail.com> | 2019-12-30 12:29:36 +0300 |
| commit | 883a71ce8e00d317e1827deacf46ee0c37934485 (patch) | |
| tree | 4af874efc9cabf30d01f20f69de755362b08ba1b /core | |
| parent | 33b2f4e29589e6a415612f7d15525659847f3a8a (diff) | |
Split the menu entry for external shares in two
The external shares entry showed a "button" that, when pressed, replaced
the button with the input to set the remote share address. The "button"
was actually a label for the input, so when the label was focused it
transferred the focus to the input and thus pressing enter or space did
not show the input. Moreover, inputs inside links are not valid HTML,
and once shown there was no way to hide the input again.
Due to all this, and for consistency with the direct link input, the
external share input was moved to a different menu item that is shown
and hidden when the button, which nows is also a real button, is
clicked.
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Diffstat (limited to 'core')
| -rw-r--r-- | core/css/public.scss | 3 | ||||
| -rw-r--r-- | core/js/public/publicpage.js | 5 |
2 files changed, 3 insertions, 5 deletions
diff --git a/core/css/public.scss b/core/css/public.scss index 6ec22cbc6d2..b171dd4970d 100644 --- a/core/css/public.scss +++ b/core/css/public.scss @@ -21,10 +21,9 @@ $footer-height: 65px; opacity: 1; } } - #save-external-share { + #external-share-menu-item { form { display: flex; - margin: 0; } .hidden { display: none; diff --git a/core/js/public/publicpage.js b/core/js/public/publicpage.js index e71bdff12ae..4fb9c67bfee 100644 --- a/core/js/public/publicpage.js +++ b/core/js/public/publicpage.js @@ -26,9 +26,8 @@ $(document).ready(function () { $(this).next('.popovermenu').toggleClass('open'); }); - $('#save-external-share').find('label').click(function () { - $(this).toggleClass('hidden'); - $('.save-form').toggleClass('hidden') + $('#save-external-share').click(function () { + $('#external-share-menu-item').toggleClass('hidden') $('#remote_address').focus(); }); |