ConEmu / old-issues

Old issues for ConEmu project (import is pending)
http://conemu.github.io/
0 stars 1 forks source link

Correct statusbar fontsize & horizontal separator line #1132

Closed ConEmu closed 9 years ago

ConEmu commented 9 years ago

From nanofo...@gmail.com on July 01, 2013 04:33:28

I using a 16 pixel tall font on 1920x1200 fullscreen. 1200/16=75 nice lines :)

But: ConEmu adds 2 pixel-lines to it's statusbar, so I need to use 14 tall font which is ugly or use 16 tall font with a gap at top of statusbar (which is more ugly...). I understand that 1 line reserved for the horizontal separator but I don't know why the other line for?

If You could modify ConEmu to dynamically add that horizontal lines to statusbar, that would be really cool!

I modified source at 3 places and now statusbar is exactly as tall as the selected font and separator line added dynamically. And "the line unknown for me" is just eliminated :)

------- Options.h: . . .

include "Status.h"

. . . int StatusBarHeight() { return StatusBarFontHeight() + /* 2 */ ((isStatusBarFlags&csf_HorzDelim) ? 1 : 0); }; . . .

-------- OptionsClass.cpp:

LRESULT CSettings::OnButtonClicked(HWND hWnd2, WPARAM wParam, LPARAM lParam) { . . . case cbStatusHorzSep: if (IsChecked(hWnd2,CB)) gpSet->isStatusBarFlags |= csf_HorzDelim; else gpSet->isStatusBarFlags &= ~csf_HorzDelim; gp ConEmu ->mp_Status->UpdateStatusFont(); // NEW! gp ConEmu ->mp_Status->UpdateStatusBar(true); break; . . . } -------- Status.cpp:

void CStatus::PaintStatus(HDC hPaint, LPRECT prcStatus /= NULL/) { . . . int nStatusHeight = rcStatus.bottom-rcStatus.top /* +1 */; . . . }


I know this is a minor thing but maybe the effort to fix it is also minor :)

Thanks for Your work on ConEmu !

Original issue: http://code.google.com/p/conemu-maximus5/issues/detail?id=1133

ConEmu commented 9 years ago

From ConEmu.M...@gmail.com on July 26, 2013 13:44:53

130726

Status: Fixed