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?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Garret Voorhees’s picture

FileSize
53.09 KB

I'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.

mrf’s picture

Assigned: Unassigned » zeropx
zeropx’s picture

Status: Active » Needs review
FileSize
438 bytes

I 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.

Bojhan’s picture

Please make sure, we dont use red for this :)

zeropx’s picture

I 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

populist’s picture

Project: Panopoly » Apps
Assigned: zeropx » Unassigned

Why 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.

nedjo’s picture

Status: Needs review » Needs work

If 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).

febbraro’s picture

I'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.