Right now, "Install Apps" and "Skip this step" are both at the same visual importance, and since "Skip this step" is on the right, it can tempt you to click it thinking you're clicking "next" when you're not reading.
Could we handle this like we handle "Confirm / cancel" elsewhere in Drupal and make the primary action (Install Apps) the button, and make the secondary action (Skip this step) a link?
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 1564954-skip_this_step-colorfix.patch | 437 bytes | zeropx |
| #3 | 1564954-skip_this_step.patch | 438 bytes | zeropx |
| #1 | panopoly_install_skip.png | 53.09 KB | Garret Voorhees |
Comments
Comment #1
Garret Voorhees commentedI'm in agreement that it shouldn't have the same button style. My suggestion would be exactly what you've said about setting the "Skip this step" text in a regular link style and possibly in red as a warning. See the attached image as a reference.
Comment #2
mrf commentedComment #3
zeropx commentedI wasn't really sure to easily change the submit button to a link with out messing with the form flow so I just added a quick bit of css that changes the visual appearance of the button it self instead to match Garret's suggestion.
I started with the confirm_form() method but the way its catching the submission and then adjusting the session details I wasn't sure how to handle that since the only thing I can provide is a path for the method.
Comment #4
Bojhan commentedPlease make sure, we dont use red for this :)
Comment #5
zeropx commentedI agree, so I submit a new patch with a color that is more suitable based on the current design. For the time being at least until Apps has been updated to accommodate.
Also, I have created a new ticket in the Apps module since this task is directly related to such.
http://drupal.org/node/1619604
Comment #6
populist commentedWhy would anyone not want to install the powerful Panopoly apps? I just removed that option from the profile entirely.
Of course, in general I think the UI/UX comments are useful for the Apps module and I am moving it to that queue.
Comment #7
nedjoIf it's going to be a link, it should be a link rather than a restyled button. For reference maybe look at the core installer approach when requirements tests fail and there is a link to try again (although there the current task is repeated rather than skipped).
Comment #8
febbraro commentedI'd be happy to review/commit this if someone can do some of the legwork to see how to get a link to move to the next install task.