Fix rendering of the sidebar in Files app

When a view is rendered it should not be concerned with where it is
going to be placed in the document; in general this should be a
responsibility of the object using the view.

Moreover, when the details view is rendered it should simply prepare a
skeleton that includes the root elements provided by the plugins; those
elements will be updated by the plugins as needed when a file or a tab
is selected.

Finally, the details view should not be explicitly rendered. The
rendering removes the previous elements, but that is needed only when
the details view is in a dirty state, that is, when new plugins were
added since the last time that it was rendered. However, that dirty
state is internally handled, and the view is automatically rendered
again if needed when a file info is set.

Due to all that the details view is no longer explicitly rendered when
updating it with a different file. Also, as each file list has its own
details view, and each details view has its own element, but there can
be only one details view/sidebar element in the document, when the file
list updates the details view it also replaces the current one in the
document with its own details view if needed (that is, if it is not the
current one already).

Besides that, when the element of a details view is replaced with the
element of a different details view the old one should be detached from
the document, but never removed. Otherwise the event handlers would not
work when that element is attached again later (when changing to a
different section in the Files app and then going back to the previous
one).

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Add acceptance tests for showing again the input field for tags

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Add acceptance tests for opening and closing again the details view

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Add acceptance tests for marking a file as favorite in the details view

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Add acceptance tests for switching to the comments of another file

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

[tx-robot] updated from transifex

Merge pull request #12531 from nextcloud/bugfix/noid/hide-download-folders

Allow to hide download option for folders shared by link

Fix whitespace and file action cleanup

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Clear link to avoid downloads

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Compile handlebars

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Allow to hide download option for folders shared by link

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Merge pull request #12544 from nextcloud/fix/better_update_share_handling

Handle permission in update of share better

Merge pull request #12552 from nextcloud/fix/11146/respect_disabled_password_reset

Respect the disabled setting for lost_password_link

Merge pull request #12491 from nextcloud/share-menu-click-fix

Fix share link password input

Fix share link password input

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Merge pull request #12350 from nextcloud/filepicker-sorting

Enable sorting in file picker

Respect the disabled setting for lost_password_link

Fixes #11146
As documented when it is set to disabled the user can't request a lost
password.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Merge pull request #12543 from nextcloud/fix/12498/bearer_tokens_are_apptokens

Bearer tokens are app token

Merge pull request #12494 from nextcloud/cache-notfound-id

return the correct value when trying to get a non existing item from cache by id

Merge pull request #12500 from nextcloud/swift-object-not-found

forward object not found error in swift as dav 404

Handle permission in update of share better

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Merge pull request #12539 from nextcloud/fix-redirect-design

Fix redirect page design details

Bearer tokens are app token

Fixes #12498

This means that we set that it is a proper app token once it is
validated. This will allow the 2FA middleware to just run the same
check.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

[tx-robot] updated from transifex

Fix CSS specificity preventing buttons displayed in pill style

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

Fix header text color on redirect page

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

add test for non existing file by id

Signed-off-by: Robin Appelman <robin@icewind.nl>

Merge pull request #12525 from nextcloud/contrast-fix-updatenotifications

Fix updatenotifications display, dependencies and csp

Add PHPDoc for thrown exception

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #12495 from nextcloud/login-theming-primary-fix

Fix login primary colour and feedback

Merge pull request #12401 from pachulo/fix/11107/fix-php-mail-warning

Fix the warning appearing in the admin section when mail_smtpmode is not configured

Fix login primary colour and feedback

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

forward object not found error in switch as dav 404

Signed-off-by: Robin Appelman <robin@icewind.nl>

Use the proper default values

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #12515 from nextcloud/papercut/12295/grid-view-lazy-loading

Make number of file list entries depending on the width for grid view

Fix updatenotifications display, dependencies and csp

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Merge pull request #12496 from nextcloud/revert-icon-theming-fix

Fix revert icon position on theming settings

[tx-robot] updated from transifex

Merge pull request #12517 from nextcloud/bugfix/noid/files-external-templates

Fix loading of files external templates file

Merge pull request #12518 from nextcloud/bugfix/noid/htaccess-woff2

Fix loading of .woff2 files in .htaccess

Fix loading of files external templates file

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Fix loading of .woff2 files in .htaccess

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Make number of file list entries depending on the width for grid view

Signed-off-by: Julius Härtl <jus@bitgrid.net>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Fix revert icon position on theming settings

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

return the correct value when trying to get a non existing item from cache by id

Signed-off-by: Robin Appelman <robin@icewind.nl>

Merge pull request #12489 from nextcloud/bugfix/11179/safari-performance

Only use width and opacity for transition

Merge pull request #12487 from nextcloud/settings-svg-icons-scss-fix

Remove unused svg api route

Only use width and opacity for transition

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Remove unused sg api route

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Merge pull request #12484 from nextcloud/loader-color-fixes

Fix loader colours for better visibility

Properly align loading indicator when switching folders

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Merge pull request #12483 from nextcloud/fix-login-loader

Fix login loader position and theming support

Flip sort indicators

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Allow sorting in the file picker

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Merge pull request #12476 from nextcloud/welcome-bot

Add welcome bot

Merge pull request #12474 from nextcloud/default-to-files-list

Rollback to default list view

Fix loader colours for better visibility

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Merge pull request #12481 from nextcloud/bugfix/noid/fix-navigation-layout-of-new-items

Fix navigation layout for items that are added on a re-render

Fix login loader position and theming support

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Fix navigation layout for items that are added on a re-render

Signed-off-by: Joas Schilling <coding@schilljs.com>

Update .github/config.yml

Co-Authored-By: rullzer <rullzer@users.noreply.github.com>

Update .github/config.yml

Co-Authored-By: rullzer <rullzer@users.noreply.github.com>

[tx-robot] updated from transifex

Add welcome bot

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Rollback to default list view

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Merge pull request #12472 from nextcloud/dav-versions-user

actually return the user when creating version dav nodes

Merge pull request #12355 from nextcloud/version/noid/15.0.0beta2

15.0.0 Beta 2

actually return the user when creating version dav nodes

Signed-off-by: Robin Appelman <robin@icewind.nl>

15.0.0 Beta 2

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #12467 from nextcloud/bugfix/11901/show-thumbnail-on-rename

Show thumbnail on rename

Enhance jsunit tests

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #12458 from nextcloud/bugfix/dav_shares_hickup

fixes dav share issue with owner

fix caldav tests

Signed-off-by: Georg Ehrke <developer@georgehrke.com>

Merge pull request #12466 from nextcloud/fix/sting-doctype-casing

Fix string doctype casing

Merge pull request #12421 from nextcloud/bugfix/noid/icons-cacher-app-prefix

Prefix icon variables with app name to avoid conflicts

Show thumbnail on rename

* does not hide thumbnail when rename is shown
* fixes layout for grid and list view
* fixes #11901

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Fix string doc type casing

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

Prefix icon variables with app name to avoid conflicts

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Merge pull request #12292 from nextcloud/fix/2947/lapse-sizelimit-error

avoid logging of "Partial search results returned: Sizelimit exceeded…"

Merge pull request #12447 from nextcloud/bugfix/12446/fix-oracle-install-again

Shorten index name of calendar changes table

Merge pull request #11200 from nextcloud/bugfix/2954/interaction-with-groups-containing-a-slash

Fix interaction with groups that contain a slash

Fix interaction with groups that contain a slash

Signed-off-by: Joas Schilling <coding@schilljs.com>

[tx-robot] updated from transifex

fixes dav share issue with owner

Signed-off-by: Georg Ehrke <developer@georgehrke.com>

Merge pull request #12448 from nextcloud/bugfix/noid/dav-loadapps

Load apps that have a dav type set before the dav server plugins

Merge pull request #12457 from nextcloud/fix/noid/fix-phpdoc

Update PHPDoc to fix typos

Update PHPDoc to fix typos

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #12417 from nextcloud/bugfix/noid/app-files_external-replace-insertIfNotExist

Fix UniqueConstraintViolationException while insert into files_external config tables

Merge pull request #12456 from nextcloud/move_to_github_bot

Sign-off check is now handled by a github app

Merge pull request #12454 from nextcloud/list-shrink-fix

Fix breakpoint and shrinking of the content + list

Sign-off check is now handled by a github app

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Merge pull request #12455 from nextcloud/twofactor-backup-settings

Don't register twofactor_backup settings as "regular" settings

Merge pull request #12450 from nextcloud/fix/11868/decryption_hardening

Throw exception if decryption fails

Allow apps to use the variables too

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

twofactor auth settings are not regular settings

Signed-off-by: Robin Appelman <robin@icewind.nl>

Log invalid settings class

Signed-off-by: Robin Appelman <robin@icewind.nl>

Verify that isPhpMailerUsed uses config as expected

Signed-off-by: Kyle Fazzari <kyrofa@ubuntu.com>

Fix breakpoint and shrinking of the content + list

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>