Posted by dww on October 25, 2009 at 10:28pm
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | update.module |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
| Issue tags: | ui-text, Update manager, Usability |
Issue Summary
Seems like the help text on the install-new-stuff pages from Update manager should link to http://drupal.org/project/modules and/or http://drupal.org/project/themes depending on the context.
Comments
#1
Uhh, the page already has those links. ;)
I guess I just missed them. :( UI review on this page?
#2
- The title 'Available updates' seems off, as the page is about installing new things. Only new things or does that include updates to already used stuff as well?
- 'URL' is quite sparse a label. 'Install from this URL'?
- 'Paste' is then quite specific :-) No need to make explicit that it needs filling out. Suggestion " The URL to a module or theme archive to install, for example: http://etc…" (e.g. is a rather abstract abbreviation, probably well known but 'for example' is simpler and clearer).
- Don't use the word 'Drupal', the Drupal context is a given and it complicates forks/profiles (see http://drupal.org/node/501452)
- The choice between giving a URL and uploading is not communicated elegantly yet. Can we make a choice on which to show by default and give a link to show/hide the other option? Show the URL field only first, link to show the upload option, hide the URL field?
- Description for the upload button is redundant, remove it and rewrite the label to say "Upload a module or theme archive to install"
#3
Eh, the description for the URL field could just be the example if the label has the basic info already:
Install from this URL
[……………………………]
For example: http://ftp.drupal.org/files/project/projectname.tar.gz
#4
Thanks for the review. This page definitely needs work. ;)
One quick reply:
- The title 'Available updates' seems off, as the page is about installing new things. Only new things or does that include updates to already used stuff as well?
That's an artifact of this page living as an action link off the "Available updates" report. It therefore doesn't get a page title of its own. There's a big debate brewing about this whole thing, which we need to hash out. Basically, Dries committed #602490: Fix help text and action links to find the update manager pages without any review from the UI team (it was just needs review, not RTBC), but that should be reopened and potentially some sweeping changes need to happen there... The action links were originally from #538660-93: Move update manager upgrade process into new authorize.php file (and make it actually work). That was just the best I could come up with on very short time to get *something* in. But, we should really step back and figure out the IA for all these pages and exactly how we expect people to find everything. But, that's for #602490 not here. ;)
#5
Here's a stab at it. I've highlighted the text I changed in the screenshot.
#6
Great, thanks! That's better. However, it still references "Drupal" in the description for the "Install from a file". See yoroy in #2:
#7
Happy to oblige.
#8
Sorry, #7 is still a bit wonky as per yoroy's feedback from #2 regarding the upload UI. I agree we could make this a lot better if we were more fancy with JS to hide/reveal the two options or something. But, I don't want to get too crazy or we risk not making *any* improvements before alpha1. So, here's a new patch that at least cleans up the labels and descriptions a bit more:
#9
The only thing, I would work on is using a normal upload form element so, such as Attach a new file, displayed on each new comment on d.o issues.
#10
Re: #9: It is a normal upload form element. That's just how they look in safari. ;)
<?php$form['project_upload'] = array(
'#type' => 'file',
'#title' => t('Upload a module or theme archive to install'),
'#description' => t('For example: %filename from your local computer', array('%filename' => 'projectname.tar.gz')),
);
?>
So, RTBC?
Should the examples include "projectname-version.tar.gz" or something? E.g. an actual URL to use would be:
http://ftp.drupal.org/files/projects/cvs_deploy-7.x-1.x-dev.tar.gz
#11
Yes, RTBC - I dont think its really necessary to include that
#12
This was committed. :)
#13
Automatically closed -- issue fixed for 2 weeks with no activity.