Originally pointed out by carlos8f at #686060-22: Explain that the Update manager only works if you have FTP or SSH access to your server but this is really unrelated to the rest of what that issue is trying to address...
The "Install" button at admin/reports/updates/install, admin/modules/install and admin/appearance/install (all of which point to the same menu callback -- update_manager_install_form()) should really be labeled "Continue".

Clicking the button doesn't necessarily trigger installation. Although it does (or at least tries to) when all the files are owned by the same user. So, maybe the button should be renamed conditionally based on the file ownership stuff? I guess it doesn't really hurt that much to always call it "Continue"...
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 976232-3.update-manager-install-continue-button.patch | 2.12 KB | dww |
| #1 | 976232-1.update-manager-install-continue-button.patch | 728 bytes | dww |
Comments
Comment #1
dwwHere's the trivial always-rename-the-button approach. Also I forgot to tag...
Comment #3
dwwOh right, some tests are trying to press that button. ;)
Comment #4
Bojhan commentedIt does hurt too much just calling it "Continue", this because people will be constantly looking for that button.
Comment #5
dww@Bojhan: Thanks for the review, although I don't know exactly what you're talking about. ;) You're saying that if the text of the button says "Continue" instead of "Install", even though it's the only button on the page, that it's going to disappear from view and no one will be able to proceed? I know you're a usability expert, but that seems to be a rather far-fetched assertion. Can you please explain your concerns?
Sounds like "needs work" is the wrong status from your perspective. If you want it to just stay "Install" we should set this to "won't fix". If you're okay with changing "Install" (since that's not what the button does) to something else, can you propose an alternative? I can't really imagine another option that anyone would be happy with, but maybe you can come up with something.
Thanks,
-Derek
Comment #6
Bojhan commentedSorry for being so short, but of course I don't think that people will just give up and navigate away. My point was, that people are looking for the word "install" - this is the trigger word here.
The problem is; by saying "Continue" we indicate there is a next step, but give the user no clue what is next (even by common sense). This to the contrary of other uses of "Continue" (the most usable one in the installer) which are part of a flow, and thereby list out the steps - where you are in the process.
So by removing the word install (which is a clear word), we are moving it to "Continue" which will leave the user wondering for a second what is happening next. So now that we are choosing between surprise the user and confuse the user, I am opting for not choosing at all, because its all bad choices - and fixing it properly in D8 (e.g. we need an in-overlay wizard indicator).
Comment #7
dwwThanks for clarifying. So now what, won't fix?
Comment #8
Bojhan commentedI think push it to D8, we can fix this by introducing a wizard indicator for the overlay.
Comment #9
carlos8f commentedThen, in that case, we should explain what's going to happen. Part of that is already slated for #686060: Explain that the Update manager only works if you have FTP or SSH access to your server. So based on whether the user needs to be redirected to authorize.php, the button should say "Continue", or "Install". D8 is nonsense for this issue; update manager may not even make it to D8.
Comment #10
EvanDonovan commentedI agree with @carlos8f on this:
Comment #11
bfroehle commentedThe logic here would need to be:
Workflow will still remain problematic on shared hosts though until #1008328: Uniqueify update-cache and update-extraction directories to prevent "Permission denied" errors or similar is solved, as another user's Drupal installation might create the update_extract directory causing us to select the wrong branch of the if statement.
Comment #12
jhodgdonThis is really a duplicate of #1207354: Install new module v/s Add new module and probably #2577407: Action of uploading module/theme files should consistently be called "Add", not "Install" too? Not sure which one(s) we should keep open but probably not all 3. I will comment on the other 2 issues too.
Comment #13
dwwThis was the earliest, so we should probably keep this one, right? 6 digit nid FTW. ;)
Comment #14
jhodgdonPerhaps, but we should make sure it covers all the ground of the others, and includes the info about why this is such a usability SNAFU, etc. Plus, which has a good patch?
The issue title here has an opinion of how to solve the usability problem; the others may have a different opinion about how to solve the same problem?
Comment #15
jhodgdonActually I think this one is not a straight duplicate of the other two. The other two issues are about what you see on admin/modules. This one is about what you see after you click the button that currently says "Install new module" and should probably say "Add new module".
Also, the other two issues have had a lot more comments and usability team involvement. If anything, we should close this one, and incorporate it into the others. We don't always choose to close the newer ones as duplicates.
Comment #16
dwwFair points, @jhodgdon, thanks. Re-reading this now (and skimming the others), I agree it's not a duplicate. For scope, we should keep this separate. It'd be easier to deal with this as a stand-alone fix, instead of getting embroiled in all the changes on admin/modules itself, which are only tangentially related to this since they also involve the word "install".
Bumping to at least 8.9.x for now. TBD.
We've only got 4 months before this becomes a decade-old bug. ;) Thanks for resurfacing it!
Cheers,
-Derek
Comment #17
jhodgdonSee #3158126-5: Change text to "You must manually enable newly added modules, here. Steps here." -- Upon further reflection, I really think we should consolidate these 3 issues into 1, because they're all steps of the same process. See also #2577407-173: Action of uploading module/theme files should consistently be called "Add", not "Install" (which is the issue I think we should consolidate on) -- I think the next step would be to bring this process in to a Usability group meeting, get the group to decide what to do.
The patch to fix all 3 of these will only affect a few lines of code -- it's just UI text -- and I really think thinking of it as the usability of the whole process is the way to go.
Comment #18
jhodgdonThere's a meta issue that should be this issue's parent.
Comment #19
jhodgdonFYI -- on the parent issue #2888657: [meta] Less confusing and more consistent wording needed in module/theme add/install/update the Usability team is discussing the UI text for this and about 10 other issues. Until decisions are made, making another patch here would probably be premature.
Comment #20
jhodgdonWe now have a plan to fix this on #2577407: Action of uploading module/theme files should consistently be called "Add", not "Install", so marking this issue as a duplicate. I've asked people on this issue to be credited there.