Many nodes under drupal support separate 'Preview' and 'Submit' functionality. Some require 'Preview' prior to 'Submit'.

The Submit button, on previewed pages, appears at the bottom of the markup text. This makes sense when you are still making changes to a previewed page. However, when you like the way it looks, you would rather not have to scroll down to click it.

I propose adding a second 'Submit' button at the bottom of the rendered page, and above the user's source for that text, on previewed pages, so that users can use either button. The functionality would of course be the same for both buttons. Just a nit, but would be nice for users.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JonBob’s picture

A related problem is that some users, we've found, fail to realize they're seeing a preview rather than a published node and click away rather than scrolling down and clicking Submit. It doesn't help that we display the node links along with the preview, enabling another way to click away from the task at hand.

moshe weitzman’s picture

those node links are gone, but this issue is still valid and adding buttons on top still sounds like a good idea to me.

RobRoy’s picture

I think if nothing else, having something like this in theme_node_preview() would be good. And a submit button on top is good too! +1

  drupal_set_message(t('Your changes have not yet been saved. Take a look at the preview of your post below and if you are satisfied, save the changes by clicking Submit button.'));

Side-issue #1: On a side note, it doesn't seem that any drupal_set_message() in theme_node_preview() is being displayed while previewing. Can anyone confirm this before I submit a bug?

Side-issue #2: Also, I just previewed this post and I don't see a preview. What's the deal with that?

RobRoy’s picture

Side-issue #1: My bad. The message is set when previewing, but #2 is still valid: Preview on this page is really not working.

LAsan’s picture

Version: » 7.x-dev
Status: Active » Fixed
keith.smith’s picture

Status: Fixed » Active

I don't see any reason this is "fixed".

okeedoak’s picture

A good idea worth exploring further...

rlee15’s picture

Has there been any update to the issue of the submit button being a the bottom of the page, requiring a lot scrolling down? I'm not seeing anything posted on it since Summer 08, but it's still a prevalent issue in creating our site. I'd love to see a submit button on top. Thanks!

Anonymous’s picture

Would it be sufficient to preface the preview with text that says something like "Here is a preview of your @node_type. If this appears acceptable, scroll to the bottom and press submit. Otherwise continue with your modifications. This @node_type will not be created until you click submit."

equinox’s picture

Just adding my two bits of a vote for options supplied with Content type to control whether preview is required, and to allow the repositioning/duplication of the Submit button. Drupal gives me carpel tunnel from all the scrolling one must do! I'm just kidding, but Submit right next to the preview would be a really nice feature.

saml’s picture

I would like to stress the need and usability of adding a submit button at the top of node edit form.

The amount of scrolling needed in Drupal UI is one of the main complaints I get from customers, but scrolling could actually be eliminated 100% for a big portion of edits (when just adding a title and content) by always supplying with another save button at the top of the page. This is how many other systems I've seen has it, and is what feels most sensible IMO.

deekayen’s picture

subscribe (yes, I know it's annoying)

deekayen’s picture

Assigned: Unassigned » deekayen
Status: Active » Needs review
FileSize
22.34 KB
1.68 KB

Patch adds a Save button between the preview and the node form.

deekayen’s picture

Just as another option, this patch includes the dsm from #3.

deekayen’s picture

Ignore #14. This patch puts the message in theme_node_preview() like #3 originally suggested. #13 is still a good message-free option.

Status: Needs review » Needs work

The last submitted patch failed testing.

deekayen’s picture

Status: Needs work » Needs review
FileSize
104.52 KB
2.01 KB

Re-roll for pifr with screenshot for visual assistance.

screenshot of new preview button

moshe weitzman’s picture

Status: Needs review » Needs work

I like this simple change a lot. Lets commit something here and get on with our lives. It has been over 5 years!

I would think this is not needed. Does save still work without it? '#submit' => array('node_form_submit'),

deekayen’s picture

Status: Needs work » Needs review
FileSize
2.03 KB

No, save does not still work without it. The "original" save button on the bottom has the same #submit item as well.

I tweaked it a bit to make the message at warning level instead of status (that means it'd be in yellow in Garland instead of bulleted in green).

moshe weitzman’s picture

Status: Needs review » Reviewed & tested by the community

I'm unsure if a warning is appropriate here but thats really a minor point. RTBC.

deekayen’s picture

Whoever commits it can pick either #17 or #19 then. They're identical except for the new notice's status level.

Dries’s picture

Status: Reviewed & tested by the community » Closed (duplicate)

This is addressed by #472126: D7UX: Move buttons to right area, add content and meta selector. I like the direction that #472126 is going in better, so let's re-group over there.

catch’s picture

Status: Closed (duplicate) » Needs review

Re-opening this - that other issue is stalled, this one deals with only a subsection of what's discussed there and I don't think they really conflict.

arianek’s picture

Issue tags: +Usability

adding tags

amc’s picture

I hate to add something rather than getting this committed already, but another possibility is a greyed "PREVIEW" watermark over the content, similar to the way the Zen theme does with "UNPUBLISHED". That way, even if they skip over the green message (as many do out of habit), they'll still notice the watermark.

Status: Needs review » Needs work

The last submitted patch failed testing.

EvanDonovan’s picture

Can someone reroll this? Looks like it would be a great improvement.

arianek’s picture

so... i was going to reroll this, but it didn't apply cleanly to the current version of HEAD for me, i'm assuming there has been a lot of changes since the patch was originally posted:

patching file modules/node/node.pages.inc
Hunk #1 succeeded at 115 with fuzz 2 (offset 3 lines).
Hunk #2 FAILED at 380.
1 out of 2 hunks FAILED -- saving rejects to file modules/node/node.pages.inc.rej
patching file modules/node/node.test
Hunk #1 succeeded at 281 with fuzz 2 (offset 27 lines).
fletch11’s picture

subscribe

yoroy’s picture

Version: 7.x-dev » 8.x-dev
Assigned: deekayen » Unassigned

Still valid, we want to further improve content creation workflows for D8.

jhedstrom’s picture

Version: 8.0.x-dev » 8.1.x-dev
Issue summary: View changes

Version: 8.1.x-dev » 8.2.x-dev

Drupal 8.1.0-beta1 was released on March 2, 2016, which means new developments and disruptive changes should now be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

anoopjohn’s picture

This issue is not relevant anymore after the implementation of #1510532: [META] Implement the new create content page design. I am marking this as duplicate. I have also added the referenced issues as related issues.