Fix scrollbar not working with mouse, fix being able to click filechooser items outside area

This commit is contained in:
dec05eba
2024-12-29 20:51:56 +01:00
parent a119220961
commit fdeebe2200
2 changed files with 20 additions and 15 deletions

View File

@@ -96,7 +96,12 @@ namespace gsr {
selected_item_background.set_color(get_color_theme().tint_color);
window.draw(selected_item_background);
}
if(!has_parent_with_selected_child_widget() && mouse_over_item == -1 && mgl::FloatRect(item_pos, item_size).contains(mouse_pos)) {
if(!has_parent_with_selected_child_widget() && mouse_over_item == -1 &&
mouse_pos.x >= scissor.position.x && mouse_pos.x <= scissor.position.x + scissor.size.x &&
mouse_pos.y >= scissor.position.y && mouse_pos.y <= scissor.position.y + scissor.size.y &&
mgl::FloatRect(item_pos, item_size).contains(mouse_pos))
{
// mgl::Rectangle selected_item_background(item_size.floor());
// selected_item_background.set_position(item_pos.floor());
// selected_item_background.set_color(mgl::Color(20, 20, 20, 150));

View File

@@ -28,12 +28,12 @@ namespace gsr {
offset.y += scroll_y;
Widget *selected_widget = selected_child_widget;
if(event.type == mgl::Event::MouseButtonPressed || event.type == mgl::Event::MouseButtonReleased) {
if(!mgl::IntRect(scissor_pos, scissor_size).contains({event.mouse_button.x, event.mouse_button.y}))
return true;
} else if(event.type == mgl::Event::MouseMoved) {
if(!mgl::IntRect(scissor_pos, scissor_size).contains({event.mouse_move.x, event.mouse_move.y}))
return true;
if(event.type == mgl::Event::MouseButtonPressed && scrollbar_rect.contains(mgl::vec2f(event.mouse_button.x, event.mouse_button.y))) {
set_widget_as_selected_in_parent();
moving_scrollbar_with_cursor = true;
scrollbar_move_cursor_start_pos = mgl::vec2f(event.mouse_button.x, event.mouse_button.y);
scrollbar_move_cursor_scroll_y_start = scroll_y;
return false;
}
if(event.type == mgl::Event::MouseButtonReleased && moving_scrollbar_with_cursor) {
@@ -42,6 +42,14 @@ namespace gsr {
return false;
}
if(event.type == mgl::Event::MouseButtonPressed || event.type == mgl::Event::MouseButtonReleased) {
if(!mgl::IntRect(scissor_pos, scissor_size).contains({event.mouse_button.x, event.mouse_button.y}))
return true;
} else if(event.type == mgl::Event::MouseMoved) {
if(!mgl::IntRect(scissor_pos, scissor_size).contains({event.mouse_move.x, event.mouse_move.y}))
return true;
}
if(selected_widget) {
if(!selected_widget->on_event(event, window, offset))
return false;
@@ -65,14 +73,6 @@ namespace gsr {
return false;
}
if(event.type == mgl::Event::MouseButtonPressed && scrollbar_rect.contains(mgl::vec2f(event.mouse_button.x, event.mouse_button.y))) {
set_widget_as_selected_in_parent();
moving_scrollbar_with_cursor = true;
scrollbar_move_cursor_start_pos = mgl::vec2f(event.mouse_button.x, event.mouse_button.y);
scrollbar_move_cursor_scroll_y_start = scroll_y;
return false;
}
return true;
}