Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Comment | File | Size | Author |
---|---|---|---|
#8 | interdiff-2009658-8.txt | 117 bytes | chrisjlee |
#8 | 2009658-menu-replace-theme-w-drupal_render-8.patch | 1.62 KB | chrisjlee |
#6 | 2009658-menu-replace-theme-w-drupal_render-6.patch | 1.73 KB | chrisjlee |
#6 | interdiff-2009658-6.txt | 1.23 KB | chrisjlee |
#2 | drupal_remove_theme_menu_module-2009658-2.patch | 1.75 KB | helga.cheberakha |
Comments
Comment #1
helga.cheberakha CreditAttribution: helga.cheberakha commentedWDG (Ukraine,Kharkov) want to implement this on Code Sprint UA.
Comment #2
helga.cheberakha CreditAttribution: helga.cheberakha commentedComment #3
helga.cheberakha CreditAttribution: helga.cheberakha commentedComment #4
helga.cheberakha CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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!