Combine skin images

[fredck]

Some skin images could be combined inside sprites.png, reducing the number of HTTP calls.

Currently, the following could be considered:

• arrowdown.gif
• arrowtop.gif
• arrowleft.gif
• arrowright.gif
• resizer.gif
• resizer_rtl.gif

[tobiasz.cudnik]

Binary files.

[fredck]

It's almost perfect. Just the following small issues:

• The arrow for the toolbar collapser is 2px mispositioned in RTL. I've noted that you have used background position 4px for LTR, but 2px for RTL. Maybe using also 4px will fix the problem.

• IE+RTL: The context menu arrow is at right.

• IE6: The context menu arrow shows white background when rollover.

I'm adding an attachment with screenshot for all the above.

[tobiasz.cudnik]

I've managed to fix all 3 mentioned issues plus others below:

• office2003 dialogs lacked vertical shadow in RTL
• propagated fix from #3969 in office2003 and v2 skins which turned out to be also affected
• fixed office2003 and v2 context menu in RTL
• fixed downarrows in office2003

[tobiasz.cudnik]

Instead of #3969 i've meant #3696.

[fredck]

I've followed the following steps:

1. Applied 3793_3.patch into my updated local trunk copy.
2. Overwritten my local copy files with all files available at skins_3.zip.

Then, while testing, I've noticed that none of the issues described in my previous comment have been addressed.

Locking closer to the patch, it looks like things have been fixed, but much probably the skins_3.zip file is wrong. so, it was not possible to properly test the fix.

Ideally the skins_3.zip should contain only the binary files that the patch can't have, or even the full directories copy, but including the properly patched files.

[tobiasz.cudnik]

skins_3.zip was supposed to provide only binary files, although it contained also normal css files from previous patch, what created the confusion.

I will prepare new zip packages for easier testing.

[fredck]

I've overwritten my local copy with skins_all_4.zip and the layout got totally broken in IE. In Firefox it's also possible to note issues when collapsing the toolbar.

[fredck]

It looks like things where broken because skins_all_4.zip is not good for it. The right way is instead having the latest patch plus skins_binary_4.zip.

[fredck]

Everything looks good, except a rendering issue in the toolbar arrows with IE7+Standards.

[tobiasz.cudnik]

Patch commited with [3890] in ​features/3793 branch.

IE7 standard incorrectly handles alpha transparency in backgrounds when element itself is filtered with alpha transparency.

[fredck]

The Kama skin looks good. There are issues with the Office2003 skin, with IE+Quirks and IE6.

• The dialog shadow is broken and not properly rendered (no support for PNG transparency)

• The dialog close button is mispositioned.

I'll be attaching a screenshot illustrating it in IE7. The same problems can be see in IE6, no matter the mode.

[fredck]

IE7 screenshot of branches/features/3793 at r3894.

[fredck]

Regarding the dialog shadow, the current trunk behavior is the expected one, having no shadow at all in IE+Quirks and IE6.

[tobiasz.cudnik]

Changeset [3896] address issue you've mentioned. Attaching screenshots.

[tobiasz.cudnik]

IE8

[tobiasz.cudnik]

IE7

[tobiasz.cudnik]

IE6

[fredck]

I've partially reverted [3896] and properly fixed the close button position with [3905].

[fredck]

I've also fixed the dialog borders in the office2003 skin for IE+Quirks+RTL with [3906].

Now, the V2 skin is broken isntead.

[tobiasz.cudnik]

Review? for [3907].

[fredck]

Kama: IE+Quirks: The Button Panel items in the toolbar (color selectors and SCAYT) have no arrows anymore.

[tobiasz.cudnik]

Review? for [3921].

[fredck]

I have made a small fix for v2 with [3923]. Things should be ok now... finally!

[tobiasz.cudnik]

Fixed with [3927] and [3928].