Make WordPress Core

Opened 17 years ago

Closed 15 years ago

Last modified 9 years ago

#4449 closed defect (bug) (duplicate)

Kubrick ordered list problem for IE7

Reported by: neoen's profile neoen Owned by: djr's profile djr
Milestone: Priority: low
Severity: normal Version: 2.2
Component: Template Keywords: has-patch needs-feedback
Focuses: Cc:

Description

Hello, in default Kubrick theme there is a problem with ordered list (<ul>
<li></li></ul>) in IE 7 which is not appearing... In another theme it works well, so I suppose that there is some problem in css.
Thank you for your help...

Attachments (4)

IE7_Firefox.PNG (42.5 KB) - added by neoen 17 years ago.
Differences between IE7 and Firefox
4449.diff (377 bytes) - added by djr 16 years ago.
4449-style.css.diff (661 bytes) - added by ninjaWR 16 years ago.
patch (from r9837) of djr's fix
4449-vk.diff (478 bytes) - added by vladimir_kolesnikov 16 years ago.
Patch - valid CSS

Download all attachments as: .zip

Change History (26)

#1 @foolswisdom
17 years ago

  • Milestone changed from 2.2.1 to 2.2.2

#2 @rob1n
17 years ago

  • Keywords reporter-feedback ie7 added
  • Milestone changed from 2.2.2 to 2.4 (future)

Can you provide a description of what's wrong? Or someone else with IE 7?

@neoen
17 years ago

Differences between IE7 and Firefox

#3 @neoen
17 years ago

In Firefox there is ordered list, but in IE7 not (missing). I use default Kubrick (Default 1.6) theme without any changes. Please see attached .png file...

#4 @JeremyVisser
17 years ago

  • Milestone 2.4 (future) deleted
  • Resolution set to invalid
  • Status changed from new to closed

This is an IE bug, and comes from not supporting the :before pseudo-class.

#5 @neoen
17 years ago

So, this cannot be repaired somehow, for example in css? It is default Wordpress theme and it is very annoying...

#6 @JeremyVisser
17 years ago

You might be able to do something like this:

* html .post ul {
    list-style-type: disc;
}

#7 @neoen
17 years ago

  • Milestone set to 2.2.2
  • Resolution invalid deleted
  • Status changed from closed to reopened

I tried to add this code to style.css, but this does not work.
But I found there following:
/* Begin Lists

Special stylized non-IE bullets
Do not work in Internet Explorer, which merely default to normal bullets. */

So there are special bullets for non IE7 browsers, but in IE7 there shuld be normal bullets. But in IE7 there are no bullets :-(
Thank you for your help...

#8 @rob1n
17 years ago

  • Milestone changed from 2.2.2 to 2.3 (trunk)

#9 @ryan
17 years ago

  • Milestone changed from 2.3 to 2.4

#10 @djr
17 years ago

  • Cc djr added
  • Owner changed from anonymous to djr
  • Status changed from reopened to new

#11 @djr
17 years ago

Let's hope I did this correctly. I attached a patch which specifically targets IE7. I realise this adds a lot of extra css to default.css, but it's the only way to have bullets across all browsers. I think it's better to move to image based bullets or include a seperate stylesheet for IE7, but nevertheless my patch works.

Tested with no side-effects: IE7/Win, FF2/Win, FF2/Mac, Safari3/Mac. A temporary demo can be seen here: http://www.iphonemobiel.nl.

#12 @djr
17 years ago

  • Keywords has-patch needs-feedback added; reporter-feedback removed

@djr
16 years ago

#13 @djr
16 years ago

Ok, I s*ck at making patches, so either ignore or please fix.
Here's the code block to be included:

/* Fix for IE7 not showing bullets */
html>body .entry ul {
	*margin-left: 0px;
	*padding: 0 0 0 30px;
	*list-style: disc;
	*padding-left: 10px;
	*text-indent: 10px;
	}

html>body .entry ul li {
	*margin: 7px 0 8px 10px;
	}
/* End fix for IE7 */

This code block need to go just below:

.entry ul li:before, #sidebar ul ul li:before {
	content: "\00BB \0020";
	}

@ninjaWR
16 years ago

patch (from r9837) of djr's fix

#14 @ninjaWR
16 years ago

  • Milestone changed from 2.9 to 2.8
  • Priority changed from normal to low

would be nice if this were to make it into 2.8, since I'm sure the posts look weird to visitors using IE... I wish IE were more compliant, but don't we all ;)

#15 @vladimir_kolesnikov
16 years ago

Have a better patch that does the same but is still valid CSS. Tragetted for IE7.

@vladimir_kolesnikov
16 years ago

Patch - valid CSS

#16 @jacobsantos
16 years ago

I've always wondered why there weren't any styles on the theme when using IE.

#17 @mrmist
16 years ago

See also #4518

#18 @jacobsantos
16 years ago

  • Keywords changed from ie7 has-patch needs-feedback to has-patch needs-feedback ie7

#19 @rowoot
15 years ago

  • Summary changed from Kubrick ordered list problem to Kubrick ordered list problem for IE7

Tested patch 4449-style.css.diff, 4449-vk.diff and 4449.diff.
None seem to work in IE7 as of now. Tried it out with the latest nightly build.

I`ll try working on a update to these patches.

#20 @Nicholas91
15 years ago

  • Keywords ie7 removed
  • Resolution set to duplicate
  • Status changed from new to closed

Duplicate: 4518

#21 @Nicholas91
15 years ago

*ups* Duplicate: #4518

#22 @DrewAPicture
9 years ago

  • Milestone 2.8 deleted
Note: See TracTickets for help on using tickets.