http://mxr.mozilla.org/mozilla-central/search?string=page-proxy-stack page-proxy-stack is unnecessary, as it contains only one node (page-proxy-favicon). In pinstripe, page-proxy-stack can be removed right away. In winstripe and gnomestripe, it has padding which should be converted to margin on page-proxy-favicon.
Created attachment 584822 [details] [diff] [review] Pemoved page-proxy-stack and page-proxy-favicon edited as requested
Assignee: nobody → mrshux
Status: NEW → ASSIGNED
Summary: Remove page-proxy-stack → Stop styling page-proxy-stack
Comment on attachment 584822 [details] [diff] [review] Pemoved page-proxy-stack and page-proxy-favicon edited as requested Thanks! A flawless first patch -- something you don't see very often. :)
Attachment #584822 - Flags: review+
It was a great example of good first bug although there was not much to fix.
Target Milestone: --- → Firefox 12
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.