summaryrefslogtreecommitdiff
path: root/windowspopover/main.c
diff options
context:
space:
mode:
Diffstat (limited to 'windowspopover/main.c')
-rw-r--r--windowspopover/main.c11
1 files changed, 2 insertions, 9 deletions
diff --git a/windowspopover/main.c b/windowspopover/main.c
index cd9ed7d..43bbb69 100644
--- a/windowspopover/main.c
+++ b/windowspopover/main.c
@@ -177,14 +177,8 @@ LRESULT CALLBACK popoverproc(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
if (dc == NULL)
xpanic("error getting Popover window DC for drawing border", GetLastError());
region = makePopoverRegion(dc, width, height);
- // TODO isolate the brush name to a constant
- // unfortunately FillRgn() doesn't document the COLOR+1 trick as working there
- brush = GetSysColorBrush(COLOR_BTNFACE);
- if (brush == NULL)
- xpanic("error getting Popover background brush", GetLastError());
- if (FillRgn(dc, region, brush) == 0)
- xpanic("error drawing Popover background", GetLastError());
- // TODO use a system color brush?
+ // don't call FillRgn(); WM_ERASEBKGND seems to do this to the non-client area for us already :S (TODO confirm)
+ // TODO arrow is black in wine
brush = (HBRUSH) GetStockObject(BLACK_BRUSH);
if (brush == NULL)
xpanic("error getting Popover border brush", GetLastError());
@@ -241,7 +235,6 @@ LRESULT CALLBACK popoverproc(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
case msgPopoverPrepareLeftRight:
case msgPopoverPrepareTopBottom:
// TODO window edge detection
- // TODO if window was partially offscreen and then moved onscreen, client rect gets fucked up
{
RECT r;
LONG x, y;