225314 - should use darker colors for dark color in ridge border, like IE

Status: RESOLVED INVALID

(Core :: Layout, defect)

Description

[bluppfisk]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6a) Gecko/20031028
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6a) Gecko/20031028

This gives a smooth 3d effect in Internet Explorer, though not at all in
Mozilla. That's a pity since I am promoting Mozilla on that very same page :)

Reproducible: Always

Steps to Reproduce:
1. goto this site http://fkserv.ugent.be/~germania/index.php?page=kalender
2. have a look at it
3. repeat steps 1 and 2 with Internet Explorer
4. Compare

Actual Results:  
EEk!

Expected Results:  
show smooth ridges

none

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I assume you mean the 3-D effect around the square for the current date,
correct?  It's a big page, and that's the first difference I noticed.

Comment 2

[bluppfisk]

Eggzactly.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Could you provide some steps to reproduce and actual/expected results that would
be useful to someone who does NOT have IE to test with?

Comment 4

[bluppfisk]

I'd love to, but I wouldn't know how.

Compare it to the nifty toolbars with apparently no buttons on them. But as soon
as you move your mouse over the image on the toolbar, a slightly beveled button
appears. But again, this is mainly seen in windows programs, and I take it
you're not one of those?

Comment 5

[Boris Zbarsky [:bzbarsky]]

Well, you could have posted a screenshot of what IE shows, for example...

The table cell for "today" has a 1px inset border.  There is really no
meaningful way to do inset borders that are only 1px thick, and I know for a
fact that Mozilla and IE use different colors to render "inset" (Mozilla's tend
to be much darker).  Is that the problem here?

Comment 6

[bluppfisk]

Attachment: Screenshot Germania Calendar IE

Well here it is.
Then again if I make the border 2px thick, it's ugly.

Comment 7

[Boris Zbarsky [:bzbarsky]]

Yep, just what I thought.  The IE algorithm uses darker colors for the "inset"
than we do (which looks better here and looks terrible in other cases).

Comment 8

[bluppfisk]

Couldn't mozilla be modified to show it smoothly then? Wouldn't take too much 
effort I guess...

Comment 9

[Boris Zbarsky [:bzbarsky]]

You clearly missed the "looks better here and looks terrible in other cases"
part of my comment on IE's rendering?  Modifying the algorithm to make this case
look better without making others look worse would be rather nontrivial.

But if you think this should be easy, feel free to fix it.  The relevant code is
at
http://lxr.mozilla.org/seamonkey/source/layout/html/style/src/nsCSSColorUtils.cpp#70

Comment 10

[bluppfisk]

Oh heh heh I thought you meant other browsers with "other cases". Anyhow, do 
you have a suggestion to have the same neat look in my website as in the 
rendering IE conceives?

Comment 11

[Boris Zbarsky [:bzbarsky]]

Not offhand, since I haven't really figured out the algorithm IE uses to pick
its colors... (or the one Mozilla uses either, for that matter).

Comment 12

[Jesiah S]

For a quick fix how about using a solid border and using the colors from IE for
each of the four borders.

Comment 13

[Bernd]

Attachment: testcase

Comment 14

[Bernd]

we render the testcase identical with Opera 7.20 win, IE however has  a two
level mechanism for the inset. One could probably find also a background color
where IE has difficulties with showing a inset. The same happens on a div border
so this is not a table issue.

Comment 15

[Johannes Buchner]

This ain't a bug in my eyes. It's just that different browsers have different default values. If you want the same behaviour, you can still set the css values for each border (width, color, ...) on each side. 

Comment 16

[Boris Zbarsky [:bzbarsky]]

The testcase _does_ set the border color.  I suggest reading the entirety of the bug and all the testcases carefully before commenting... ;)

Comment 17

[j.j.]

I'd say this is invalid.
See also bug 50699. That bug should be fixed, of course.

Comment 18

[j.j.]

INVALID especially since IE9 stopped using two colors for inset borders.