I want to be able to delete nodes, but don't want the 'Preview' button showing. I'd therefore like to request that the 'Hide standard buttons' setting be separated into a checkbox per button (including #550982: Enable removal of Ubercart's 'Save and continue' button).

I'm actually wondering about the purpose of hiding the Delete button at all. Seems that if people didn't want a delete button, they'd find a way to remove it from all content types, not just Multistep-enabled ones...

CommentFileSizeAuthor
#2 multistep-560748.patch1.77 KBvkareh

Comments

vkareh’s picture

I guess you're right. You can always set the node permissions to allow deletion or not. Being able to hide the "Preview" button seems to be something that should already be part of core, but I'll leave that for those that know. :P

I don't really see a need for keeping Ubercart's "Save and continue" button, since the Save button does the exact same thing. Duplicate buttons would clutter the form. What I might do is just leave it to the admin to decide. There might even be a case where you don't want the "Previous" button, or even the "Save"... I dunno... I can just throw in a bunch of checkboxes and let each website administrator handle their own forms. :P

vkareh’s picture

StatusFileSize
new1.77 KB

This quick patch will tentatively take care of allowing the users to choose whether they want to remove the Preview button. The Delete button will be handled by the node permissions; in other words, multistep won't interfere with the Delete button. Ubercart's "Save and continue" button will be removed regardless of any choice, since it has duplicate functionality.

Eventually I might add an option to individually select which buttons to show/remove, but for now this should take care of the Delete/Preview issue.

Please test and let me know. (Make sure you do a checkout of the latest cvs snapshot).

vkareh’s picture

Status: Active » Needs review

Sorry, forgot to change status.

Anonymous’s picture

Status: Needs review » Closed (duplicate)

Code from this patch now included in this issue: #530848: Revisions and Published state

vkareh’s picture

Status: Closed (duplicate) » Fixed

Rolled the patch into the latest development snapshot.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.