Image Map editor should be removed

VERIFIED FIXED in mozilla0.9.1

Status

()

Core
Editor
P1
critical
VERIFIED FIXED
17 years ago
16 years ago

People

(Reporter: Simon Fraser, Assigned: Charles Manske)

Tracking

Trunk
mozilla0.9.1
All
Mac System 8.5
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [dialog])

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
I just tried out the image map editor. It has too many problems to try to fix 
right now; I think we should remove it.
(Reporter)

Comment 1

17 years ago
Target at 0.9.1 to do the removal for beta. We just need to take the buttons out 
of the image dialog.
Target Milestone: --- → mozilla0.9.1

Comment 2

17 years ago
who is going to do the work?
when can it be done?
(Assignee)

Comment 3

17 years ago
It certainly easy to do. I volunteer. Will do it Tuesday, 5/29.

Comment 4

17 years ago
need to get Help Docs updated about the removal

Comment 5

17 years ago
I forgot to mention that we should not re-add the option until it is reworked 
and tested thoroughly. 
Assignee: beppe → cmanske
Severity: normal → critical
Keywords: correctness
Priority: -- → P1
Whiteboard: [dialog]
(Assignee)

Comment 6

17 years ago
Created attachment 36362 [details] [diff] [review]
Don't show the "Edit" button for image map editor.
(Assignee)

Comment 7

17 years ago
Ready for review. This is minimal change: Don't remove anything from UI, just
don't show the only UI element ("Edit" button in Image Dialog) that starts the
feature.
Status: NEW → ASSIGNED
Keywords: patch, review
Whiteboard: [dialog] → [dialog] FIX IN HAND, need r=, sr=

Comment 8

17 years ago
sr=kin@netscape.com
Whiteboard: [dialog] FIX IN HAND, need r=, sr= → [dialog] FIX IN HAND, need r=
(Reporter)

Comment 9

17 years ago
r=sfraser

Comment 10

17 years ago
a=selmer for PDT.
a=blizzard for 0.9.1
(Assignee)

Comment 12

17 years ago
Checked in
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Keywords: patch, review
Resolution: --- → FIXED
Whiteboard: [dialog] FIX IN HAND, need r= → [dialog]

Comment 13

17 years ago
this is still not acceptable to me. I want:

1) rip out that entire Image Map UI area in image props
    a) Remove button
    b) Image map section

did we not do this because of impact on UI screen shots ?

we're gonna get incoming bugs reports on why is there a "Remove"
button in the Image Map area and why doesn't it work.

Comment 14

17 years ago
verified. I guess there would be screenshot impact.
Status: RESOLVED → VERIFIED

Comment 15

17 years ago
removing an image map is ok; 4.x had that functionality
the button should still function

Comment 16

17 years ago
got it....

Comment 17

16 years ago
Bug 100617 is now asking for the image map editor to be re-enabled.

Comment 18

16 years ago
bug 84303 is about re-enabling it
You need to log in before you can comment on or make changes to this bug.