Closed Bug 700162 Opened 13 years ago Closed 12 years ago

Add mozapps/extensions/newaddon.css to Modern

Categories

(SeaMonkey :: Themes, defect)

defect
Not set
normal

Tracking

(seamonkey2.7+ fixed, seamonkey2.8 fixed, seamonkey2.9 fixed)

RESOLVED FIXED
seamonkey2.9
Tracking Status
seamonkey2.7 + fixed
seamonkey2.8 --- fixed
seamonkey2.9 --- fixed

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

References

Details

(Keywords: modern)

Attachments

(1 file)

Bug 476430, which landed for FF 8 introduced a new CSS file to mozapps (newaddon.css). This file is only used by newaddon.xul, which is only supported starting with SM 2.7 (since bug 696598 landed). Modern needs a copy for the new functionality to work correctly with the alternate theme.

STR:
1. Download or create an XPI of an add-on that is compatible with the tested SM version and rename the XPI to <guid>.xpi (where <guid> is the em:id from install.rdf).
2. Choose an SM profile where the chosen add-on is not installed yet or make sure that's the case. Also make sure that profile has Modern as the active theme.
3. Copy the XPI into the extensions folder of the tested SM profile folder.
4. Start SM
Simpler STR: Type the following into the Location bar:
about:newaddon?id={1018e4d6-728f-4b20-ad56-37578a4de76b}
(assuming you have that extension installed.)

For some reason SeaMonkey just closes suddenly if you use the same GUID twice. (no crash report)
(In reply to Philip Chee from comment #1)
> Simpler STR: Type the following into the Location bar:
> about:newaddon?id={1018e4d6-728f-4b20-ad56-37578a4de76b}
> (assuming you have that extension installed.)
> 
> For some reason SeaMonkey just closes suddenly if you use the same GUID
> twice. (no crash report)

IIUC this is bug 684190, fixed-on-trunk 2011-10-07, see bug 684190 comment #5 (but there are followups to that bug, if you're using a newer build maybe one of them cought you).
Keywords: modern
We certainly should port this to modern, the specific coloring here is minimal. I hope we can make 2.7 for this, Jens any chance you can tackle this?
(In reply to Philip Chee from comment #1)
> Simpler STR: Type the following into the Location bar:
> about:newaddon?id={1018e4d6-728f-4b20-ad56-37578a4de76b}
> (assuming you have that extension installed.)
> 
> For some reason SeaMonkey just closes suddenly if you use the same GUID
> twice. (no crash report)

If you hit Esc really quickly (and have some luck), you can get the page for any ID. Best to be tried with at least one other tab open. ;-)

I just found that the easiest way to get the page for real is to disable an add-on (I used Debug & QA, which I always disable first) and then try it, e.g.:

about:newaddon?id=debugQA@mozilla.org

Now that I'm past that I can try to fix this bug. :-)
Neil, Callek wants to still have this for our next beta, so maybe you can prioritize this? Thanks in advance.

Changes over winstripe version:
* ported general styling from about pages (about.css): colors and borders from html and body
* replaced Graytext by font-style:italic
* maybe more that I forgot...
Assignee: nobody → jh
Status: NEW → ASSIGNED
Attachment #588778 - Flags: review?(neil)
Comment on attachment 588778 [details] [diff] [review]
patch [Checkin: Comment 8]

>+@import url("chrome://global/skin/inContentUI.css");
We don't have this, so no need to import it. r=me with that fixed.
Attachment #588778 - Flags: review?(neil) → review+
Comment on attachment 588778 [details] [diff] [review]
patch [Checkin: Comment 8]

Base bug 696598 already landed for 2.7, requesting approval accordingly for very safe patch.
Attachment #588778 - Flags: approval-comm-beta?
Attachment #588778 - Flags: approval-comm-aurora?
Attachment #588778 - Flags: approval-comm-beta?
Attachment #588778 - Flags: approval-comm-beta+
Attachment #588778 - Flags: approval-comm-aurora?
Attachment #588778 - Flags: approval-comm-aurora+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.9
Depends on: 751253
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: