cheng/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Make wxBU_EXACTFIT buttons less tall in wxMSW

Browse Source 
The ellipsis buttons of wxFilePickerCtrl (and similar controls) with
wxFLP_SMALL style looked too big because their min height was greater than
that of the accompanying text control.

Fix this by making the buttons as tall as text controls and not as tall as
standard buttons, this seems to be more aesthetically pleasing.
This commit is contained in:
Vadim Zeitlin 2016-06-08 03:19:51 +02:00
parent 0d86c01b8b
commit bd388e9827
1 changed files with 15 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
src/msw/anybutton.cpp
View File

@ -414,7 +414,7 @@ wxSize wxMSWButton::GetFittingSize(wxWindow *win,
{
// We still need some margin or the text would be overwritten, just
// make it as small as possible.
sizeBtn.x += (3*win->GetCharWidth());
sizeBtn.x += 2*win->GetCharWidth();
}
else
{
@ -443,30 +443,23 @@ wxSize wxMSWButton::IncreaseToStdSizeAndCache(wxControl *btn, const wxSize& size
{
wxSize sizeBtn(size);
// The 50x14 button size is documented in the "Recommended sizing and
// spacing" section of MSDN layout article.
//
// Note that we intentionally don't use GetDefaultSize() here, because
// it's inexact -- dialog units depend on this dialog's font.
const wxSize sizeDef = btn->ConvertDialogToPixels(wxSize(50, 14));
// All buttons should have at least the standard size, unless the user
// explicitly wants them to be as small as possible and used wxBU_EXACTFIT
// style to indicate this.
const bool incToStdSize = !btn->HasFlag(wxBU_EXACTFIT);
if ( incToStdSize )
// By default all buttons have at least the standard size.
if ( !btn->HasFlag(wxBU_EXACTFIT) )
{
if ( sizeBtn.x < sizeDef.x )
sizeBtn.x = sizeDef.x;
// The 50x14 button size is documented in the "Recommended sizing and
// spacing" section of MSDN layout article.
//
// Note that we intentionally don't use GetDefaultSize() here, because
// it's inexact -- dialog units depend on this dialog's font.
const wxSize sizeDef = btn->ConvertDialogToPixels(wxSize(50, 14));
sizeBtn.IncTo(sizeDef);
}
// Notice that we really want to make all buttons with text label equally
// high, otherwise they look ugly and the existing code using wxBU_EXACTFIT
// only uses it to control width and not height.
if ( incToStdSize || !btn->GetLabel().empty() )
else // wxBU_EXACTFIT case
{
if ( sizeBtn.y < sizeDef.y )
sizeBtn.y = sizeDef.y;
// Such buttons are typically used alongside a text control or similar,
// so make them as high as it.
sizeBtn.y = EDIT_HEIGHT_FROM_CHAR_HEIGHT(size.y);
}
btn->CacheBestSize(sizeBtn);