Closed
Bug 591969
Opened 15 years ago
Closed 15 years ago
feed handler icons not shown in feed preview menupopup ( feed reader )
Categories
(Firefox Graveyard :: RSS Discovery and Preview, defect)
Tracking
(blocking2.0 beta7+)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
blocking2.0 | --- | beta7+ |
People
(Reporter: mcdavis941.bugs, Assigned: sicking)
References
Details
Attachments
(2 files)
116.50 KB,
image/png
|
Details | |
1.86 KB,
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
Looking at the menupopup on the feed preview page, I noticed the icons are missing. Is that an intentional change?
Also, the current menuitem is highlighting with dark-on-dark colors.
Also, the right border on the menupopup goes missing frequently.
Also, if you happen to be using a third-party theme which displays the icons, you notice that the icon for "choose application" no longer shows. Instead, it shows the RSS icon, which seems to be because the menuitem receives image="page-livemarks.png", which prevents "application.png" icon from applying via in-theme CSS to the menuitem. DOM Inspector shows the image attribute is not present on that menuitem in Fx3.6.
STR:
- show the feed preview by loading http://feeds.nytimes.com/nyt/rss/HomePage
- open the menupopup in the content area at the top of the feed preview page
- notice the icons don't show and other issues (see screenshot)
Mozilla/5.0 (Windows NT 6.0; WOW64; rv:2.0b5pre) Gecko/20100829 Firefox/4.0b5pre
Not sure who to CC on this, but adding some of the people from other feed preview bugs and Jonas Sicking because Bug 546857 touched feed previews.
Reporter | ||
Comment 1•15 years ago
|
||
Reporter | ||
Comment 2•15 years ago
|
||
Also, it doesn't seem right for the feed handlers of disabled add-ons to show in the list, but that's probably a separate issue.
Assignee | ||
Comment 3•15 years ago
|
||
I wonder if this is windows specific. I definitely worked to make sure that the icons kept working, and they are working on my mac.
Assignee: nobody → jonas
blocking2.0: --- → beta6+
Assignee | ||
Updated•15 years ago
|
Blocks: kill-remote-xul
Comment 4•15 years ago
|
||
It's not Windows-specific, it's Aero-specific. If you had 55 lines of context in https://bug546857.bugzilla.mozilla.org/attachment.cgi?id=467560 then the browser/themes/winstripe/browser/jar.mn hunk would have shown the warning in http://mxr.mozilla.org/mozilla-central/source/browser/themes/winstripe/browser/jar.mn#4
Assignee | ||
Updated•15 years ago
|
Attachment #470495 -
Flags: review?(philringnalda) → review?(dolske)
Updated•15 years ago
|
Attachment #470495 -
Flags: review?(dolske) → review+
missing icon, bug 398130
Comment 7•15 years ago
|
||
the missing right border looks like DW bug 571447.
Assignee | ||
Comment 8•15 years ago
|
||
Forgot to mark this one fixed. Fix was checked in September 1st.
http://hg.mozilla.org/mozilla-central/rev/bf04d87edcba
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•