58501 - Remove bogus instances of -moz-opacity

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect, P3)

Description

[Pierre Saslawsky]

This bug is extracted from bug 57307. See my comments from [2000-10-25 02:39] and 
[2000-10-26 18:25].

We should remove all the cases where the opacity is set to 99%.  It makes no 
sense and I imagine it can only use more CPU power than having opacity set to 
100%.

Comment 1

[Pierre Saslawsky]

See bug 58517 about ekrock's suggestion to the name of the property.

Comment 2

[chris hofmann]

got the patch?
should we take it for m0.8 to get some testing
or performance kick?

if not, then change the milestone.

Comment 3

[karnaze (gone)]

Moving to Mozilla0.9

Comment 4

[Mike Kaply [:mkaply]]

The only places where opacity is set to .99 is in the image map editor.

Removing these lines from image map editor makes image map editor work on OS/2 
since we don't support opacity yet.

We will be attaching a diff in a minute and humbly begging to get this into .8.

We just realized that this was a problem on OS/2 this morning.

Comment 5

[Mike Kaply [:mkaply]]

Attachment: Remove .99 opacity from ImageHotSpotEditor

Comment 6

[Mike Kaply [:mkaply]]

*** Bug 55101 has been marked as a duplicate of this bug. ***

Comment 7

[Kathleen :Brade]

r=brade

Comment 8

[Pierre Saslawsky]

Back to moz0.8 + reassigned to Michael who's got the fix and a CVS account.
Ask [e-mail] or [e-mail] for approval.

Comment 9

[Simon Fraser [no longer active]]

cmanske, brade: do we know why the image editor used opacity of .99 everywhere?

Comment 10

[Brian King [:kinger]]

It was only being used in one place to "wash out" the image for the High 
Contrast feature.  This feature is now soon to be taken out as it was not 
functioning anyway.

Comment 11

[Asa Dotzler [:asa]]

mkaply, is this something you're interested in getting in for 0.8?

Comment 12

[Mike Kaply [:mkaply]]

Asa. Yes definitely. I was waiting for an sr= so I could send it to drivers.

Without this fix, the hotspot editor doesn't work on OS/2.

Comment 13

[Simon Fraser [no longer active]]

sr=sfraser

Comment 14

[Asa Dotzler [:asa]]

a=asa for checkin on the 0.8 branch

Comment 15

[Mike Kaply [:mkaply]]

Fix checked in on branch.

Leaving open until trunk opens so I can get it on the trunk.

Comment 16

[Hixie (not reading bugmail)]

Netscape's standard compliance QA team reorganised itself once again, so taking 
remaining non-tables style bugs. Sorry about the spam. I tried to get this done 
directly at the database level, but apparently that is "not easy because of the 
shadow db", "plus it screws up the audit trail", so no can do...

Comment 17

[Mike Kaply [:mkaply]]

Fix checked in to trunk

Comment 18

[Pierre Saslawsky]

We finally have the explanation as of why someone set the opacity to 99% in the 
ImageMap Editor files...  See bug 92282: when the opacity is 100%, we can't 
change it dynamically; when it is 99%, we can.