disable menu stuff for images

VERIFIED FIXED in mozilla0.9.9

Status

P1
normal
VERIFIED FIXED
17 years ago
11 years ago

People

(Reporter: bugzilla, Assigned: Biesinger)

Tracking

Trunk
mozilla0.9.9

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

17 years ago
we might as well disable the following stuff under Edit for images:
Text Zoom
Use Stylesheets
Charater Coding

and under Search
everything...
you wouldn't want to disable "Search the Web".

anyway, taking
Assignee: blaker → cbiesinger
Status: NEW → ASSIGNED
OS: Windows 2000 → All
Priority: -- → P1
Hardware: PC → All
Target Milestone: --- → mozilla0.9.9
Posted patch patch (obsolete) — Splinter Review
this should fix it.

btw, you need not cc me on bugs that are assigned to me :)

Comment 3

17 years ago
Comment on attachment 69993 [details] [diff] [review]
patch

noImage should be isImage and i'm opposed to disabling view-source even though
it's currently broken
Attachment #69993 - Flags: review+

Comment 4

17 years ago
why did you remove the references to the translateMenu?
alecf: because I'm now using the broadcaster to change the disabled state of the
translate menu

alec or blake, could one of you super-review?
cc'ing marlon in case he might have opinions here...
Blocks: 108099
Summary: disable stuff for images → disable menu stuff for images

Updated

17 years ago
No longer blocks: 108099

Comment 7

17 years ago
don't disable anything underneath "search the web" either. otherwise disabling
those other things makes sense.
uh, there _is_ nothing underneath search the web...

Comment 9

17 years ago
uh, there _is_ in commercial...
ah sure, forgot about that. such entries will not be disabled because of my patch.
Posted patch new patchSplinter Review
fixes timeless' comments
this patch should also fix the problem smaug reported in bug 122144.
Attachment #69993 - Attachment is obsolete: true

Updated

17 years ago
Attachment #70315 - Flags: review+

Comment 12

17 years ago
Comment on attachment 70315 [details] [diff] [review]
new patch

sr=alecf
Attachment #70315 - Flags: superreview+
checked in, just in time for 0.9.9 :)
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
(Reporter)

Comment 14

17 years ago
v 20020221
Status: RESOLVED → VERIFIED
Product: Core → Mozilla Application Suite

Updated

11 years ago
Component: XP Apps: GUI Features → UI Design
You need to log in before you can comment on or make changes to this bug.