Last Comment Bug 292191 - If using background-image in an element, we should reverse the result of reverse selection algorithm when the element's foreground color and background color is similar
: If using background-image in an element, we should reverse the result of reve...
Status: ASSIGNED
:
Product: Core
Classification: Components
Component: Selection (show other bugs)
: Trunk
: All All
: P3 enhancement with 3 votes (vote)
: ---
Assigned To: Masayuki Nakano [:masayuki] (Mozilla Japan)
:
Mentors:
http://www.samurai-zero.jp/
: 292660 294485 332612 345210 (view as bug list)
Depends on: 170951
Blocks: 56314
  Show dependency treegraph
 
Reported: 2005-04-28 00:20 PDT by Masayuki Nakano [:masayuki] (Mozilla Japan)
Modified: 2015-05-06 23:54 PDT (History)
14 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch rv1.0 (7.80 KB, patch)
2005-08-17 03:35 PDT, Masayuki Nakano [:masayuki] (Mozilla Japan)
no flags Details | Diff | Review
Patch rv1.0 -u8pw (7.42 KB, patch)
2005-08-17 03:36 PDT, Masayuki Nakano [:masayuki] (Mozilla Japan)
no flags Details | Diff | Review

Description Masayuki Nakano [:masayuki] (Mozilla Japan) 2005-04-28 00:20:07 PDT
If an page is using background in an element, the reverse selection system is
faild when the background color is not similar to background image.
If the element's foreground color and background color is similar and existing
background image, we should reverse the current reverse selection result.
Comment 1 Justin Wood (:Callek) 2005-04-28 14:02:49 PDT
Perhaps, but I wonder if exposing (if not already exposed, forgive me for not
knowing image related api) a method such as HasTransparant in the Image would be
worth it, and if there are transparant bits for the image, we could then assume
that the author does intend the specified background color to be 'correct'. [and
as such may be worth exchanging selection colors as currently designed]
Comment 2 Masayuki Nakano [:masayuki] (Mozilla Japan) 2005-04-28 14:14:29 PDT
Justin:
If webpage author using transparent image, I think that the foreground color and
background color are _not_ similar.
So, I want to change the behavior when the colors are similar.
Comment 3 Justin Wood (:Callek) 2005-04-28 14:54:53 PDT
Good enough for me, but just for thought, in some of my personal designs I have
done slight gradients (transparant/alpha-scale) where the actual specified
background color is very close to what the image shows, though I can easily
conceive that I am a minority in that use.
Comment 4 Masayuki Nakano [:masayuki] (Mozilla Japan) 2005-05-03 07:28:14 PDT
*** Bug 292660 has been marked as a duplicate of this bug. ***
Comment 5 Masayuki Nakano [:masayuki] (Mozilla Japan) 2005-08-17 03:30:20 PDT
additional testcases:
https://infosec.navy.mil/
http://mozilla.minutedesign.com/
Comment 6 Masayuki Nakano [:masayuki] (Mozilla Japan) 2005-08-17 03:35:23 PDT
Created attachment 192919 [details] [diff] [review]
Patch rv1.0
Comment 7 Masayuki Nakano [:masayuki] (Mozilla Japan) 2005-08-17 03:36:04 PDT
Created attachment 192920 [details] [diff] [review]
Patch rv1.0 -u8pw
Comment 8 Masayuki Nakano [:masayuki] (Mozilla Japan) 2005-08-17 09:50:42 PDT
*** Bug 294485 has been marked as a duplicate of this bug. ***
Comment 9 Masayuki Nakano [:masayuki] (Mozilla Japan) 2005-08-17 10:19:47 PDT
My patch doesn't work fine on http://www.uoguelph.ca/~stuartr/
This is very difficult problem. The page can look the text if we don't load
images. So, the page's text color and background color conbination has
sufficient contrast. On this case, we cannot fix with my approach.
Comment 10 Masayuki Nakano [:masayuki] (Mozilla Japan) 2005-08-17 10:39:53 PDT
I have an idea. But I don't like this way.

If the page uses background image and it's spcified by other element(not equal
background color specified element), we use text color for selection background
color. If so, the text color and actual background color combination may have
sufficient contrast. But I think that the looks is not good.
# If background image and color specified same element, we trust the background
color.
Comment 11 Boris Zbarsky [:bz] (Out June 25-July 6) 2005-08-17 11:17:50 PDT
I won't be able to review this any time in the near future (at least 2+ weeks).
 I'd really appreciate it if you could find another reviewer...
Comment 12 Masayuki Nakano [:masayuki] (Mozilla Japan) 2005-08-17 11:18:10 PDT
Umm... I wrote test patch by comment 10. But it is not better than previous
patch. Please ignore comment 10.
Comment 13 Masayuki Nakano [:masayuki] (Mozilla Japan) 2005-09-05 12:23:36 PDT
Comment on attachment 192919 [details] [diff] [review]
Patch rv1.0

I cancel the review. Because I try to fix bug 170951 and bug 113161 before
this.
Comment 14 Masayuki Nakano [:masayuki] (Mozilla Japan) 2006-07-19 09:28:01 PDT
*** Bug 345210 has been marked as a duplicate of this bug. ***
Comment 15 Masayuki Nakano [:masayuki] (Mozilla Japan) 2006-07-19 09:29:47 PDT
*** Bug 332612 has been marked as a duplicate of this bug. ***
Comment 16 :Gijs Kruitbosch 2007-05-17 13:30:29 PDT
http://www.ed.ac.uk/studying/

is another example (text in the white column on the right will select with a white background due to the blue bg colour of the page itself).

Masayuki, now that the two bugs you mentioned in comment #13 are fixed, is there any chance you'll try to fix this one next? :-)
Comment 17 Worcester12345 2008-09-04 09:51:12 PDT
(In reply to comment #16)
> http://www.ed.ac.uk/studying/
> 
> is another example (text in the white column on the right will select with a
> white background due to the blue bg colour of the page itself).

I'm not seeing a white column on the right here. Did the page change?

Note You need to log in before you can comment on or make changes to this bug.