Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review wording with VFS #7101

Open
jnweiger opened this issue Mar 19, 2019 · 4 comments
Open

Review wording with VFS #7101

jnweiger opened this issue Mar 19, 2019 · 4 comments
Assignees
Labels
Design & UX feature:vfs native virtual files and placeholder implementation
Milestone

Comments

@jnweiger
Copy link
Contributor

Seen with 2.6.0daily 20190313 on win10:

When VFS is enabled, and a new folder appears on the server, the client still says 'downloaded.'
that is misleading, if default is online only.
grafik

When I select 'Immer auf diesem Gerät behalten' the client pops up a notification with exactly the same wording.

Making the folder virtual again with 'Speicherplatz freigeben' the message is a vague 'updated':
grafik

@guruz guruz added this to the 2.6.0 milestone Mar 20, 2019
@guruz guruz changed the title review wording with VFS Review wording with VFS Mar 20, 2019
@ckamm ckamm self-assigned this Mar 25, 2019
@ckamm ckamm added the feature:vfs native virtual files and placeholder implementation label Mar 25, 2019
@ckamm
Copy link
Contributor

ckamm commented Mar 25, 2019

Indeed, I had updated the sync protocol entries in f786ef1 but haven't adjusted the notifications.

Now, what would be a good text for notifications though? Currently each action-kind in (new, deleted, updated, renamed, conflict, error) triggers a single notification that mentions the first such file by name. I wonder whether that's all that useful? But let's leave that for a different ticket.

For now I propose to adjust the wording for the "new" case to not say "downloaded". Continue to have "new-virtual" grouped with "new-regular". But put "newly-hydrated" into "updated", where "newly-dehydrated" already is.

ckamm added a commit that referenced this issue Mar 25, 2019
These files may very well just be new virtual files that were explicitly
*not* downloaded.
ckamm added a commit that referenced this issue Mar 25, 2019
Previously it'd be NEW(ItemTypeFile), but now it has changed to be
SYNC(ItemTypeVirtualFileDownload) which allows better classification.
ckamm added a commit that referenced this issue Mar 26, 2019
Previously it'd be NEW(ItemTypeFile), but now it has changed to be
SYNC(ItemTypeVirtualFileDownload) which allows better classification.
ckamm added a commit that referenced this issue Mar 28, 2019
These files may very well just be new virtual files that were explicitly
*not* downloaded.
ckamm added a commit that referenced this issue Mar 28, 2019
Previously it'd be NEW(ItemTypeFile), but now it has changed to be
SYNC(ItemTypeVirtualFileDownload) which allows better classification.
@ckamm ckamm added ReadyToTest QA, please validate the fix/enhancement and removed PR available labels Mar 28, 2019
@HanaGemela
Copy link
Contributor

Adding a new virtual file says 'added'
Adding a new always downloaded file says 'added'
The above was done by adding a new file via the webUI

When I free up local space or make files always available locally, the message is still updated

@jnweiger
Copy link
Contributor Author

'Immer auf diesem Gerät behalten' == 'Make alway available locally'
'Speicherplatz freigeben' == 'Free up local space'

Please do not invent new terms. I'd suggest to re-use the existing vocabulary as much as possible.
E.g. like this:

  1. Server adds files, client "downloads" placeholders
    Notification "Foo.txt and 13 other files were added."
  2. user changes to 'Make always available locally'
    Notification "Foo.txt and 13 other files are now always available locally."
  3. user changes to 'Free up local space'
    Notification "Foo.txt and 13 others are now dehydrated."
    or
    Notification "Foo.txt and 13 others are now virtual."

We already have a redundancy of terms for e.g. hydrated=physical and dehydrated=virtual,
I personally would prefer the pair (physical/virtual) over (hydrated/dehydrated).

@HanaGemela HanaGemela modified the milestones: 2.6.0, 2.6.1 Jul 23, 2019
@HanaGemela HanaGemela removed the ReadyToTest QA, please validate the fix/enhancement label Jul 23, 2019
@ckamm
Copy link
Contributor

ckamm commented Jul 24, 2019

@HanaGemela Your test confirms what I intended to do, see #7101 (comment). Thanks for moving further work on this to 2.6.1.

@jnweiger I agree. I've tried to keep "hydrated"/"dehydrated" (comes from cfapi) completely internal, users shouldn't see these terms. The reason cfapi introduced these is that there every file is a "placeholder" file, no matter whether hydrated or dehydrated.

@michaelstingl michaelstingl modified the milestones: 2.6.1, 2.6.2 Nov 26, 2019
@TheOneRing TheOneRing modified the milestones: 2.6.2, 2.9.0 Mar 17, 2021
@TheOneRing TheOneRing modified the milestones: 2.11, Backlog Feb 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design & UX feature:vfs native virtual files and placeholder implementation
Projects
None yet
Development

No branches or pull requests

6 participants