]> source.dussan.org Git - nextcloud-server.git/commit
Split the menu entry for external shares in two 17861/head
authorDaniel Calviño Sánchez <danxuliu@gmail.com>
Thu, 7 Nov 2019 20:30:56 +0000 (21:30 +0100)
committerDaniel Calviño Sánchez <danxuliu@gmail.com>
Mon, 30 Dec 2019 09:29:36 +0000 (10:29 +0100)
commit883a71ce8e00d317e1827deacf46ee0c37934485
tree4af874efc9cabf30d01f20f69de755362b08ba1b
parent33b2f4e29589e6a415612f7d15525659847f3a8a
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>
core/css/public.scss
core/js/public/publicpage.js
lib/public/AppFramework/Http/Template/ExternalShareMenuAction.php