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.
This is seen during a fresh Drupal 6.12 install in both Firefox 3.0.10 and Internet Explorer 8.
Comments
Comment #1
mcrittenden CreditAttribution: mcrittenden commentedThis is still a problem. Reproduced in Chrome 5.
Comment #2
mcrittenden CreditAttribution: mcrittenden commentedComment #3
Caligan CreditAttribution: Caligan commentedVerified still a problem in 6.19 on Firefox 3.6 and IE 8. Patch fixes it in both.
Comment #5
daniels220 CreditAttribution: daniels220 commentedReroll, see if that fixes it.Never mind, D6 patch... The fix makes some sense, although it's a little odd that LIs would need padding-left? Normally they're list-style-position:outside; meaning that it's the [UO]L that needs the padding.
Anyways, still needs review from someone who can verify the fix. I'm not familiar enough with Drupal to test it.
Comment #6
pillarsdotnet CreditAttribution: pillarsdotnet commentedAnother approach. The styles for menu items should not be applied to bullets within the content block.
Comment #7
pillarsdotnet CreditAttribution: pillarsdotnet commentedComment #9
pillarsdotnet CreditAttribution: pillarsdotnet commentedGratuitous re-roll; no code changes; still works.
Comment #10
Gábor HojtsyBugs are fixed first in the current development version. I assume same bug applies to Garland in all Drupal versions.
Comment #11
pillarsdotnet CreditAttribution: pillarsdotnet commentedI'm not entirely sure that the bug exists within the d8 version of Garland, which is very different from the d6 version.
Nonetheless, here is a forward-port of the patch:
Comment #12
pillarsdotnet CreditAttribution: pillarsdotnet commented#11: garland_style_css-no_menu_styles_in_content_block-485414-11.patch queued for re-testing.
Comment #13
jhoodGarland was removed from core in #911054: Remove Garland from Core
Comment #14
xjm#11: garland_style_css-no_menu_styles_in_content_block-485414-11.patch queued for re-testing.
Let's see if the previous patch applies to 7.x.
Comment #15
xjmPatch applies to D7. Now let's confirm that the css bug is reproducible in D7 and is fixed by this. I'd suggest screenshots before and after applying the patch.
Note that the D7 installer will not have this screen and will not be in Garland, so you will need to test with a different nested list. Edit: There might be a specific inheritance that causes it as well.
Comment #16
tom friedhof CreditAttribution: tom friedhof commentedThis is no longer an issue in D7. This was already fixed in garland for Drupal 7. Nested lists are no longer an issue.
Using the same html that is output in D6 that creates this issue and pasting it in a basic page on a vanilla install of Drupal 7 shows that nested lists are no longer an issue. Attached is the screenshot showing that this is fixed without needing to apply any patch at all.
Comment #17
philosurfer CreditAttribution: philosurfer commented#16 +1
Comment #18
bvirtual CreditAttribution: bvirtual commented#16 +1
Comment #19
pillarsdotnet CreditAttribution: pillarsdotnet commentedSo it appears that this problem only exists in D6. Re-uploading the patch from #9.
Comment #20
philosurfer CreditAttribution: philosurfer commented#19 not working...
Comment #21
xjmUm. How is it not working, and if it is not working, then why is it RTBC? :)