Remove new MailNews dependency on Navigator

VERIFIED FIXED in mozilla0.9.4

Status

VERIFIED FIXED
18 years ago
10 years ago

People

(Reporter: bugzilla, Assigned: tao)

Tracking

Trunk
mozilla0.9.4
x86
Windows 2000

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

18 years ago
Tao, your chrome/ui separation added a MailNews dependency on Navigator.  The 
search panel is available to all apps, and you had it include 
region.properties, located in xpfe/browser.  This needs to move to 
xpfe/communicator so the Search panel continues to work properly from MailNews 
when Navigator isn't installed.
(Reporter)

Updated

18 years ago
Keywords: mozilla0.9, nsbeta1
(Assignee)

Comment 1

18 years ago
there were two copies of this values, one in search-panels.properties
and used in search-panel.js; while the other was in navigator.properties.

I'd suggest we find out if both are valid before moving the url again.
(Assignee)

Comment 2

18 years ago
Hi, Paul:

Any idea who owns "Search" component now?

reassign to the owner of "search" until we know where the search url should live.
Assignee: tao → pchen
(Reporter)

Comment 3

18 years ago
The one in search-panel.properties is the valid one.

Updated

18 years ago
Keywords: nsbeta1 → nsbeta1-
nav pretriage: moving to Future. priority P5. 
Priority: -- → P5
Target Milestone: --- → Future
(Reporter)

Comment 5

18 years ago
Why is this on Paul's plate?  The person who caused the regression should fix 
it, please.
Assignee: pchen → tao
Keywords: mozilla0.9, nsbeta1-
Priority: P5 → --
Target Milestone: Future → ---
(Assignee)

Updated

18 years ago
Target Milestone: --- → mozilla0.9.3
(Reporter)

Comment 6

18 years ago
Doesn't look like this is getting fixed before the freeze tomorrow night. 
Pushing out a milestone.  Please correct if I'm mistaken.
Target Milestone: mozilla0.9.3 → mozilla0.9.4
(Assignee)

Comment 7

18 years ago
Created attachment 45319 [details] [diff] [review]
move search url to communicator
(Assignee)

Updated

18 years ago
Whiteboard: need r/sr=
this looks OK - r=jbetak
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
Whiteboard: need r/sr= → need sr=

Comment 9

18 years ago
sr=mscott
(Assignee)

Comment 10

18 years ago
Created attachment 47344 [details] [diff] [review]
package the new region.properties into US.jar
(Assignee)

Updated

18 years ago
Whiteboard: need sr= → ready
(Assignee)

Comment 12

18 years ago
thanks, checked in!
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
Whiteboard: ready

Comment 13

17 years ago
Verified with 0906 0.9.4 build. It's fixed.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.