Translation of image is still irelevant (AFAIK there isn't any translator with built-in OCR), so this menu item should be disabled for them (actually does nothing or just redirecting). Gecko/2002012608/WinNT4
Assignee: asa → blaker
Component: Browser-General → XP Apps: GUI Features
QA Contact: doronr → sairuh
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows NT → All
Hardware: PC → All
->morse, who's working on menu items.
Assignee: blaker → morse
Assignee: morse → cbiesinger
Target Milestone: Future → mozilla0.9.9
17 years ago
Comment on attachment 69949 [details] [diff] [review] patch sr=blake This should not visibly affect page load times at all. If for any reason it does, of course, we'll have to back it out.
Attachment #69949 - Flags: superreview+
Comment on attachment 69949 [details] [diff] [review] patch biesi will fix whitespace and change the regexp to ^image/
Attachment #69949 - Flags: review+
putting [FIX] in summary instead of URL field
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Created attachment 69982 [details] [diff] [review] fixes strict warning ok, the patch introduced a strict warning. this patch fixes it.
Attachment #69949 - Attachment is obsolete: true
is there another patch for disable to Text Zoom too for images?
Comment on attachment 69982 [details] [diff] [review] fixes strict warning sr=blake
Attachment #69982 - Flags: superreview+
i've filed http://bugzilla.mozilla.org/show_bug.cgi?id=126059 on more disable on images
Does this need to be reopened? It's been backed out, right?
yes, it was backed out, but I checked it in again when the backout showed no changes to Txul time.
smaug: *sigh*. ok, I will fix this as part of bug 126059
vrfy'd for at least the Translate item... :) *zap*
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.