Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
theme system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
1 Jun 2013 at 08:23 UTC
Updated:
29 Jul 2014 at 22:27 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
helga.cheberakha commentedWDG (Ukraine,Kharkov) want to implement this on Code Sprint UA.
Comment #2
helga.cheberakha commentedComment #3
helga.cheberakha commentedComment #4
helga.cheberakha commentedComment #5
star-szrThanks for working on this @helga.chb! Looks like a great start :)
Trailing whitespace here, should be removed per http://drupal.org/coding-standards#indenting. Spotted with Dreditor.
This mode change should not be part of the patch.
Comment #6
chrisjlee commentedMoving 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.
Comment #8
chrisjlee commentedForgot to change the default.settings file mode
Comment #10
star-szr#8: 2009658-menu-replace-theme-w-drupal_render-8.patch queued for re-testing.
Comment #11
azinoman commentedI review the patch and uploaded a menu block and everything looks good. This patch works!
Comment #12
alexpottCommitted b6eb3d4 and pushed to 8.x. Thanks!