From 4cfb31bf6abb2d42181e78e8d0758cf74a8a774a Mon Sep 17 00:00:00 2001 From: Daniel Wilhelm Date: Fri, 18 Apr 2014 17:30:04 +0200 Subject: 6.0 --- wx+/grid.cpp | 423 +++++++++++++++++++++++++----------------------- wx+/grid.h | 4 + wx+/image_resources.cpp | 2 + wx+/popup_dlg.cpp | 6 +- 4 files changed, 230 insertions(+), 205 deletions(-) (limited to 'wx+') diff --git a/wx+/grid.cpp b/wx+/grid.cpp index 87f81dda..95baaf3b 100644 --- a/wx+/grid.cpp +++ b/wx+/grid.cpp @@ -246,7 +246,7 @@ public: Connect(wxEVT_ERASE_BACKGROUND, wxEraseEventHandler(SubWindow::onEraseBackGround), nullptr, this); //SetDoubleBuffered(true); slow as hell! - + SetBackgroundStyle(wxBG_STYLE_PAINT); Connect(wxEVT_SET_FOCUS, wxFocusEventHandler(SubWindow::onFocus), nullptr, this); @@ -263,8 +263,6 @@ public: Connect(wxEVT_MOUSEWHEEL, wxMouseEventHandler(SubWindow::onMouseWheel ), nullptr, this); Connect(wxEVT_MOUSE_CAPTURE_LOST, wxMouseCaptureLostEventHandler(SubWindow::onMouseCaptureLost), nullptr, this); - Connect(wxEVT_CHAR, wxKeyEventHandler(SubWindow::onChar ), nullptr, this); - Connect(wxEVT_KEY_UP, wxKeyEventHandler(SubWindow::onKeyUp ), nullptr, this); Connect(wxEVT_KEY_DOWN, wxKeyEventHandler(SubWindow::onKeyDown), nullptr, this); assert(GetClientAreaOrigin() == wxPoint()); //generally assumed when dealing with coordinates below @@ -297,7 +295,7 @@ protected: SetToolTip(new wxToolTip(L"a b\n\ a b")); //ugly, but is working (on Windows) tt = GetToolTip(); //should be bound by now - assert(tt); + assert(tt); if (tt) tt->SetTip(text); } @@ -318,9 +316,12 @@ private: virtual void onMouseMovement (wxMouseEvent& event) { event.Skip(); } virtual void onLeaveWindow (wxMouseEvent& event) { event.Skip(); } virtual void onMouseCaptureLost(wxMouseCaptureLostEvent& event) { event.Skip(); } - virtual void onChar (wxKeyEvent& event) { event.Skip(); } - virtual void onKeyUp (wxKeyEvent& event) { event.Skip(); } - virtual void onKeyDown(wxKeyEvent& event) { event.Skip(); } + + void onKeyDown(wxKeyEvent& event) + { + if (!sendEventNow(event)) //let parent collect all key events + event.Skip(); + } void onMouseWheel(wxMouseEvent& event) { @@ -332,10 +333,8 @@ private: On OS X there is no such propagation! => we need a redirection (the same wxGrid implements) */ - if (wxEvtHandler* evtHandler = parent_.GetEventHandler()) - if (evtHandler->ProcessEvent(event)) - return; - event.Skip(); + if (!sendEventNow(event)) + event.Skip(); } void onPaintEvent(wxPaintEvent& event) @@ -844,53 +843,15 @@ public: ~MainWin() { assert(!gridUpdatePending); } - void makeRowVisible(size_t row) - { - const wxRect labelRect = rowLabelWin_.getRowLabelArea(row); //returns empty rect if column not found - if (labelRect.height > 0) - { - int scrollPosX = 0; - refParent().GetViewStart(&scrollPosX, nullptr); - - int pixelsPerUnitY = 0; - refParent().GetScrollPixelsPerUnit(nullptr, &pixelsPerUnitY); - if (pixelsPerUnitY <= 0) return; - - const int clientPosY = refParent().CalcScrolledPosition(labelRect.GetTopLeft()).y; - if (clientPosY < 0) - { - const int scrollPosY = labelRect.GetTopLeft().y / pixelsPerUnitY; - refParent().Scroll(scrollPosX, scrollPosY); - refParent().updateWindowSizes(); //may show horizontal scroll bar - } - else if (clientPosY + labelRect.GetHeight() > rowLabelWin_.GetClientSize().GetHeight()) - { - auto execScroll = [&](int clientHeight) - { - const int scrollPosY = std::ceil((labelRect.GetTopLeft().y - clientHeight + - labelRect.GetHeight()) / static_cast(pixelsPerUnitY)); - refParent().Scroll(scrollPosX, scrollPosY); - refParent().updateWindowSizes(); //may show horizontal scroll bar - }; - - const int clientHeightBefore = rowLabelWin_.GetClientSize().GetHeight(); - execScroll(clientHeightBefore); - - //client height may decrease after scroll due to a new horizontal scrollbar, resulting in a partially visible last row - const int clientHeightAfter = rowLabelWin_.GetClientSize().GetHeight(); - if (clientHeightAfter < clientHeightBefore) - execScroll(clientHeightAfter); - } - } - } + size_t getCursor() const { return cursorRow; } + size_t getAnchor() const { return selectionAnchor; } - void setCursor(size_t row) + void setCursor(size_t newCursorRow, size_t newAnchorRow) { - cursorRow = row; + cursorRow = newCursorRow; + selectionAnchor = newAnchorRow; activeSelection.reset(); //e.g. user might search with F3 while holding down left mouse button - selectionAnchor = row; } - size_t getCursor() const { return cursorRow; } private: virtual void render(wxDC& dc, const wxRect& rect) @@ -923,12 +884,13 @@ private: if (auto prov = refParent().getDataProvider()) { + RecursiveDcClipper dummy2(dc, rect); //do NOT draw background on cells outside of invalidated rect invalidating foreground text! + //draw background lines for (int row = rowFirst; row < rowLast; ++row) { const wxRect rowRect(cellAreaTL + wxPoint(0, row * rowHeight), wxSize(totalWidth, rowHeight)); - RecursiveDcClipper dummy2(dc, rowRect); //solve issues with drawBackground() painting in area outside of rect - //(which is not also refreshed by renderCell()) -> keep small scope! + RecursiveDcClipper dummy3(dc, rowRect); prov->renderRowBackgound(dc, rowRect, row, refParent().IsThisEnabled(), drawAsSelected(row)); } @@ -942,8 +904,7 @@ private: for (int row = rowFirst; row < rowLast; ++row) { const wxRect cellRect(cellAreaTL.x, cellAreaTL.y + row * rowHeight, cw.width_, rowHeight); - RecursiveDcClipper clip(dc, cellRect); - + RecursiveDcClipper dummy3(dc, cellRect); prov->renderCell(dc, cellRect, row, cw.type_, drawAsSelected(row)); } cellAreaTL.x += cw.width_; @@ -1095,144 +1056,6 @@ private: event.Skip(); } - virtual void onKeyDown(wxKeyEvent& event) - { - int keyCode = event.GetKeyCode(); - if (GetLayoutDirection() == wxLayout_RightToLeft) - { - if (keyCode == WXK_LEFT) - keyCode = WXK_RIGHT; - else if (keyCode == WXK_RIGHT) - keyCode = WXK_LEFT; - else if (keyCode == WXK_NUMPAD_LEFT) - keyCode = WXK_NUMPAD_RIGHT; - else if (keyCode == WXK_NUMPAD_RIGHT) - keyCode = WXK_NUMPAD_LEFT; - } - - const ptrdiff_t rowCount = refParent().getRowCount(); - if (rowCount <= 0) - { - event.Skip(); - return; - } - - auto setSingleSelection = [&](ptrdiff_t row) - { - numeric::confine(row, 0, rowCount - 1); - refParent().setGridCursor(row); //behave like an "external" set cursor! - }; - - auto setSelectionRange = [&](ptrdiff_t row) - { - //unlike "setSingleSelection" this function doesn't seem to belong into Grid: management of selectionAnchor should be local - - numeric::confine(row, 0, rowCount - 1); - - refParent().selection.clear(); //clear selection, do NOT fire event - refParent().selectRangeAndNotify(selectionAnchor, row); //set new selection + fire event - - cursorRow = row; //don't call setCursor() since it writes to "selectionAnchor"! - this->makeRowVisible(row); - refParent().Refresh(); - }; - - switch (keyCode) - { - case WXK_UP: - case WXK_NUMPAD_UP: - if (event.ShiftDown()) - setSelectionRange(cursorRow - 1); - else if (event.ControlDown()) - refParent().scrollDelta(0, -1); - else - setSingleSelection(cursorRow - 1); - return; //swallow event: wxScrolledWindow, wxWidgets 2.9.3 on Kubuntu x64 processes arrow keys: prevent this! - - case WXK_DOWN: - case WXK_NUMPAD_DOWN: - if (event.ShiftDown()) - setSelectionRange(cursorRow + 1); - else if (event.ControlDown()) - refParent().scrollDelta(0, 1); - else - setSingleSelection(cursorRow + 1); - return; //swallow event - - case WXK_LEFT: - case WXK_NUMPAD_LEFT: - if (event.ControlDown()) - refParent().scrollDelta(-1, 0); - else if (event.ShiftDown()) - ; - else - setSingleSelection(cursorRow); - return; - - case WXK_RIGHT: - case WXK_NUMPAD_RIGHT: - if (event.ControlDown()) - refParent().scrollDelta(1, 0); - else if (event.ShiftDown()) - ; - else - setSingleSelection(cursorRow); - return; - - case WXK_HOME: - case WXK_NUMPAD_HOME: - if (event.ShiftDown()) - setSelectionRange(0); - else if (event.ControlDown()) - setSingleSelection(0); - else - setSingleSelection(0); - return; - - case WXK_END: - case WXK_NUMPAD_END: - if (event.ShiftDown()) - setSelectionRange(rowCount - 1); - else if (event.ControlDown()) - setSingleSelection(rowCount - 1); - else - setSingleSelection(rowCount - 1); - return; - - case WXK_PAGEUP: - case WXK_NUMPAD_PAGEUP: - if (event.ShiftDown()) - setSelectionRange(cursorRow - GetClientSize().GetHeight() / rowLabelWin_.getRowHeight()); - else if (event.ControlDown()) - ; - else - setSingleSelection(cursorRow - GetClientSize().GetHeight() / rowLabelWin_.getRowHeight()); - return; - - case WXK_PAGEDOWN: - case WXK_NUMPAD_PAGEDOWN: - if (event.ShiftDown()) - setSelectionRange(cursorRow + GetClientSize().GetHeight() / rowLabelWin_.getRowHeight()); - else if (event.ControlDown()) - ; - else - setSingleSelection(cursorRow + GetClientSize().GetHeight() / rowLabelWin_.getRowHeight()); - return; - - case 'A': //Ctrl + A - select all - if (event.ControlDown()) - refParent().selectRangeAndNotify(0, rowCount); - break; - - case WXK_NUMPAD_ADD: //CTRL + '+' - auto-size all - if (event.ControlDown()) - refParent().autoSizeColumns(ALLOW_GRID_EVENT); - return; - } - - event.Skip(); - } - virtual void onFocus(wxFocusEvent& event) { Refresh(); event.Skip(); } class MouseSelection : private wxEvtHandler @@ -1402,16 +1225,18 @@ Grid::Grid(wxWindow* parent, return labelFont.GetPixelSize().GetHeight(); }(); - Connect(wxEVT_PAINT, wxPaintEventHandler(Grid::onPaintEvent ), nullptr, this); - Connect(wxEVT_ERASE_BACKGROUND, wxEraseEventHandler(Grid::onEraseBackGround), nullptr, this); - Connect(wxEVT_SIZE, wxSizeEventHandler (Grid::onSizeEvent ), nullptr, this); - SetTargetWindow(mainWin_); SetInitialSize(size); //"Most controls will use this to set their initial size" -> why not assert(GetClientSize() == GetSize()); //borders are NOT allowed for Grid //reason: updateWindowSizes() wants to use "GetSize()" as a "GetClientSize()" including scrollbars + + Connect(wxEVT_PAINT, wxPaintEventHandler(Grid::onPaintEvent ), nullptr, this); + Connect(wxEVT_ERASE_BACKGROUND, wxEraseEventHandler(Grid::onEraseBackGround), nullptr, this); + Connect(wxEVT_SIZE, wxSizeEventHandler (Grid::onSizeEvent ), nullptr, this); + + Connect(wxEVT_KEY_DOWN, wxKeyEventHandler(Grid::onKeyDown), nullptr, this); } @@ -1574,6 +1399,144 @@ wxSize Grid::GetSizeAvailableForScrollTarget(const wxSize& size) void Grid::onPaintEvent(wxPaintEvent& event) { wxPaintDC dc(this); } +void Grid::onKeyDown(wxKeyEvent& event) +{ + int keyCode = event.GetKeyCode(); + if (GetLayoutDirection() == wxLayout_RightToLeft) + { + if (keyCode == WXK_LEFT) + keyCode = WXK_RIGHT; + else if (keyCode == WXK_RIGHT) + keyCode = WXK_LEFT; + else if (keyCode == WXK_NUMPAD_LEFT) + keyCode = WXK_NUMPAD_RIGHT; + else if (keyCode == WXK_NUMPAD_RIGHT) + keyCode = WXK_NUMPAD_LEFT; + } + + const ptrdiff_t rowCount = getRowCount(); + const ptrdiff_t cursorRow = mainWin_->getCursor(); + + auto moveCursorTo = [&](ptrdiff_t row) + { + if (rowCount > 0) + { + numeric::confine(row, 0, rowCount - 1); + setGridCursor(row); + } + }; + + auto selectWithCursorTo = [&](ptrdiff_t row) + { + if (rowCount > 0) + { + numeric::confine(row, 0, rowCount - 1); + selectWithCursor(row); + } + }; + + switch (keyCode) + { + //case WXK_TAB: + // if (Navigate(event.ShiftDown() ? wxNavigationKeyEvent::IsBackward : wxNavigationKeyEvent::IsForward)) + // return; + // break; + + case WXK_UP: + case WXK_NUMPAD_UP: + if (event.ShiftDown()) + selectWithCursorTo(cursorRow - 1); + else if (event.ControlDown()) + scrollDelta(0, -1); + else + moveCursorTo(cursorRow - 1); + return; //swallow event: wxScrolledWindow, wxWidgets 2.9.3 on Kubuntu x64 processes arrow keys: prevent this! + + case WXK_DOWN: + case WXK_NUMPAD_DOWN: + if (event.ShiftDown()) + selectWithCursorTo(cursorRow + 1); + else if (event.ControlDown()) + scrollDelta(0, 1); + else + moveCursorTo(cursorRow + 1); + return; //swallow event + + case WXK_LEFT: + case WXK_NUMPAD_LEFT: + if (event.ControlDown()) + scrollDelta(-1, 0); + else if (event.ShiftDown()) + ; + else + moveCursorTo(cursorRow); + return; + + case WXK_RIGHT: + case WXK_NUMPAD_RIGHT: + if (event.ControlDown()) + scrollDelta(1, 0); + else if (event.ShiftDown()) + ; + else + moveCursorTo(cursorRow); + return; + + case WXK_HOME: + case WXK_NUMPAD_HOME: + if (event.ShiftDown()) + selectWithCursorTo(0); + else if (event.ControlDown()) + moveCursorTo(0); + else + moveCursorTo(0); + return; + + case WXK_END: + case WXK_NUMPAD_END: + if (event.ShiftDown()) + selectWithCursorTo(rowCount - 1); + else if (event.ControlDown()) + moveCursorTo(rowCount - 1); + else + moveCursorTo(rowCount - 1); + return; + + case WXK_PAGEUP: + case WXK_NUMPAD_PAGEUP: + if (event.ShiftDown()) + selectWithCursorTo(cursorRow - GetClientSize().GetHeight() / rowLabelWin_->getRowHeight()); + else if (event.ControlDown()) + ; + else + moveCursorTo(cursorRow - GetClientSize().GetHeight() / rowLabelWin_->getRowHeight()); + return; + + case WXK_PAGEDOWN: + case WXK_NUMPAD_PAGEDOWN: + if (event.ShiftDown()) + selectWithCursorTo(cursorRow + GetClientSize().GetHeight() / rowLabelWin_->getRowHeight()); + else if (event.ControlDown()) + ; + else + moveCursorTo(cursorRow + GetClientSize().GetHeight() / rowLabelWin_->getRowHeight()); + return; + + case 'A': //Ctrl + A - select all + if (event.ControlDown()) + selectRangeAndNotify(0, rowCount); + break; + + case WXK_NUMPAD_ADD: //CTRL + '+' - auto-size all + if (event.ControlDown()) + autoSizeColumns(ALLOW_GRID_EVENT); + return; + } + + event.Skip(); +} + + void Grid::setColumnLabelHeight(int height) { colLabelHeight = std::max(0, height); @@ -1939,8 +1902,8 @@ wxRect Grid::getCellArea(size_t row, ColumnType colType) const void Grid::setGridCursor(size_t row) { - mainWin_->setCursor(row); - mainWin_->makeRowVisible(row); + mainWin_->setCursor(row, row); + makeRowVisible(row); selection.clear(); //clear selection, do NOT fire event selectRangeAndNotify(row, row); //set new selection + fire event @@ -1950,6 +1913,62 @@ void Grid::setGridCursor(size_t row) } +void Grid::selectWithCursor(ptrdiff_t row) +{ + const size_t anchorRow = mainWin_->getAnchor(); + + mainWin_->setCursor(row, anchorRow); + makeRowVisible(row); + + selection.clear(); //clear selection, do NOT fire event + selectRangeAndNotify(anchorRow, row); //set new selection + fire event + + mainWin_->Refresh(); + rowLabelWin_->Refresh(); +} + + +void Grid::makeRowVisible(size_t row) +{ + const wxRect labelRect = rowLabelWin_->getRowLabelArea(row); //returns empty rect if column not found + if (labelRect.height > 0) + { + int scrollPosX = 0; + GetViewStart(&scrollPosX, nullptr); + + int pixelsPerUnitY = 0; + GetScrollPixelsPerUnit(nullptr, &pixelsPerUnitY); + if (pixelsPerUnitY <= 0) return; + + const int clientPosY = CalcScrolledPosition(labelRect.GetTopLeft()).y; + if (clientPosY < 0) + { + const int scrollPosY = labelRect.GetTopLeft().y / pixelsPerUnitY; + Scroll(scrollPosX, scrollPosY); + updateWindowSizes(); //may show horizontal scroll bar + } + else if (clientPosY + labelRect.GetHeight() > rowLabelWin_->GetClientSize().GetHeight()) + { + auto execScroll = [&](int clientHeight) + { + const int scrollPosY = std::ceil((labelRect.GetTopLeft().y - clientHeight + + labelRect.GetHeight()) / static_cast(pixelsPerUnitY)); + Scroll(scrollPosX, scrollPosY); + updateWindowSizes(); //may show horizontal scroll bar + }; + + const int clientHeightBefore = rowLabelWin_->GetClientSize().GetHeight(); + execScroll(clientHeightBefore); + + //client height may decrease after scroll due to a new horizontal scrollbar, resulting in a partially visible last row + const int clientHeightAfter = rowLabelWin_->GetClientSize().GetHeight(); + if (clientHeightAfter < clientHeightBefore) + execScroll(clientHeightAfter); + } + } +} + + void Grid::selectRangeAndNotify(ptrdiff_t rowFrom, ptrdiff_t rowTo, bool positive) { //sort + convert to half-open range diff --git a/wx+/grid.h b/wx+/grid.h index 9bbb6c99..aefcb549 100644 --- a/wx+/grid.h +++ b/wx+/grid.h @@ -199,9 +199,13 @@ private: void onPaintEvent(wxPaintEvent& event); void onEraseBackGround(wxEraseEvent& event) {} //[!] void onSizeEvent(wxSizeEvent& event) { updateWindowSizes(); event.Skip(); } + void onKeyDown(wxKeyEvent& event); void updateWindowSizes(bool updateScrollbar = true); + void selectWithCursor(ptrdiff_t row); + void makeRowVisible(size_t row); + void redirectRowLabelEvent(wxMouseEvent& event); virtual wxSize GetSizeAvailableForScrollTarget(const wxSize& size); //required since wxWidgets 2.9 if SetTargetWindow() is used diff --git a/wx+/image_resources.cpp b/wx+/image_resources.cpp index 6da12c17..7b74c161 100644 --- a/wx+/image_resources.cpp +++ b/wx+/image_resources.cpp @@ -62,6 +62,8 @@ private: void GlobalResources::init(const Zstring& filename) { + assert(bitmaps.empty() && anims.empty()); + wxFFileInputStream input(utfCvrtTo(filename)); if (input.IsOk()) //if not... we don't want to react too harsh here { diff --git a/wx+/popup_dlg.cpp b/wx+/popup_dlg.cpp index 2c7887fa..d7d1960e 100644 --- a/wx+/popup_dlg.cpp +++ b/wx+/popup_dlg.cpp @@ -62,11 +62,11 @@ void setBestInitialSize(wxTextCtrl& ctrl, const wxString& text, wxSize maxSize) } #if defined ZEN_WIN || defined ZEN_LINUX - const int rowGap = 0; + const int rowGap = 0; #elif defined ZEN_MAC - const int rowGap = 1; + const int rowGap = 1; #endif - const wxSize bestSize(bestWidth + scrollbarWidth, std::min(rowCount * (rowHeight + rowGap), maxSize.y)); + const wxSize bestSize(bestWidth + scrollbarWidth, std::min(rowCount * (rowHeight + rowGap), maxSize.y)); ctrl.SetMinSize(bestSize); //alas, SetMinClientSize() is just not working! } } -- cgit