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

Reduce amount of text in updater settings section to make it easier to understand #172

Closed
jancborchardt opened this issue Sep 11, 2015 · 14 comments
Milestone

Comments

@jancborchardt
Copy link
Member

Similar to #170

There’s still a lot of text in the updater section:
u

We should think about how to reduce that and make it simpler. @owncloud/designers @te-online @VicDeo thoughts?

For example:

  • the »Please note update is possible only to a version greater than the current one etc…« block of text can be cut and made into a tooltip of an »i« info button next to the dropdown.
  • same for the »Updates will be available within a few days bla bla … if we need that clarification at all
  • the whole updater backup note is strange – what is the backup function good for if a separate backup should be done anyway? Sure it could fail, but yeah …
@karlitschek
Copy link

Yes. I agree!

@jancborchardt jancborchardt added this to the 8.2-current milestone Sep 11, 2015
@jancborchardt
Copy link
Member Author

Setting to 8.2 cause we’re ambitious. @VicDeo @te-online does anyone of you want to work on that? :)

@VicDeo
Copy link
Member

VicDeo commented Sep 11, 2015

@jancborchardt I'm busy with core before freeze.
As for same for the »Updates will be available within a few days bla bla … if we need that clarification at all I will left you with a pleasure to respond to each and every WHY? I? DON'T? SEE? UPDATE? on your own. ;)
They are appearing after each release here and in core

@VicDeo
Copy link
Member

VicDeo commented Sep 11, 2015

latest are
#153
#156
#160

@jancborchardt
Copy link
Member Author

@VicDeo ah ok! Then still we can reword to:

(Updates will be available a few days after the update announcement.)

In parentheses, on the same line as the »Up to date.« And the info about »Last checked on 9 September etc« can be a tooltip on the »Up to date.« text. Ok?

@te-online
Copy link
Contributor

@jancborchardt @karlitschek I would love to reword the messages and also make them a bit more fun to read (a bit less technical). I'll see what I can do today or tomorrow. I think your suggestion in #172 (comment) is a really good start.

Concerning the note @jancborchardt: It it very important to understand that an updater backup does not include any owncloud-storage-data! In my opinion, this has to be absolutely clear, because you would expect the feature to do a complete backup otherwise (see #165). By the way: I introduced that here #40 and no-one complained – yet, I agree that there is too much text in general.

@te-online
Copy link
Contributor

@karlitschek @jancborchardt @VicDeo
What do you think about this? (Please ignore that I did this mockup with 8.1 where there are still two separate sections. 😉)

bildschirmfoto 2015-09-12 um 11 46 12

I would like to add the »Checked on...« and maybe the »Backup directory« as a tooltip. Is there any visual hint you could place for that? The »i« is always for the links to documentation, I think?

@karlitschek
Copy link

nice improvement IMHO! 👍

@karlitschek
Copy link

what do you think @VicDeo @jancborchardt ?

@jancborchardt
Copy link
Member Author

@te-online yess, awesome! :) Is there a pull request?

@te-online
Copy link
Contributor

@jancborchardt No, but I will create one. 👍

Do you think people translating can catch up on so many changes at once – and do you think the information are in general accurate (e.g. the upgrade path rule)?

And one last thing: Do you know anything about the tooltips I mentioned?

te-online added a commit to te-online/updater that referenced this issue Sep 18, 2015
…le and nicer in general (see owncloud#172). Fix some very short sentences in error messages.
@ghost
Copy link

ghost commented Sep 23, 2015

@VicDeo can you review the PR please?

VicDeo added a commit that referenced this issue Sep 30, 2015
Rewrite text on admin-page to reduce it … (#172)
@ghost
Copy link

ghost commented Oct 1, 2015

can we close this one?

@te-online
Copy link
Contributor

I hope so. ☺️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants