per summary. broken off of 73712.
->bryner in hopes that he can make us suck a little less for 1.1
Assignee: kerz → bryner
Still hoping to see distinct icons under Linux for Browser & Mail in 1.1. Any chance?
is this the same as / related to bug 29856?
biesi: IIRC, bug 29856 blocks this one as those icons can't be used once they're checked in. Marking as dependency.
Depends on: 29856
Not related to 29856.
Status: NEW → ASSIGNED
No longer depends on: 29856
I went ahead and had mcafee copy the windows .ico files in the repository to xpfe/bootstrap/icons/windows, for consistency and so we don't have so many icon files cluttering up xpfe/bootstrap.
Comment on attachment 94902 [details] [diff] [review] patch r/sr=blizzard
Attachment #94902 - Flags: review+
Comment on attachment 94902 [details] [diff] [review] patch >+ifneq (,$(filter windows gtk,$(MOZ_WIDGET_TOOLKIT))) Oh, can you add gtk2 to that list? I would appreciate it.
I wasn't adding gtk2 yet since the code to support loading the icons isn't there.
huh? http://lxr.mozilla.org/seamonkey/source/widget/src/gtk2/nsWindow.cpp#802 the location might need to be changed, and I've already filed a bug to use a better resoltion scheme. gtk2 definitely supports icon loading, though.
My bad. New patch coming up.
Created attachment 95015 [details] [diff] [review] patch v2 handle gtk2
Attachment #94902 - Attachment is obsolete: true
rginda's interested in this.
I'm curious how this is "not related to bug 29856"? This patch seems like a copy of the cheezy windows hack, whereas 29856 would be a general purpose (and much cleaner) solution. Why check in all this hard-coded path/file extension stuff when we can just set a wmclass and let the window manager decide what to do?
I'm not saying 29856 isn't a good fix to have, but I think it addresses a different problem from what this is addressing. Setting a wmclass does not automatically get you a particular window icon, and asking users to do custom window manager configuration to set such an icon (when there's obviously a more supported mechanism for setting it) is not acceptable. If you're using the GNOME panel, it doesn't have any visible support for associating an icon with a particular window class.
I'm making one minor change, not going to attach a new patch for it. Instead of using nsMemory::Free() for freeing the hash entry string, I changed it to use free() since that matches with strdup()'s usage of malloc().
Re Comment #18, may I suggest reading some comments in Bug #29856 (around #51) for discussion about nsMemory and strdup. I only mention this, not to say that 29856 is related to this bug, but because the discussion may be relevant to your last change.
*** Bug 162540 has been marked as a duplicate of this bug. ***
Comment on attachment 95015 [details] [diff] [review] patch v2 r=pavlov
Attachment #95015 - Flags: review+
So now we just need a "sr" and an "a" on this bug and v1.1 will be ready.
I was just looking at the Status messages for the Attachments, and none of them have a "sr" or an "a". Comment #9 may say "sr", but blizzard didn't make that change to the attachment. However, if I was the owner, I'd probably check it into the trunk anyway. But it still needs an "a" for the v1.1 release.
Comment on attachment 95015 [details] [diff] [review] patch v2 dgk, blizzard offered either an r or sr. you can't put that on attachment tracker. that's no problem. but if it makes you feel better, I can put the has-super-review there.
Attachment #95015 - Flags: superreview+
Comment on attachment 95015 [details] [diff] [review] patch v2 a=asa (on behalf of drivers) for checkin to 1.1
Attachment #95015 - Flags: approval+
Checked in on trunk and 1.1 branch.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Stupid question (this might be spam, but I don't come along with this bug): With Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.1b) Gecko/20020815, BuildID 2002081521 I just see under kde 2.2.1 only the XWindow icon as desktop/taskbar icons ... This wasn't intended, wasn't it?
Hmm, I don't see this Problem (yesterday's trunk CVS build, RH7.3 Linux) - I have a different, Mozilla-specific icon for each application. While I think that the muted (as in dark and weak in color contrast - I'm not so sure of my English vocabulary here) colors make the icons a bit hard to distinguish, I really like the design. It's great to have different icons in Linux at last! :-)
Icon for the calendar component is missing.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Daniel, that's bug 163371 and happens on windows too. marking fixed again.
Status: REOPENED → RESOLVED
Last Resolved: 16 years ago → 16 years ago
Resolution: --- → FIXED
oops :/ I should learn to read. anyway, filing a new bug was the right thing.
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2b) Gecko/2002100214 I have no Mozilla icons in any windows and not in the KDE taskbar. Always the X of X window is used. Most windows had correct icons in RPMs of Friday 13 September. Looks like bug 165318 is the bug which says this is not fixed. pi
You need to log in before you can comment on or make changes to this bug.