nextcloud / server

☁️ Nextcloud server, a safe home for all your data
https://nextcloud.com
GNU Affero General Public License v3.0
27.54k stars 4.08k forks source link

Add tooltips on server status indicators for federated sharing / tidy up Federated/Federation settings section(s) #48242

Open nfebe opened 2 months ago

nfebe commented 2 months ago

Description

Sample image

Steps to reproduce

See notes on federating : https://docs.nextcloud.com/server/latest/admin_manual/configuration_files/federated_cloud_sharing_configuration.html

Expected behavior

cc: @nextcloud/d

Nextcloud Server version

master

Operating system

None

PHP engine version

None

Web server

None

Database engine version

None

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

None

What user-backends are you using?

Configuration report

No response

List of activated Apps

No response

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

No response

joshtrichards commented 2 months ago

A red light means the connection failed. The yellow light indicates a successful connection, with no user names exchanged. The green light indicates a successful connection with user names exchanged.

Maybe we change how we indicate the state of things a bit.

It seems there are really four states:

https://github.com/nextcloud/server/blob/da591eaec3050ab02b4a74d1304db78335eeb7fa/apps/federation/lib/TrustedServers.php#L25-L32

And we try to reduce them to three single color icons:

https://github.com/nextcloud/server/blob/da591eaec3050ab02b4a74d1304db78335eeb7fa/apps/federation/templates/settings-admin.php#L19-L37

Maybe:

Also:

I addition to existing overview text, users can be directed to hover over the status icon to learn more about the current status.

I've wanted to clean up that entire paragraph a few times already. Maybe we adjust it to something like:

You may optionally create a list of trusted servers for Federation sharing. Trusted servers will share account address books and enable auto-completion in share dialogs. Granting trusted server status and exchanging account directories, however, is not necessary in order to create federated shares. Hover (or click the status icon?) next to a configured trusted server to learn more about the current status.

Should also consider cleaning up the presentation a bit under /settings/admin/sharing (these are fairly inter-related to each other):