Summary: | PNG as cursor image format instead of TIFF | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Lars Næsbye Christensen <lars> | ||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Enhancement | CC: | ap | ||||
Priority: | P2 | ||||||
Version: | 420+ | ||||||
Hardware: | Mac | ||||||
OS: | OS X 10.4 | ||||||
URL: | http://www.biotopia.dk/css-cursors/ | ||||||
Attachments: |
|
Description
Lars Næsbye Christensen
2006-10-14 08:04:06 PDT
Created attachment 11087 [details]
Patch to switch format for cursors to PNG
Since the files added are just conversions of current included images (one is made thicker by me, though), they should be covered by the BSD. If I have any rights to that file, I am happy to transfer it to the WebKit project under the BSD license.
Re: ImageIO support for .cur - at the moment, it's quite broken (can be seen with a WebKit nightly on maps.google.com). I do not know whether Apple is going to fix this, or WebKit will have to provide its own .cur decoding. rdar://4556355 rdar://4639253 I don't see the benefit of ths. Comment on attachment 11087 [details]
Patch to switch format for cursors to PNG
Actually this will help other platforms, since TIFF is only supported on the Mac now.
(In reply to comment #3) > I don't see the benefit of ths. Primarily I thought of it as the first step in a series of cursor rewriting updating as I explained on the link. The PNG format is smaller, decompresses faster and is more suited to this task. Maybe this is why Apple stores cursordata in PNG format internally.....? We talked a bit about in on #webkit, and sadly we don't have support for a raster image format that supports full alpha, true color AND animation. Which is why I filed a request to Apple for MNG support. However, the other suggestions I have are not dependent on this, e.g. I could make them in TIFF anyway. alexey: as for the support of .cur files, yes that Google Maps cursor looks terrible :-( Until ImageIO handles them properly, we should perhaps come up with a better local solution. Nice that it got accepted. I'll work on the next series of cursor patches soon :-) |