Closed Bug 1251586 Opened 8 years ago Closed 5 years ago

get rid of preprocessing for javascript files where possible in mail+mailnews, using AppConstants.jsm instead

Categories

(Thunderbird :: General, defect)

Product:
Thunderbird
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WORKSFORME

People

(Reporter: mkmelin, Unassigned)

References

Details

Pre-processed JavaScript files make them harder to work on as you need to recompile to make the changes show up. It's also a non-standard js feature. 

Using AppConstants.jsm instead is an easy to clean this up.
Summary: get rid of preprocessing for javascript files where possible in mail+mailinews, using AppConstants.jsm instead → get rid of preprocessing for javascript files where possible in mail+mailnews, using AppConstants.jsm instead
Yes, thanks.

Not all the preprocessor constants are yet in AppConstants, but we can see what can be done. Also we still can't fix .xul files (unless we add code to hide/remove elements from JS instead of the preprocessor), but probably the .js files are more important to avoid processing.
Depends on: 1278337, 1278082
OS: Unspecified → All
Hardware: Unspecified → All
Version: unspecified → Trunk
Depends on: 1304529
Depends on: 1304842

Looks like it's all done.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.