This is seen during a fresh Drupal 6.12 install in both Firefox 3.0.10 and Internet Explorer 8.

CommentFileSizeAuthor
#19 garland_style_css-no_menu_styles_in_content_block-485414-19.patch913 bytespillarsdotnet
#16 testing-nested-lists.jpg157.97 KBtom friedhof
#11 garland_style_css-no_menu_styles_in_content_block-485414-11.patch943 bytespillarsdotnet
#9 garland_style_css-no_menu_styles_in_content_block-485414-9-d6.patch913 bytespillarsdotnet
#6 garland.list-style.patch446 bytespillarsdotnet
#5 bullet_overlap_485414_D6.patch316 bytesdaniels220
#3 drupal-485414-1.patch523 bytescaligan
Drupal_6.12_bullets_glitch.png69.7 KBthreexk

Comments

mcrittenden’s picture

mcrittenden commented
Version: 6.12 » 6.x-dev

This is still a problem. Reproduced in Chrome 5.

mcrittenden’s picture

mcrittenden commented
Issue tags: +Novice
caligan’s picture

caligan commented
Status: Active » Needs review
StatusFileSize
new drupal-485414-1.patch523 bytes

Verified still a problem in 6.19 on Firefox 3.6 and IE 8. Patch fixes it in both.

Status: Needs review » Needs work

The last submitted patch, drupal-485414-1.patch, failed testing.

daniels220’s picture

daniels220 commented
Status: Needs work » Needs review
StatusFileSize
new bullet_overlap_485414_D6.patch316 bytes

Reroll, 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.

pillarsdotnet’s picture

pillarsdotnet commented
StatusFileSize
new garland.list-style.patch446 bytes

Another approach. The styles for menu items should not be applied to bullets within the content block.

pillarsdotnet’s picture

pillarsdotnet commented
Title: Bullets overlap their text in install page » Bullets overlap their text in install page (Styles for menu items should not be applied within the .content block)

Status: Needs review » Needs work

The last submitted patch, garland.list-style.patch, failed testing.

pillarsdotnet’s picture

pillarsdotnet commented
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new garland_style_css-no_menu_styles_in_content_block-485414-9-d6.patch913 bytes

Gratuitous re-roll; no code changes; still works.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Version: 6.x-dev » 8.x-dev
Status: Reviewed & tested by the community » Needs review

Bugs are fixed first in the current development version. I assume same bug applies to Garland in all Drupal versions.

pillarsdotnet’s picture

pillarsdotnet commented
StatusFileSize
new garland_style_css-no_menu_styles_in_content_block-485414-11.patch943 bytes

I'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:

pillarsdotnet’s picture

pillarsdotnet commented

#11: garland_style_css-no_menu_styles_in_content_block-485414-11.patch queued for re-testing.

jhood’s picture

jhood
he/him
Primary language English
Location Ohio, United States
commented
Version: 8.x-dev » 7.x-dev

Garland was removed from core in #911054: Remove Garland from Core

xjm’s picture

xjm
she/her
Primary language English
commented

#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.

xjm’s picture

xjm
she/her
Primary language English
commented
Issue tags: +Needs manual testing

Patch 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.

tom friedhof’s picture

tom friedhof commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new testing-nested-lists.jpg157.97 KB

This 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.

philosurfer’s picture

philosurfer commented

#16 +1

bvirtual’s picture

bvirtual commented
Status: Reviewed & tested by the community » Needs review

#16 +1

pillarsdotnet’s picture

pillarsdotnet commented
Version: 7.x-dev » 6.x-dev
StatusFileSize
new garland_style_css-no_menu_styles_in_content_block-485414-19.patch913 bytes

So it appears that this problem only exists in D6. Re-uploading the patch from #9.

philosurfer’s picture

philosurfer commented
Status: Needs review » Reviewed & tested by the community

#19 not working...

xjm’s picture

xjm
she/her
Primary language English
commented
Status: Reviewed & tested by the community » Needs review

Um. How is it not working, and if it is not working, then why is it RTBC? :)

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.