Minor toolbar button fixes and tweaks

- Fix potential crash on opening recent file from start page
- Increase rounding slightly on edge of toolbar buttons

src/MainEditor/UI/StartPanel.cpp

@@ -210,6 +210,7 @@ class RecentFileButton : public SToolBarButton
 		filename_{ path.fileName() }
 	{
 		action_wx_id_offset_ = index;
+		click_can_delete_    = true;
 
 		wxWindow::SetBackgroundColour(backgroundColour());
 		wxWindow::SetForegroundColour(wxColour(app::isDarkTheme() ? LINK_COLOUR_DARK : LINK_COLOUR_LIGHT));

src/UI/SToolBar/SToolBarButton.cpp

@@ -320,15 +320,15 @@ void SToolBarButton::drawContent(wxGraphicsContext* gc, bool mouse_over)
 	auto width_inner  = width - (2. * pad_outer_);
 	auto height       = GetSize().y;
 	auto height_inner = height - (2. * pad_outer_);
-	auto scale_px     = 1;
+	auto scale_px     = FromDIP(2);
 
 	// Draw toggled border/background
 	if (isChecked())
 	{
 		// Draw border
 		gc->SetBrush(*wxTRANSPARENT_BRUSH);
 		gc->SetPen(wxPen(col_hilight, scale_px));
-		gc->DrawRoundedRectangle(pad_outer_, pad_outer_, width_inner, height_inner, 1);
+		gc->DrawRoundedRectangle(pad_outer_, pad_outer_, width_inner, height_inner, FromDIP(2));
 	}
 
 	// Draw border on mouseover
@@ -345,7 +345,7 @@ void SToolBarButton::drawContent(wxGraphicsContext* gc, bool mouse_over)
 		// Draw border
 		gc->SetBrush(col_trans);
 		gc->SetPen(*wxTRANSPARENT_PEN);
-		gc->DrawRoundedRectangle(pad_outer_, pad_outer_, width_inner, height_inner, 1);
+		gc->DrawRoundedRectangle(pad_outer_, pad_outer_, width_inner, height_inner, FromDIP(2));
 	}
 
 	auto icon = icon_.GetBitmapFor(this);
@@ -496,6 +496,9 @@ void SToolBarButton::onMouseEvent(wxMouseEvent& e)
 			else
 				sendClickedEvent();
 
+			if (click_can_delete_)
+				return;
+
 			pressed_ = false;
 			refresh  = true;
 		}
@@ -505,7 +508,7 @@ void SToolBarButton::onMouseEvent(wxMouseEvent& e)
 			parent_window->SetStatusText("");
 	}
 
-	if (refresh && !IsBeingDeleted())
+	if (refresh)
 	{
 		Update();
 		Refresh();

src/UI/SToolBar/SToolBarButton.h

@@ -56,6 +56,12 @@ class SToolBarButton : public wxControl
 	int  text_offset_ = 0; // Space between icon and text
 	bool exact_fit_   = true;
 
+	// Recent file buttons on the start page can potentially be deleted after
+	// being clicked (as the recent file list is updated). This is needed to
+	// avoid crashes due to the button being deleted while still executing the
+	// mouse event handler.
+	bool click_can_delete_ = false;
+
 	void         setup(bool show_name, string_view icon);
 	void         sendClickedEvent();
 	void         updateSize();