Lets have only one "save" button rather than two. We already know that this occasionally throws off users, and it ends up being a primary button mania.

CommentFileSizeAuthor
#5 interdiff.txt752 bytesswentel
#5 2142637-5.patch1.4 KBswentel
#2 2142637-2.patch678 bytesswentel
Screen Shot 2013-11-23 at 14.43.10.png123.64 KBBojhan

Comments

Bojhan’s picture

Bojhan commented
Issue summary: View changes
swentel’s picture

swentel commented
Status: Active » Needs review
StatusFileSize
new 2142637-2.patch678 bytes
Bojhan’s picture

Bojhan commented
Status: Needs review » Reviewed & tested by the community

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: 2142637-2.patch, failed testing.

swentel’s picture

swentel commented
Status: Needs work » Needs review
StatusFileSize
new 2142637-5.patch1.4 KB
new interdiff.txt752 bytes

Fixing the tests

swentel’s picture

swentel commented
Status: Needs review » Reviewed & tested by the community
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Thanks for doing this! This confused the crap out of me yesterday when I was testing the button patch. :)

+++ b/core/modules/field_ui/field_ui.module
@@ -200,6 +200,8 @@ function field_ui_form_node_type_form_alter(&$form, $form_state) {
+    // Hide the 'Save content type' button.
+    $form['actions']['submit']['#access'] = FALSE;

I was rather surprised that this was the patch until I realized that save button is provided by node module and field UI is optional.

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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