Support from Acquia helps fund testing for Drupal Acquia logo

Comments

helga.cheberakha’s picture

Assigned: Unassigned » helga.cheberakha

WDG (Ukraine,Kharkov) want to implement this on Code Sprint UA.

helga.cheberakha’s picture

helga.cheberakha’s picture

Status: Active » Needs review
helga.cheberakha’s picture

Assigned: helga.cheberakha » Unassigned
star-szr’s picture

Status: Needs review » Needs work

Thanks for working on this @helga.chb! Looks like a great start :)

+++ b/core/modules/menu/menu.admin.incundefined
@@ -287,9 +287,14 @@ function theme_menu_overview_form($variables) {
+      ¶
+      $indent = array(
+        '#theme' => 'indentation',
+        '#size' => $element['#item']['depth'] - 1,
+      );
+      ¶

@@ -303,8 +308,18 @@ function theme_menu_overview_form($variables) {
+  ¶
+  $table = array(
+    '#theme' => 'table',
+    '#header' => $header,
+    '#rows' => $rows,
+    '#attributes' => array(
+      'id' => 'menu-overview',
+    ),
+  );
+  ¶

Trailing whitespace here, should be removed per http://drupal.org/coding-standards#indenting. Spotted with Dreditor.

+++ b/core/modules/menu/menu.admin.incundefined
diff --git a/sites/default/default.settings.php b/sites/default/default.settings.php
old mode 100644
new mode 100755

This mode change should not be part of the patch.

chrisjlee’s picture

Moving this along. Removed whitespace as requested in #5.

Whoops https://drupal.org/files/2009658-menu-replace-theme-w-drupal_render-5.patch . It's the same as 6. But i forgot to delete it from the list.

The last submitted patch, 2009658-menu-replace-theme-w-drupal_render-6.patch, failed testing.

chrisjlee’s picture

Forgot to change the default.settings file mode

Status: Needs review » Needs work

The last submitted patch, 2009658-menu-replace-theme-w-drupal_render-8.patch, failed testing.

star-szr’s picture

Status: Needs work » Needs review
azinoman’s picture

Status: Needs review » Reviewed & tested by the community

I review the patch and uploaded a menu block and everything looks good. This patch works!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed b6eb3d4 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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