Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 UTC on 18 March 2024, to get $100 off your ticket.
Comment | File | Size | Author |
---|---|---|---|
#2 | suppress-admin-menu-285020-2.patch | 1.59 KB | pwolanin |
#1 | suppress-admin-menu-285020-1.patch | 1.57 KB | pwolanin |
Comments
Comment #1
pwolanin CreditAttribution: pwolanin commentedThis patch obviously depends on the final form of the one in the other issue, but works with the patch there in #7.
Comment #2
pwolanin CreditAttribution: pwolanin commentedand simpler code for #8 in the other issue.
Comment #3
pwolanin CreditAttribution: pwolanin commentedI've testing this patch and it works - don't know if you want someone independent to set as rtbc.
Comment #4
redndahead CreditAttribution: redndahead commentedQuickly did a manual patch and this seems to be working fine. Can't say if it needs a reroll, but looks like it will be ok.
Thanks pwolanin
Comment #5
merlinofchaos CreditAttribution: merlinofchaos commentedThis is committed. I think we need a better way to do this in the long term, but Drupal 6 may not ever have it. I'm not really happy about advanced help or admin menu having to know about the other for this to work; that said, there's no way around it, and I agree that having that menu on these pages is bad. So we'll go with this for now, and leave an asterisk on it.
Comment #6
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.