nextcloud/apps
Daniel Calviño Sánchez e39db808fb 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>
2018-11-22 05:58:25 +01:00
..
accessibility Merge pull request #12262 from nextcloud/dependabot/npm_and_yarn/apps/accessibility/webpack-4.24.0 2018-11-05 09:28:38 +01:00
admin_audit Shorten index name of calendar changes table 2018-11-14 10:14:23 +01:00
cloud_federation_api Update version requirements and versions of shipped apps 2018-09-05 14:27:06 +02:00
comments Shorten index name of calendar changes table 2018-11-14 10:14:23 +01:00
dav Merge pull request #12500 from nextcloud/swift-object-not-found 2018-11-20 09:49:57 +01:00
encryption Shorten index name of calendar changes table 2018-11-14 10:14:23 +01:00
federatedfilesharing [tx-robot] updated from transifex 2018-11-16 01:12:14 +00:00
federation Merge pull request #12447 from nextcloud/bugfix/12446/fix-oracle-install-again 2018-11-15 10:47:51 +01:00
files Fix rendering of the sidebar in Files app 2018-11-22 05:58:25 +01:00
files_external Fix loading of files external templates file 2018-11-18 11:13:46 +01:00
files_sharing Fix whitespace and file action cleanup 2018-11-20 19:46:49 +01:00
files_trashbin Rollback to default list view 2018-11-15 20:29:10 +01:00
files_versions [tx-robot] updated from transifex 2018-11-16 01:12:14 +00:00
lookup_server_connector Shorten index name of calendar changes table 2018-11-14 10:14:23 +01:00
oauth2 [tx-robot] updated from transifex 2018-11-21 01:12:15 +00:00
provisioning_api Merge pull request #12447 from nextcloud/bugfix/12446/fix-oracle-install-again 2018-11-15 10:47:51 +01:00
sharebymail Merge pull request #12447 from nextcloud/bugfix/12446/fix-oracle-install-again 2018-11-15 10:47:51 +01:00
systemtags Merge pull request #12447 from nextcloud/bugfix/12446/fix-oracle-install-again 2018-11-15 10:47:51 +01:00
testing Shorten index name of calendar changes table 2018-11-14 10:14:23 +01:00
theming Fix revert icon position on theming settings 2018-11-16 18:35:42 +01:00
twofactor_backupcodes [tx-robot] updated from transifex 2018-11-16 01:12:14 +00:00
updatenotification [tx-robot] updated from transifex 2018-11-21 01:12:15 +00:00
user_ldap [tx-robot] updated from transifex 2018-11-16 01:12:14 +00:00
workflowengine [tx-robot] updated from transifex 2018-11-19 01:12:04 +00:00