matrix-react-sdk icon indicating copy to clipboard operation
matrix-react-sdk copied to clipboard

Adding description for Clear Cache and Reload Button in Settings

Open tawfiek opened this issue 4 years ago • 16 comments

Fixes vector-im/element-web#5800 Screenshot from 2021-03-26 18-54-23


This PR currently has none of the required changelog labels.

A reviewer can add one of: T-Deprecation, T-Enhancement, T-Defect, T-Task to indicate what type of change this is, or add Type: [enhancement/defect/task] to the description and I'll add them for you.

tawfiek avatar Mar 26 '21 16:03 tawfiek

I can provide an Arabic translation for the new added description, should I do this into the same PR ?!

tawfiek avatar Mar 26 '21 16:03 tawfiek

should I do this into the same PR

No as it'll cause conflicts in Weblate, all translations must be done from Weblate only

t3chguy avatar Mar 26 '21 16:03 t3chguy

I can provide Arabic translation for the new added description, should I do this into the same PR ?!

No, the translations are done through Webplate

SimonBrandner avatar Mar 26 '21 16:03 SimonBrandner

so should I delete what I added for en_US ?!

tawfiek avatar Mar 26 '21 17:03 tawfiek

so should I delete what I added for en_US ?!

That should have been autogenerated using yarn i18n

Since you didn't do that you have globe-with-meridians-i18n failing

SimonBrandner avatar Mar 26 '21 17:03 SimonBrandner

Design review

This is much easier to understand for me than the current version. Just the copy would ideally be slightly clearer.

  • New copy
    • Clear cache and reload Element
    • This will delete all cached data, reload Element and automatically log you in. No key backups will be deleted.

Is it called key backups or backup keys in Element? Is it more useful to just say generally 'no security information will be deleted?

niquewoodhouse avatar Jul 12 '21 19:07 niquewoodhouse

and automatically log you in.

This is misleading, as logging you in would make you expect the New sign in toasts on other devices which will not happen here. Your device ID (unique to each login) will not change

t3chguy avatar Jul 12 '21 19:07 t3chguy

"No key backups will be deleted" is hard to understand for those not in the know, I'd say "Your end-to-end-encryption keys will not be affected" would work better

ShadowJonathan avatar Jul 12 '21 19:07 ShadowJonathan

"No key backups will be deleted" is hard to understand for those not in the know, I'd say "Your end-to-end-encryption keys will not be affected" would work better

That's definitely easier for me to understand (and I'm not in the know) but do we refer to them as end-to-end-encryption keys anywhere else? I just wonder if you'd then be looking for that exact phrase in other areas of Element, and so it further complicates all the names we have/use. Eg "I'm looking for my e2e keys" Would rather use the exact same phrases or something so generic it doesn't sound potentially specific e.g:

"Your encryption settings will not be deleted or affected in anyway." But I don't know much about this area of Element.

niquewoodhouse avatar Jul 13 '21 08:07 niquewoodhouse

@tawfiek Please update the PR with the copy suggestions above. This one looks almost good to go 😊

Bundling all of the points above, perhaps:

  • Clear cache and reload Element
  • This will delete all cached data and reload Element (you will still be signed in). Your encryption settings and keys will not be deleted or affected in anyway.

MadLittleMods avatar Sep 15 '21 00:09 MadLittleMods

Yes, this much cleaner and very obvious, I'll add this to my schedule for the next week. I think that I'll need to go through all the building and configuration stuff to run Matrix, it's been a long time since I contributed here. But it'll be my pleasure to finish what I started with you guys 😇

tawfiek avatar Dec 11 '21 13:12 tawfiek

No login required

I just noticed this, maybe it should be "No re-login required" or something along the lines?

Maybe "You'll stay logged in" is better.

ShadowJonathan avatar Dec 11 '21 15:12 ShadowJonathan

@tawfiek any chance you could update this as requested? Thanks!

andybalaam avatar Aug 25 '22 15:08 andybalaam

Feel free to pick this up when you get to it, but I'll mark it as draft for now so that it drops off our radar since it's not ready.

andybalaam avatar Sep 15 '22 15:09 andybalaam

Please mark it as non-draft if you get this ready for another review. Thanks!

andybalaam avatar Sep 15 '22 15:09 andybalaam

Hello, I need small assistance to get back into the community, How to join Element channels?

tawfiek avatar Nov 30 '22 15:11 tawfiek