public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
From: Jon Turney <jon.turney@dronecode.org.uk>
To: cygwin-apps@cygwin.com
Subject: Re: [PATCH setup] Make Enter in the user URL box cause ADD instead of NEXT
Date: Fri, 08 Dec 2017 15:55:00 -0000	[thread overview]
Message-ID: <d04798f1-36fe-a917-3505-27541596de6f@dronecode.org.uk> (raw)
In-Reply-To: <ed3d4dfa-582e-da72-72f0-11901ea3cd24@cornell.edu>

On 08/12/2017 14:47, Ken Brown wrote:
> On 12/7/2017 11:55 PM, Brian Inglis wrote:
>> On 2017-12-07 13:46, Ken Brown wrote:
>>> In my testing, 'Next' does indeed become the default button after I 
>>> click
>>> outside of the textbox, but there's no visual indication of this.
>>> The MSDN page for DM_SETDEFID
>>> (https://msdn.microsoft.com/en-us/library/windows/desktop/ms645413(v=vs.85).aspx) 
>>> mentions
>>> a different situation where button highlighting doesn't accurately 
>>> reflect the
>>> default button.  In that case it suggests sending a BM_SETSTYLE 
>>> message to
>>> change the border style.
>>> I looked at the documentation for BM_SETSTYLE, but it wasn't obvious 
>>> to me what
>>> do do.
>>
>> Seems like you want to change the button state not style - checking 
>> around you
>> might want to use Button_SetState macro or BM_SETSTATE message TRUE to
>> highlight, FALSE to reset:
>>
>> https://msdn.microsoft.com/en-us/library/windows/desktop/bb849168(v=vs.85).aspx 

No, 'state' is the highlight when the button is pressed, which is 
different to the outline 'style' when the button is the default (ofc, 
MSDN documentation is as clear as mud, as usual)

> Thanks for the suggestions, but I tried various BM_SETSTATE and 
> BM_SETSTYLE messages, and I couldn't find anything that worked: 'Next' 
> remained unhighlighted.

The trick here, as you might notice from the patch in my reply, is that 
there's a window hierarchy involved: the IDD_CHOOSE window is embedded 
in the propsheet window, which owns the 'next' control.

  reply	other threads:[~2017-12-08 15:55 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-04 15:59 Ken Brown
2017-12-05 12:58 ` Jon Turney
2017-12-05 16:03   ` Ken Brown
2017-12-05 17:32     ` Ken Brown
2017-12-07 18:35       ` Jon Turney
2017-12-07 20:46         ` Ken Brown
2017-12-08  4:55           ` Brian Inglis
2017-12-08 14:47             ` Ken Brown
2017-12-08 15:55               ` Jon Turney [this message]
2017-12-08 15:48           ` Jon Turney
2017-12-08 16:14             ` Ken Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=d04798f1-36fe-a917-3505-27541596de6f@dronecode.org.uk \
    --to=jon.turney@dronecode.org.uk \
    --cc=cygwin-apps@cygwin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).