Dynamic CButton margin












0















I am using this code to set the initial size of my buttons:



// Now check button sizes
auto *pDC = m_btnPreview.GetDC();
if (pDC != nullptr)
{
pDC->SelectObject(GetFont());
auto sizeActualSize = pDC->GetTextExtent(strButtonText);
sizeActualSize.cx += 10;
if (sizeActualSize.cx > rctOK.Width())
{
// We need to consistently resize both buttons
int iButtonGap = rctCancel.left - rctOK.right;

rctCancel.left = rctCancel.right - sizeActualSize.cx;
rctOK.right = rctCancel.left - iButtonGap;
rctOK.left = rctOK.right - sizeActualSize.cx;

GetDlgItem(IDOK)->MoveWindow(rctCancel);
m_btnPreview.MoveWindow(rctOK);
}

ReleaseDC(pDC);
}


It works fine. I correctly set the font so that the size returned is right. But I have added a value of 10 to pad it a bit so that the button looks nicer:



Moved button



Is there a specific MFC constant for the default gap around button? Rather than fudging it with a value of 10?










share|improve this question


















  • 3





    CButton::GetIdealSize() is propably better suitable and should take any padding defined by the theme into account. That is way better than hardcoding a magic constant (without even DPI-scaling it).

    – zett42
    Nov 15 '18 at 9:14











  • @zett42 I did not know about this! I will try it soon.

    – Andrew Truckle
    Nov 15 '18 at 9:17






  • 1





    @zett42 Worked well, thanks.

    – Andrew Truckle
    Nov 15 '18 at 14:59
















0















I am using this code to set the initial size of my buttons:



// Now check button sizes
auto *pDC = m_btnPreview.GetDC();
if (pDC != nullptr)
{
pDC->SelectObject(GetFont());
auto sizeActualSize = pDC->GetTextExtent(strButtonText);
sizeActualSize.cx += 10;
if (sizeActualSize.cx > rctOK.Width())
{
// We need to consistently resize both buttons
int iButtonGap = rctCancel.left - rctOK.right;

rctCancel.left = rctCancel.right - sizeActualSize.cx;
rctOK.right = rctCancel.left - iButtonGap;
rctOK.left = rctOK.right - sizeActualSize.cx;

GetDlgItem(IDOK)->MoveWindow(rctCancel);
m_btnPreview.MoveWindow(rctOK);
}

ReleaseDC(pDC);
}


It works fine. I correctly set the font so that the size returned is right. But I have added a value of 10 to pad it a bit so that the button looks nicer:



Moved button



Is there a specific MFC constant for the default gap around button? Rather than fudging it with a value of 10?










share|improve this question


















  • 3





    CButton::GetIdealSize() is propably better suitable and should take any padding defined by the theme into account. That is way better than hardcoding a magic constant (without even DPI-scaling it).

    – zett42
    Nov 15 '18 at 9:14











  • @zett42 I did not know about this! I will try it soon.

    – Andrew Truckle
    Nov 15 '18 at 9:17






  • 1





    @zett42 Worked well, thanks.

    – Andrew Truckle
    Nov 15 '18 at 14:59














0












0








0








I am using this code to set the initial size of my buttons:



// Now check button sizes
auto *pDC = m_btnPreview.GetDC();
if (pDC != nullptr)
{
pDC->SelectObject(GetFont());
auto sizeActualSize = pDC->GetTextExtent(strButtonText);
sizeActualSize.cx += 10;
if (sizeActualSize.cx > rctOK.Width())
{
// We need to consistently resize both buttons
int iButtonGap = rctCancel.left - rctOK.right;

rctCancel.left = rctCancel.right - sizeActualSize.cx;
rctOK.right = rctCancel.left - iButtonGap;
rctOK.left = rctOK.right - sizeActualSize.cx;

GetDlgItem(IDOK)->MoveWindow(rctCancel);
m_btnPreview.MoveWindow(rctOK);
}

ReleaseDC(pDC);
}


It works fine. I correctly set the font so that the size returned is right. But I have added a value of 10 to pad it a bit so that the button looks nicer:



Moved button



Is there a specific MFC constant for the default gap around button? Rather than fudging it with a value of 10?










share|improve this question














I am using this code to set the initial size of my buttons:



// Now check button sizes
auto *pDC = m_btnPreview.GetDC();
if (pDC != nullptr)
{
pDC->SelectObject(GetFont());
auto sizeActualSize = pDC->GetTextExtent(strButtonText);
sizeActualSize.cx += 10;
if (sizeActualSize.cx > rctOK.Width())
{
// We need to consistently resize both buttons
int iButtonGap = rctCancel.left - rctOK.right;

rctCancel.left = rctCancel.right - sizeActualSize.cx;
rctOK.right = rctCancel.left - iButtonGap;
rctOK.left = rctOK.right - sizeActualSize.cx;

GetDlgItem(IDOK)->MoveWindow(rctCancel);
m_btnPreview.MoveWindow(rctOK);
}

ReleaseDC(pDC);
}


It works fine. I correctly set the font so that the size returned is right. But I have added a value of 10 to pad it a bit so that the button looks nicer:



Moved button



Is there a specific MFC constant for the default gap around button? Rather than fudging it with a value of 10?







button mfc






share|improve this question













share|improve this question











share|improve this question




share|improve this question










asked Nov 15 '18 at 8:54









Andrew TruckleAndrew Truckle

5,72252353




5,72252353








  • 3





    CButton::GetIdealSize() is propably better suitable and should take any padding defined by the theme into account. That is way better than hardcoding a magic constant (without even DPI-scaling it).

    – zett42
    Nov 15 '18 at 9:14











  • @zett42 I did not know about this! I will try it soon.

    – Andrew Truckle
    Nov 15 '18 at 9:17






  • 1





    @zett42 Worked well, thanks.

    – Andrew Truckle
    Nov 15 '18 at 14:59














  • 3





    CButton::GetIdealSize() is propably better suitable and should take any padding defined by the theme into account. That is way better than hardcoding a magic constant (without even DPI-scaling it).

    – zett42
    Nov 15 '18 at 9:14











  • @zett42 I did not know about this! I will try it soon.

    – Andrew Truckle
    Nov 15 '18 at 9:17






  • 1





    @zett42 Worked well, thanks.

    – Andrew Truckle
    Nov 15 '18 at 14:59








3




3





CButton::GetIdealSize() is propably better suitable and should take any padding defined by the theme into account. That is way better than hardcoding a magic constant (without even DPI-scaling it).

– zett42
Nov 15 '18 at 9:14





CButton::GetIdealSize() is propably better suitable and should take any padding defined by the theme into account. That is way better than hardcoding a magic constant (without even DPI-scaling it).

– zett42
Nov 15 '18 at 9:14













@zett42 I did not know about this! I will try it soon.

– Andrew Truckle
Nov 15 '18 at 9:17





@zett42 I did not know about this! I will try it soon.

– Andrew Truckle
Nov 15 '18 at 9:17




1




1





@zett42 Worked well, thanks.

– Andrew Truckle
Nov 15 '18 at 14:59





@zett42 Worked well, thanks.

– Andrew Truckle
Nov 15 '18 at 14:59












0






active

oldest

votes











Your Answer






StackExchange.ifUsing("editor", function () {
StackExchange.using("externalEditor", function () {
StackExchange.using("snippets", function () {
StackExchange.snippets.init();
});
});
}, "code-snippets");

StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "1"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});

function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: true,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});


}
});














draft saved

draft discarded


















StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53315606%2fdynamic-cbutton-margin%23new-answer', 'question_page');
}
);

Post as a guest















Required, but never shown

























0






active

oldest

votes








0






active

oldest

votes









active

oldest

votes






active

oldest

votes
















draft saved

draft discarded




















































Thanks for contributing an answer to Stack Overflow!


  • Please be sure to answer the question. Provide details and share your research!

But avoid



  • Asking for help, clarification, or responding to other answers.

  • Making statements based on opinion; back them up with references or personal experience.


To learn more, see our tips on writing great answers.




draft saved


draft discarded














StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53315606%2fdynamic-cbutton-margin%23new-answer', 'question_page');
}
);

Post as a guest















Required, but never shown





















































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown

































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown







Popular posts from this blog

Bressuire

Vorschmack

Quarantine