From 38a523be0085a9b529043bddc61e4aee04a6768d Mon Sep 17 00:00:00 2001 From: crupest Date: Tue, 13 Nov 2018 22:24:17 +0800 Subject: Add enhanced bound check. --- CruUI-Generate/cru_ui.hpp | 14 ++++++++++++++ 1 file changed, 14 insertions(+) (limited to 'CruUI-Generate/cru_ui.hpp') diff --git a/CruUI-Generate/cru_ui.hpp b/CruUI-Generate/cru_ui.hpp index d6df9b13..2196c918 100644 --- a/CruUI-Generate/cru_ui.hpp +++ b/CruUI-Generate/cru_ui.hpp @@ -1596,6 +1596,8 @@ namespace cru::ui //Absolute point to local point. Point WindowToControl(const Point& point) const; + // Default implement in Control is test point in border geometry's + // fill and stroke with width of border. virtual bool IsPointInside(const Point& point); @@ -1743,6 +1745,8 @@ namespace cru::ui void RaisePositionChangedEvent(events::PositionChangedEventArgs& args); void RaiseSizeChangedEvent(events::SizeChangedEventArgs& args); + void RegenerateBorderGeometry(); + //*************** region: mouse event *************** virtual void OnMouseEnter(events::MouseEventArgs& args); virtual void OnMouseLeave(events::MouseEventArgs& args); @@ -1847,6 +1851,8 @@ namespace cru::ui bool is_bordered_ = false; BorderProperty border_property_; + Microsoft::WRL::ComPtr border_geometry_ = nullptr; + AnyMap additional_property_map_{}; bool is_focus_on_pressed_ = true; @@ -2094,6 +2100,9 @@ namespace cru::ui //This method has no effect for a window. Use SetClientSize instead. void SetSize(const Size& size) override final; + //Override. If point is in client area, it is in window. + bool IsPointInside(const Point& point) override final; + //*************** region: layout *************** void WindowInvalidateLayout(); @@ -3025,6 +3034,11 @@ namespace cru::ui namespace cru::ui { + inline D2D1_POINT_2F Convert(const Point& point) + { + return D2D1::Point2F(point.x, point.y); + } + inline D2D1_RECT_F Convert(const Rect& rect) { return D2D1::RectF(rect.left, rect.top, rect.left + rect.width, rect.top + rect.height); -- cgit v1.2.3 From 6d0a44597eb6e35993b46642ebf85abd94bca2d7 Mon Sep 17 00:00:00 2001 From: crupest Date: Tue, 13 Nov 2018 22:36:40 +0800 Subject: Add check for IsBordered. --- CruUI-Generate/cru_ui.cpp | 49 ++++++++++++++++++++++++++++++++------------- CruUI-Generate/cru_ui.hpp | 2 +- src/ui/control.cpp | 51 +++++++++++++++++++++++++++++++++-------------- src/ui/control.hpp | 2 +- 4 files changed, 73 insertions(+), 31 deletions(-) (limited to 'CruUI-Generate/cru_ui.hpp') diff --git a/CruUI-Generate/cru_ui.cpp b/CruUI-Generate/cru_ui.cpp index 418d2498..252dfd9a 100644 --- a/CruUI-Generate/cru_ui.cpp +++ b/CruUI-Generate/cru_ui.cpp @@ -982,11 +982,20 @@ namespace cru::ui { if (border_geometry_ != nullptr) { - BOOL contains; - border_geometry_->FillContainsPoint(Convert(point), D2D1::Matrix3x2F::Identity(), &contains); - if (!contains) - border_geometry_->StrokeContainsPoint(Convert(point), GetBorderProperty().GetStrokeWidth(), nullptr, D2D1::Matrix3x2F::Identity(), &contains); - return contains != 0; + if (IsBordered()) + { + BOOL contains; + border_geometry_->FillContainsPoint(Convert(point), D2D1::Matrix3x2F::Identity(), &contains); + if (!contains) + border_geometry_->StrokeContainsPoint(Convert(point), GetBorderProperty().GetStrokeWidth(), nullptr, D2D1::Matrix3x2F::Identity(), &contains); + return contains != 0; + } + else + { + BOOL contains; + border_geometry_->FillContainsPoint(Convert(point), D2D1::Matrix3x2F::Identity(), &contains); + return contains != 0; + } } return false; } @@ -1306,16 +1315,28 @@ namespace cru::ui void Control::RegenerateBorderGeometry() { - const auto bound_rect = GetRect(RectRange::HalfBorder); - const auto bound_rounded_rect = D2D1::RoundedRect(Convert(bound_rect), - GetBorderProperty().GetRadiusX(), - GetBorderProperty().GetRadiusY()); + if (IsBordered()) + { + const auto bound_rect = GetRect(RectRange::HalfBorder); + const auto bound_rounded_rect = D2D1::RoundedRect(Convert(bound_rect), + GetBorderProperty().GetRadiusX(), + GetBorderProperty().GetRadiusY()); - Microsoft::WRL::ComPtr geometry; - ThrowIfFailed( - graph::GraphManager::GetInstance()->GetD2D1Factory()->CreateRoundedRectangleGeometry(bound_rounded_rect, &geometry) - ); - border_geometry_ = std::move(geometry); + Microsoft::WRL::ComPtr geometry; + ThrowIfFailed( + graph::GraphManager::GetInstance()->GetD2D1Factory()->CreateRoundedRectangleGeometry(bound_rounded_rect, &geometry) + ); + border_geometry_ = std::move(geometry); + } + else + { + const auto bound_rect = GetRect(RectRange::Padding); + Microsoft::WRL::ComPtr geometry; + ThrowIfFailed( + graph::GraphManager::GetInstance()->GetD2D1Factory()->CreateRectangleGeometry(Convert(bound_rect), &geometry) + ); + border_geometry_ = std::move(geometry); + } } void Control::OnMouseEnter(MouseEventArgs & args) diff --git a/CruUI-Generate/cru_ui.hpp b/CruUI-Generate/cru_ui.hpp index 2196c918..302ce611 100644 --- a/CruUI-Generate/cru_ui.hpp +++ b/CruUI-Generate/cru_ui.hpp @@ -1851,7 +1851,7 @@ namespace cru::ui bool is_bordered_ = false; BorderProperty border_property_; - Microsoft::WRL::ComPtr border_geometry_ = nullptr; + Microsoft::WRL::ComPtr border_geometry_ = nullptr; AnyMap additional_property_map_{}; diff --git a/src/ui/control.cpp b/src/ui/control.cpp index 9d0afe06..42e5f71f 100644 --- a/src/ui/control.cpp +++ b/src/ui/control.cpp @@ -193,11 +193,20 @@ namespace cru::ui { if (border_geometry_ != nullptr) { - BOOL contains; - border_geometry_->FillContainsPoint(Convert(point), D2D1::Matrix3x2F::Identity(), &contains); - if (!contains) - border_geometry_->StrokeContainsPoint(Convert(point), GetBorderProperty().GetStrokeWidth(), nullptr, D2D1::Matrix3x2F::Identity(), &contains); - return contains != 0; + if (IsBordered()) + { + BOOL contains; + border_geometry_->FillContainsPoint(Convert(point), D2D1::Matrix3x2F::Identity(), &contains); + if (!contains) + border_geometry_->StrokeContainsPoint(Convert(point), GetBorderProperty().GetStrokeWidth(), nullptr, D2D1::Matrix3x2F::Identity(), &contains); + return contains != 0; + } + else + { + BOOL contains; + border_geometry_->FillContainsPoint(Convert(point), D2D1::Matrix3x2F::Identity(), &contains); + return contains != 0; + } } return false; } @@ -517,16 +526,28 @@ namespace cru::ui void Control::RegenerateBorderGeometry() { - const auto bound_rect = GetRect(RectRange::HalfBorder); - const auto bound_rounded_rect = D2D1::RoundedRect(Convert(bound_rect), - GetBorderProperty().GetRadiusX(), - GetBorderProperty().GetRadiusY()); - - Microsoft::WRL::ComPtr geometry; - ThrowIfFailed( - graph::GraphManager::GetInstance()->GetD2D1Factory()->CreateRoundedRectangleGeometry(bound_rounded_rect, &geometry) - ); - border_geometry_ = std::move(geometry); + if (IsBordered()) + { + const auto bound_rect = GetRect(RectRange::HalfBorder); + const auto bound_rounded_rect = D2D1::RoundedRect(Convert(bound_rect), + GetBorderProperty().GetRadiusX(), + GetBorderProperty().GetRadiusY()); + + Microsoft::WRL::ComPtr geometry; + ThrowIfFailed( + graph::GraphManager::GetInstance()->GetD2D1Factory()->CreateRoundedRectangleGeometry(bound_rounded_rect, &geometry) + ); + border_geometry_ = std::move(geometry); + } + else + { + const auto bound_rect = GetRect(RectRange::Padding); + Microsoft::WRL::ComPtr geometry; + ThrowIfFailed( + graph::GraphManager::GetInstance()->GetD2D1Factory()->CreateRectangleGeometry(Convert(bound_rect), &geometry) + ); + border_geometry_ = std::move(geometry); + } } void Control::OnMouseEnter(MouseEventArgs & args) diff --git a/src/ui/control.hpp b/src/ui/control.hpp index 75c10c04..776eacb0 100644 --- a/src/ui/control.hpp +++ b/src/ui/control.hpp @@ -372,7 +372,7 @@ namespace cru::ui bool is_bordered_ = false; BorderProperty border_property_; - Microsoft::WRL::ComPtr border_geometry_ = nullptr; + Microsoft::WRL::ComPtr border_geometry_ = nullptr; AnyMap additional_property_map_{}; -- cgit v1.2.3