Closed Bug 53276 Opened 24 years ago Closed 24 years ago

Remove a feature from the Prefs/ Appearances/ Color panel

Categories

(SeaMonkey :: Preferences, defect, P2)

defect

Tracking

(Not tracked)

VERIFIED WONTFIX

People

(Reporter: johng, Assigned: alecf)

References

Details

(Whiteboard: [nsbeta3-][PDTP2][rtm need info]fix in hand, waiting on review)

Attachments

(2 files)

In preferences, Appearances / Colors, the last or third box of functionality is "When a web page provides its own colors ....". Due to bug 22963, we have never been able to get this to work correctly. Therefore removing the feature.
nav triage team: +, P1 in order to save us the time necessary to get this feature to work
Keywords: nsbeta3
Priority: P3 → P1
Whiteboard: [nsbeta3+]
PDT: the feature itself isn't a ship stopper. Removing the UI would be a kindness. P2 for removal and cleanup. This would be a UI change and cannot be done without coordination with L10N.
Priority: P1 → P2
Whiteboard: [nsbeta3+] → [nsbeta3+][PDTP2]
Let's deal with this in RTM as long as l10n doesn't get hosed by the delay ...
Keywords: rtm
Whiteboard: [nsbeta3+][PDTP2] → [nsbeta3-][PDTP2][rtm+]
Vera: release note
Keywords: relnote3
Still waiting on bug #22963.
Depends on: 22963
Whiteboard: [nsbeta3-][PDTP2][rtm+] → [nsbeta3-][PDTP2][RTM NEED INFO]
patch needs review
Keywords: patch
Whiteboard: [nsbeta3-][PDTP2][RTM NEED INFO] → [nsbeta3-][PDTP2]patch needs review
Can't mark this rtm+ until the review and approval is confirmed. Besides, we're still waiting on the resolution of bug #22963.
Whiteboard: [nsbeta3-][PDTP2]patch needs review → [nsbeta3-][PDTP2][rtm need info]fix in hand, waiting on review
cc'ing attinasi who's working on bug 22963.
I'm pretty close to having this feature working, which might mean that we don't need to remvoe the pref. Of course, my changes may not be accepted by PDT, so it is good to be ready to go with this. See bug 22963, where I'll post the patches when they are ready. Other related bugs include bug 20760 and bug 31816, which may similarly need the pref UI 'altered' or removed if my fixes are not accepted (not sure if there is a bug for those, of if it should be included in this one). Also, thanks to S.E.V. for cluing me into this bug!
patch is a simple #if 0 around the XUL UI, it would be easy to back out to take attinasi's fix.
Still waiting on Marc's fixes to land on the branch.
Taking over - I'll keep an eye on this.
Assignee: matt → alecf
Attached patch patch to remove ui just-in-case (deleted) — Splinter Review
oh, heh, we already had a patch :)
Status: NEW → ASSIGNED
I have checked in the changes to implement the prefs, on the branch, and so far nobody has told me to back them out! Woo Hoo!!! Maybe this one can be closed out as WONTFIX once my changes have survived the weekend?
Seeing as how no babies have died in today's build ... i'm marking this WONTFIX now and finally.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → WONTFIX
vrfy wontfix
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: