Re: Problems with setup colors in snapshot

2006-05-23 Thread Igor Peshansky

On Mon, 22 May 2006, mwoehlke wrote:


Igor Peshansky wrote:
 On Mon, 22 May 2006, mwoehlke wrote:
  A while back I posted about the colors in setup.exe. I noticed that the
  text background color is fixed but that there is still a problem with
  the tree structure; specifically, the clickable [+] and [-] icons are,
  on my colors, effectively invisible. On some other color schemes I
  tried, they *are* 100% invisible.

 Yes, this is most likely reproducible (as the foreground color of those
 icons is *always* black.

 3: If I pick, e.g. (64,0,128) for my window background color, the
 tree becomes the *exact* same color as the background.

 Hmm, weird -- for me the foreground color of the icons is *always*
 black.

Unfortunately, a: this isn't the case for me, and b: as you've so
astutely pointed out, if it was that would still be problematic :-). (In
reality, I've learned that *someone* is always going to exercise poor
design and so I usually pick a background light enough for black to be
at least marginally visible against it).


Besides, I was wrong -- the foreground isn't always black, it may
sometimes be the inverted background color.


Anyway, I'm going to violate policy and attach a screenshot, as it
seems prudent in this case. The screenshot is with my normal settings;
if you open it in a graphics editor with a non-fuzzy bucket fill tool
(e.g. M$Paint), you can flood the background and see that the tree is
actually being painted, but in a /very/ subtly different color (in fact
it is (66,74,80), which I *can* barely on my other monitor at much
closer range - or if I magnify it). However this is clearly not black
;-).


Right.


 Theoretically, it should be a simple matter of using MaskBlt instead
 of BitBlt with the properly configured pen color and the properly set
 up mask.  In practice, making this work on all platforms (including
 Win9x), correctly, and with the minimum amount of code changes is a
 big pain.

I'm using Win2k3 R2 x64 and (see attached screenshot) I can assure you
it doesn't work on my system. Maybe my OS is the problem? (Hmm... yup,
the problem has to be Windows ;-).)

Also, I'd test on my XP system but it is consistently dereferencing a
NULL :-(. (On W2k3 R2 it did that *once* and has since run just fine.)


MaskBlt isn't available on Win9x systems.


 If you're interested in looking at the code and providing a patch,
 grep the setup sources for bitmap_dc (or BitBlt) and patch all those
 places.

 If someone can take one of the bitmaps from the setup sources, and
 send me a *complete* small program that draws that bitmap on a window
 drawing context with the foreground and background colors coming from
 the Windows color scheme, I'll see what I can do about incorporating
 that code into setup.  As is, I'm too busy to wade through MSDNs
 incomplete documentation on this.

I wish I could get to it, but I don't have the time right now :-(. I
would appreciate if someone else can do it, otherwise if I haven't heard
anything I may try to tackle this myself when I *do* have time.


Actually, I looked into this last night, and it turned out to be not too
bad for the common case.  There are still some glitches left, but it's
better than what we have now.  I'll send a preliminary patch to
cygwin-apps shortly.


  Also, thanks again to whoever fixed the text color!

 The ChangeLogs should show the name of the culprit.

Yeah, I'm too lazy to go download it right now and send a private e-mail,
etc... I was hoping they watch the list ;-).


No need to download anything:
http://cygwin.com/cgi-bin/cvsweb.cgi/setup/ChangeLog?rev=2.547content-type=text/x-cvsweb-markupcvsroot=cygwin-apps
BTW, private emails are really discouraged, and most people working on
setup do read this list.
Igor
--
http://cs.nyu.edu/~pechtcha/
 |\  _,,,---,,_ [EMAIL PROTECTED] | [EMAIL PROTECTED]
ZZZzz /,`.-'`'-.  ;-;;,_Igor Peshansky, Ph.D. (name changed!)
|,4-  ) )-,_. ,\ (  `'-'old name: Igor Pechtchanski
   '---''(_/--'  `-'\_) fL  a.k.a JaguaR-R-R-r-r-r-.-.-.  Meow!

Las! je suis sot... -Mais non, tu ne l'es pas, puisque tu t'en rends compte.
But no -- you are no fool; you call yourself a fool, there's proof enough in
that! -- Rostand, Cyrano de Bergerac

--
Unsubscribe info:  http://cygwin.com/ml/#unsubscribe-simple
Problem reports:   http://cygwin.com/problems.html
Documentation: http://cygwin.com/docs.html
FAQ:   http://cygwin.com/faq/



Re: Problems with setup colors in snapshot

2006-05-23 Thread mwoehlke

Brian Dessent wrote:

mwoehlke wrote:

A while back I posted about the colors in setup.exe. I noticed that the
text background color is fixed but that there is still a problem with
the tree structure; specifically, the clickable [+] and [-] icons are,
on my colors, effectively invisible. On some other color schemes I
tried, they *are* 100% invisible.


As the one responsible for this breakage I'll try to look into this soon
and see if it can be fixed.  I thought that I tested this by varying all
the colors of the window but obviously I never tried an inverted scheme.


Thanks... Please let me know if I can help in any way.

And in case you didn't notice my non-directed appreciation, thanks again 
for fixing the text background problem :-).


--
Matthew
Interix, Sphinterix. Cygwin apps don't crash. :-)


--
Unsubscribe info:  http://cygwin.com/ml/#unsubscribe-simple
Problem reports:   http://cygwin.com/problems.html
Documentation: http://cygwin.com/docs.html
FAQ:   http://cygwin.com/faq/



Re: Problems with setup colors in snapshot

2006-05-23 Thread mwoehlke

Igor Peshansky wrote:

On Mon, 22 May 2006, mwoehlke wrote:

Igor Peshansky wrote:

Hmm, weird -- for me the foreground color of the icons is *always*
black.


Unfortunately, a: this isn't the case for me, and b: as you've so
astutely pointed out, if it was that would still be problematic :-). (In
reality, I've learned that *someone* is always going to exercise poor
design and so I usually pick a background light enough for black to be
at least marginally visible against it).


Besides, I was wrong -- the foreground isn't always black, it may
sometimes be the inverted background color.


That was *my* first thought, but obviously that's not what I'm seeing. 
I'd be curious if anyone *does* know what's happening, because I can't 
figure it out (not without digging in the code, at least). :-)



I wish I could get to it, but I don't have the time right now :-(. I
would appreciate if someone else can do it, otherwise if I haven't heard
anything I may try to tackle this myself when I *do* have time.


Actually, I looked into this last night, and it turned out to be not too
bad for the common case.  There are still some glitches left, but it's
better than what we have now.  I'll send a preliminary patch to
cygwin-apps shortly.


Thanks... I'll let you and Brian get your heads together and figure this 
out. Again, let me know if you need any help.



Yeah, I'm too lazy to go download it right now and send a private e-mail,
etc... I was hoping they watch the list ;-).


No need to download anything:
http://cygwin.com/cgi-bin/cvsweb.cgi/setup/ChangeLog?rev=2.547content-type=text/x-cvsweb-markupcvsroot=cygwin-apps 


BTW, private emails are really discouraged, and most people working on
setup do read this list.
Igor


That was the *other* reason I just said it here ;-). Oh... ok, and it 
took a while to actually find the change (in 2.511). Thanks (again) Brian.


And thanks for the pointer, I'll try to remember that. :-)

--
Matthew
Interix, Sphinterix. Cygwin apps don't crash. :-)


--
Unsubscribe info:  http://cygwin.com/ml/#unsubscribe-simple
Problem reports:   http://cygwin.com/problems.html
Documentation: http://cygwin.com/docs.html
FAQ:   http://cygwin.com/faq/



Re: Problems with setup colors in snapshot

2006-05-22 Thread Igor Peshansky
On Mon, 22 May 2006, mwoehlke wrote:

 A while back I posted about the colors in setup.exe. I noticed that the
 text background color is fixed but that there is still a problem with
 the tree structure; specifically, the clickable [+] and [-] icons are,
 on my colors, effectively invisible. On some other color schemes I
 tried, they *are* 100% invisible.

 I didn't see any response last time, so... is anyone able to reproduce
 this? I know P are TC but I don't have time to play with this right now
 :-(.

Yes, this is most likely reproducible (as the foreground color of those
icons is *always* black.

Theoretically, it should be a simple matter of using MaskBlt instead of
BitBlt with the properly configured pen color and the properly set up
mask.  In practice, making this work on all platforms (including Win9x),
correctly, and with the minimum amount of code changes is a big pain.

If you're interested in looking at the code and providing a patch, grep
the setup sources for bitmap_dc (or BitBlt) and patch all those places.

If someone can take one of the bitmaps from the setup sources, and send me
a *complete* small program that draws that bitmap on a window drawing
context with the foreground and background colors coming from the Windows
color scheme, I'll see what I can do about incorporating that code into
setup.  As is, I'm too busy to wade through MSDNs incomplete documentation
on this.

 Also, thanks again to whoever fixed the text color!

The ChangeLogs should show the name of the culprit.

 More info:

 0: I am looking at the latest, 2.529 snapshot of setup.exe, however I
 also see it in the (older) most recent release setup.exe.

 1: It seems that window background is the only color that has any
 effect on the tree color. Therefore, the color must be somehow
 calculated from this(?). This might even be a bug in the UI component
 being used (i.e. a bug in M$ code).

 2: My current window background color is (70,74,80), which makes the
 tree effectively invisible, but tinkering with a screenshot in M$Paint
 shows that it is still a *marginally* different color.

 3: If I pick, e.g. (64,0,128) for my window background color, the tree
 becomes the *exact* same color as the background.

Hmm, weird -- for me the foreground color of the icons is *always* black.
Igor
-- 
http://cs.nyu.edu/~pechtcha/
  |\  _,,,---,,_[EMAIL PROTECTED] | [EMAIL PROTECTED]
ZZZzz /,`.-'`'-.  ;-;;,_Igor Peshansky, Ph.D. (name changed!)
 |,4-  ) )-,_. ,\ (  `'-'   old name: Igor Pechtchanski
'---''(_/--'  `-'\_) fL a.k.a JaguaR-R-R-r-r-r-.-.-.  Meow!

Las! je suis sot... -Mais non, tu ne l'es pas, puisque tu t'en rends compte.
But no -- you are no fool; you call yourself a fool, there's proof enough in
that! -- Rostand, Cyrano de Bergerac

--
Unsubscribe info:  http://cygwin.com/ml/#unsubscribe-simple
Problem reports:   http://cygwin.com/problems.html
Documentation: http://cygwin.com/docs.html
FAQ:   http://cygwin.com/faq/



Re: Problems with setup colors in snapshot

2006-05-22 Thread mwoehlke

Igor Peshansky wrote:

On Mon, 22 May 2006, mwoehlke wrote:

A while back I posted about the colors in setup.exe. I noticed that the
text background color is fixed but that there is still a problem with
the tree structure; specifically, the clickable [+] and [-] icons are,
on my colors, effectively invisible. On some other color schemes I
tried, they *are* 100% invisible.


Yes, this is most likely reproducible (as the foreground color of those
icons is *always* black.


 3: If I pick, e.g. (64,0,128) for my window background color, the
 tree becomes the *exact* same color as the background.

 Hmm, weird -- for me the foreground color of the icons is *always*
 black.

Unfortunately, a: this isn't the case for me, and b: as you've so 
astutely pointed out, if it was that would still be problematic :-). (In 
reality, I've learned that *someone* is always going to exercise poor 
design and so I usually pick a background light enough for black to be 
at least marginally visible against it).


Anyway, I'm going to violate policy and attach a screenshot, as it 
seems prudent in this case. The screenshot is with my normal settings; 
if you open it in a graphics editor with a non-fuzzy bucket fill tool 
(e.g. M$Paint), you can flood the background and see that the tree is 
actually being painted, but in a /very/ subtly different color (in fact 
it is (66,74,80), which I *can* barely on my other monitor at much 
closer range - or if I magnify it). However this is clearly not black ;-).



Theoretically, it should be a simple matter of using MaskBlt instead of
BitBlt with the properly configured pen color and the properly set up
mask.  In practice, making this work on all platforms (including Win9x),
correctly, and with the minimum amount of code changes is a big pain.


I'm using Win2k3 R2 x64 and (see attached screenshot) I can assure you 
it doesn't work on my system. Maybe my OS is the problem? (Hmm... yup, 
the problem has to be Windows ;-).)


Also, I'd test on my XP system but it is consistently dereferencing a 
NULL :-(. (On W2k3 R2 it did that *once* and has since run just fine.)



If you're interested in looking at the code and providing a patch, grep
the setup sources for bitmap_dc (or BitBlt) and patch all those places.

If someone can take one of the bitmaps from the setup sources, and send me
a *complete* small program that draws that bitmap on a window drawing
context with the foreground and background colors coming from the Windows
color scheme, I'll see what I can do about incorporating that code into
setup.  As is, I'm too busy to wade through MSDNs incomplete documentation
on this.


I wish I could get to it, but I don't have the time right now :-(. I 
would appreciate if someone else can do it, otherwise if I haven't heard 
anything I may try to tackle this myself when I *do* have time.



Also, thanks again to whoever fixed the text color!


The ChangeLogs should show the name of the culprit.


Yeah, I'm too lazy to go download it right now and send a private 
e-mail, etc... I was hoping they watch the list ;-).


--
Matthew
All of my signatures are 100% original. Including this one.

--
Unsubscribe info:  http://cygwin.com/ml/#unsubscribe-simple
Problem reports:   http://cygwin.com/problems.html
Documentation: http://cygwin.com/docs.html
FAQ:   http://cygwin.com/faq/

Re: Problems with setup colors in snapshot

2006-05-22 Thread mwoehlke

mwoehlke wrote:

Igor Peshansky wrote:

On Mon, 22 May 2006, mwoehlke wrote:

A while back I posted about the colors in setup.exe. I noticed that the
text background color is fixed but that there is still a problem with
the tree structure; specifically, the clickable [+] and [-] icons are,
on my colors, effectively invisible. On some other color schemes I
tried, they *are* 100% invisible.


Yes, this is most likely reproducible (as the foreground color of those
icons is *always* black.


3: If I pick, e.g. (64,0,128) for my window background color, the
tree becomes the *exact* same color as the background.


Hmm, weird -- for me the foreground color of the icons is *always*
black.


Unfortunately, a: this isn't the case for me, and b: as you've so 
astutely pointed out, if it was that would still be problematic :-). (In 
reality, I've learned that *someone* is always going to exercise poor 
design and so I usually pick a background light enough for black to be 
at least marginally visible against it).



Theoretically, it should be a simple matter of using MaskBlt instead of
BitBlt with the properly configured pen color and the properly set up
mask.  In practice, making this work on all platforms (including Win9x),
correctly, and with the minimum amount of code changes is a big pain.


I'm using Win2k3 R2 x64 and (see attached screenshot) I can assure you 
it doesn't work on my system. Maybe my OS is the problem? (Hmm... yup, 
the problem has to be Windows ;-).)


Also, I'd test on my XP system but it is consistently dereferencing a 
NULL :-(. (On W2k3 R2 it did that *once* and has since run just fine.)


After tweaking and rebuilding 2.529 from the tarball, I can confirm, for 
whatever it's worth, that I see the same behavior on XP. Maybe now that 
I have the sources built I'll be more likely to submit a patch. :-) Time 
permitting, of course.


--
Matthew
All of my signatures are 100% original. Including this one.


--
Unsubscribe info:  http://cygwin.com/ml/#unsubscribe-simple
Problem reports:   http://cygwin.com/problems.html
Documentation: http://cygwin.com/docs.html
FAQ:   http://cygwin.com/faq/



Re: Problems with setup colors in snapshot

2006-05-22 Thread Brian Dessent
mwoehlke wrote:

 A while back I posted about the colors in setup.exe. I noticed that the
 text background color is fixed but that there is still a problem with
 the tree structure; specifically, the clickable [+] and [-] icons are,
 on my colors, effectively invisible. On some other color schemes I
 tried, they *are* 100% invisible.

As the one responsible for this breakage I'll try to look into this soon
and see if it can be fixed.  I thought that I tested this by varying all
the colors of the window but obviously I never tried an inverted scheme.

Brian

--
Unsubscribe info:  http://cygwin.com/ml/#unsubscribe-simple
Problem reports:   http://cygwin.com/problems.html
Documentation: http://cygwin.com/docs.html
FAQ:   http://cygwin.com/faq/