Ticket #4449 (new defect (bug))

Opened 2 years ago

Last modified 1 week ago

Kubrick ordered list problem

Reported by: neoen Assigned to: djr
Priority: low Milestone: 2.8
Component: Template Version: 2.2
Severity: normal Keywords: ie7 has-patch needs-feedback
Cc: djr

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

IE7_Firefox.PNG (42.5 kB) - added by neoen on 06/14/07 08:00:38.
Differences between IE7 and Firefox
4449.diff (377 bytes) - added by djr on 03/18/08 13:06:33.
4449-style.css.diff (0.6 kB) - added by ninjaWR on 11/22/08 03:26:38.
patch (from r9837) of djr's fix
4449-vk.diff (478 bytes) - added by vladimir_kolesnikov on 12/11/08 03:54:37.
Patch - valid CSS

Change History

06/13/07 16:32:19 changed by foolswisdom

  • milestone changed from 2.2.1 to 2.2.2.

06/13/07 16:42:11 changed by rob1n

  • keywords set to reporter-feedback ie7.
  • 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?

06/14/07 08:00:38 changed by neoen

  • attachment IE7_Firefox.PNG added.

Differences between IE7 and Firefox

06/14/07 08:01:32 changed by neoen

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

06/14/07 08:09:06 changed by JeremyVisser

  • status changed from new to closed.
  • resolution set to invalid.
  • milestone deleted.

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

06/14/07 08:12:54 changed by neoen

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

06/14/07 10:01:11 changed by JeremyVisser

You might be able to do something like this: 

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

06/15/07 07:49:39 changed by neoen

  • status changed from closed to reopened.
  • resolution deleted.
  • milestone set to 2.2.2.

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

06/15/07 16:41:58 changed by rob1n

  • milestone changed from 2.2.2 to 2.3 (trunk).

09/13/07 21:24:52 changed by ryan

  • milestone changed from 2.3 to 2.4.

03/14/08 04:24:29 changed by djr

  • cc set to djr.
  • owner changed from anonymous to djr.
  • status changed from reopened to new.

03/16/08 17:42:51 changed by djr

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.

03/16/08 17:49:24 changed by djr

  • keywords changed from reporter-feedback ie7 to ie7 has-patch needs-feedback.

03/18/08 13:06:33 changed by djr

  • attachment 4449.diff added.

03/18/08 13:09:52 changed by djr

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";
	}

11/22/08 03:26:38 changed by ninjaWR

  • attachment 4449-style.css.diff added.

patch (from r9837) of djr's fix

11/22/08 03:39:05 changed by ninjaWR

  • priority changed from normal to low.
  • milestone changed from 2.9 to 2.8.

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 ;)

12/11/08 03:54:13 changed by vladimir_kolesnikov

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

12/11/08 03:54:37 changed by vladimir_kolesnikov

  • attachment 4449-vk.diff added.

Patch - valid CSS

12/27/08 21:38:32 changed by jacobsantos

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

01/01/09 16:45:09 changed by mrmist

See also #4518