From 91797f570f4a0f5815bebe030a52575e8338f69f Mon Sep 17 00:00:00 2001 From: Jesper Juhl Date: Mon, 2 Feb 2026 15:41:40 +0100 Subject: [PATCH 1/3] Remove some unneeded blank lines from CSS For a bit more internal consistency and a few fewer bytes transfered to browsers per request. --- doc/doxygen-awesome.css | 30 ------------------------------ 1 file changed, 30 deletions(-) diff --git a/doc/doxygen-awesome.css b/doc/doxygen-awesome.css index af68d5fea..270e0e874 100644 --- a/doc/doxygen-awesome.css +++ b/doc/doxygen-awesome.css @@ -346,7 +346,6 @@ a.anchor { /* Title and top navigation */ - #top { background: var(--header-background); border-bottom: 1px solid var(--separator-color); @@ -538,11 +537,9 @@ a.anchor { padding: 0 var(--spacing-large); } - /* Search box */ - #MSearchBox { height: var(--searchbar-height); background: var(--searchbar-background); @@ -776,7 +773,6 @@ html.dark-mode iframe#MSearchResults { /* Tree view */ - #side-nav { padding: 0 !important; background: var(--side-nav-background); @@ -841,7 +837,6 @@ html.dark-mode iframe#MSearchResults { background: var(--primary-color); } - #nav-tree a { color: var(--side-nav-foreground) !important; font-weight: normal; @@ -886,7 +881,6 @@ html.dark-mode iframe#MSearchResults { /* Contents */ - div.header { border-bottom: 1px solid var(--separator-color); background-color: var(--page-background-color); @@ -1095,7 +1089,6 @@ h1.glow, h2.glow, h3.glow, h4.glow, h5.glow, h6.glow { /* Table of Contents */ - div.contents .toc { max-height: var(--toc-max-height); min-width: var(--toc-width); @@ -1145,7 +1138,6 @@ div.toc li a.aboveActive { color: var(--page-secondary-foreground-color) !important; } - @media screen and (max-width: 999px) { div.contents .toc { max-height: 45vh; @@ -1213,7 +1205,6 @@ div.toc li a.aboveActive { /* Code & Fragments */ - code, div.fragment, pre.fragment { border-radius: var(--border-radius-small); border: 1px solid var(--separator-color); @@ -1358,7 +1349,6 @@ div.line.glow { /* dl warning, attention, note, deprecated, bug, ... */ - dl.bug dt a, dl.deprecated dt a, dl.todo dt a { font-weight: bold !important; } @@ -1447,7 +1437,6 @@ dl.invariant dt, dl.pre dt, dl.post dt { /* memitem */ - div.memdoc, div.memproto, h2.memtitle { box-shadow: none; background-image: none; @@ -1587,7 +1576,6 @@ table.mlabels > tbody > tr:first-child { /* reflist */ - dl.reflist { box-shadow: var(--box-shadow); border-radius: var(--border-radius-medium); @@ -1614,7 +1602,6 @@ dl.reflist dt { color: var(--page-foreground-color) } - dl.reflist dd { background: none; } @@ -1622,7 +1609,6 @@ dl.reflist dd { /* Table */ - .contents table:not(.memberdecls):not(.mlabels):not(.fieldtable):not(.memname), .contents table:not(.memberdecls):not(.mlabels):not(.fieldtable):not(.memname) tbody { display: inline-block; @@ -1895,7 +1881,6 @@ table.memberdecls img { table.memberdecls img[src="closed.png"], div.dynheader img[src="closed.png"] { transform: rotate(-90deg); - } .compoundTemplParams { @@ -1905,7 +1890,6 @@ div.dynheader img[src="closed.png"] { } @media screen and (max-width: 767px) { - table.memberdecls .memItemLeft, table.memberdecls .memItemRight, table.memberdecls .mdescLeft, @@ -1980,11 +1964,9 @@ div.dynheader img[src="closed.png"] { } } - /* Horizontal Rule */ - hr { margin-top: var(--spacing-large); margin-bottom: var(--spacing-large); @@ -2116,7 +2098,6 @@ html.dark-mode .iconfopen, html.dark-mode .iconfclosed { /* Class list */ - .classindex dl.odd { background: var(--odd-color); border-radius: var(--border-radius-small); @@ -2129,7 +2110,6 @@ html.dark-mode .iconfopen, html.dark-mode .iconfclosed { /* Class Index Doxygen 1.8 */ - table.classindex { margin-left: 0; margin-right: 0; @@ -2156,7 +2136,6 @@ div.qindex { /* Footer and nav-path */ - #nav-path { width: 100%; } @@ -2243,7 +2222,6 @@ li.navelem:first-child:before { /* Scrollbars for Webkit */ - #nav-tree::-webkit-scrollbar, div.fragment::-webkit-scrollbar, pre.fragment::-webkit-scrollbar, @@ -2326,7 +2304,6 @@ div.contents .toc { /* Scrollbars for Firefox */ - #nav-tree, div.fragment, pre.fragment, @@ -2343,7 +2320,6 @@ div.contents .toc, /* Optional Dark mode toggle button */ - doxygen-awesome-dark-mode-toggle { display: inline-block; margin: 0 0 0 var(--spacing-small); @@ -2442,7 +2418,6 @@ doxygen-awesome-fragment-copy-button.success { /* Optional paragraph link button */ - a.anchorlink { font-size: 90%; margin-left: var(--spacing-small); @@ -2473,7 +2448,6 @@ h2:hover a.anchorlink, h1:hover a.anchorlink, h3:hover a.anchorlink, h4:hover a. /* Optional tab feature */ - .tabbed > ul { padding-inline-start: 0px; margin: 0; @@ -2576,11 +2550,9 @@ h2:hover a.anchorlink, h1:hover a.anchorlink, h3:hover a.anchorlink, h4:hover a. height: 3px; } - /* Navigation Buttons */ - .section_buttons:not(:empty) { margin-top: calc(var(--spacing-large) * 3); } @@ -2649,13 +2621,11 @@ h2:hover a.anchorlink, h1:hover a.anchorlink, h3:hover a.anchorlink, h4:hover a. padding-right: var(--spacing-large); } - .section_buttons tr td.markdownTableBodyRight a::after { content: '〉'; padding-left: var(--spacing-large); } - .section_buttons tr td.markdownTableBodyLeft a:hover::before { color: var(--page-foreground-color) !important; transform: translateX(-3px); From b9121914a218abbcbe4262993bbff60db5fe59cd Mon Sep 17 00:00:00 2001 From: Jesper Juhl Date: Mon, 2 Feb 2026 16:57:06 +0100 Subject: [PATCH 2/3] Don't use NULL, use nullptr --- gui-builder/src/main.cpp | 2 +- include/TGUI/Backend/Renderer/SDL_GPU/CanvasSDLGPU.hpp | 2 +- src/Backend/SDL-GPU.cpp | 4 ++-- src/Backend/Window/SDL/BackendSDL.cpp | 2 +- src/DefaultBackendWindow.cpp | 4 ++-- tests/Tests.cpp | 2 +- 6 files changed, 8 insertions(+), 8 deletions(-) diff --git a/gui-builder/src/main.cpp b/gui-builder/src/main.cpp index c495995c6..f2bd8d2fd 100644 --- a/gui-builder/src/main.cpp +++ b/gui-builder/src/main.cpp @@ -61,7 +61,7 @@ int main(int, char* argv[]) int argcW; LPWSTR* argvW = CommandLineToArgvW(commandLineStr, &argcW); - if (argvW == NULL) + if (argvW == nullptr) { std::cerr << "Failed to access command line arguments\n"; return 1; diff --git a/include/TGUI/Backend/Renderer/SDL_GPU/CanvasSDLGPU.hpp b/include/TGUI/Backend/Renderer/SDL_GPU/CanvasSDLGPU.hpp index c6dfab3b5..f840f88e5 100644 --- a/include/TGUI/Backend/Renderer/SDL_GPU/CanvasSDLGPU.hpp +++ b/include/TGUI/Backend/Renderer/SDL_GPU/CanvasSDLGPU.hpp @@ -52,7 +52,7 @@ namespace tgui /// colorTargetInfo.clear_color = clearColor; /// colorTargetInfo.load_op = SDL_GPU_LOADOP_CLEAR; /// colorTargetInfo.store_op = SDL_GPU_STOREOP_STORE; - /// SDL_GPURenderPass* renderPass = SDL_BeginGPURenderPass(cmdBuffer, &colorTargetInfo, 1, NULL); + /// SDL_GPURenderPass* renderPass = SDL_BeginGPURenderPass(cmdBuffer, &colorTargetInfo, 1, nullptr); /// // Draw to the texture of the canvas /// SDL_EndGPURenderPass(renderPass); /// @endcode diff --git a/src/Backend/SDL-GPU.cpp b/src/Backend/SDL-GPU.cpp index 8524e2267..2cb933b17 100644 --- a/src/Backend/SDL-GPU.cpp +++ b/src/Backend/SDL-GPU.cpp @@ -90,7 +90,7 @@ namespace tgui return; SDL_GPUTexture* swapchainTexture; - if (!SDL_WaitAndAcquireGPUSwapchainTexture(cmdBuffer, m_window, &swapchainTexture, NULL, NULL)) + if (!SDL_WaitAndAcquireGPUSwapchainTexture(cmdBuffer, m_window, &swapchainTexture, nullptr, nullptr)) return; if (swapchainTexture) @@ -103,7 +103,7 @@ namespace tgui colorTargetInfo.load_op = SDL_GPU_LOADOP_CLEAR; colorTargetInfo.store_op = SDL_GPU_STOREOP_STORE; - SDL_GPURenderPass* renderPass = SDL_BeginGPURenderPass(cmdBuffer, &colorTargetInfo, 1, NULL); + SDL_GPURenderPass* renderPass = SDL_BeginGPURenderPass(cmdBuffer, &colorTargetInfo, 1, nullptr); draw(renderPass); SDL_EndGPURenderPass(renderPass); } diff --git a/src/Backend/Window/SDL/BackendSDL.cpp b/src/Backend/Window/SDL/BackendSDL.cpp index 3ea6cac29..e941c6127 100644 --- a/src/Backend/Window/SDL/BackendSDL.cpp +++ b/src/Backend/Window/SDL/BackendSDL.cpp @@ -502,7 +502,7 @@ namespace tgui // back to the SDL cursor if we don't force a refresh. if (m_customCursorX11) { - SDL_SetCursor(NULL); + SDL_SetCursor(nullptr); m_customCursorX11 = false; } #else diff --git a/src/DefaultBackendWindow.cpp b/src/DefaultBackendWindow.cpp index 76c9f2395..d4e8963f3 100644 --- a/src/DefaultBackendWindow.cpp +++ b/src/DefaultBackendWindow.cpp @@ -295,7 +295,7 @@ namespace tgui if (!cmdBuffer) return; SDL_GPUTexture* swapchainTexture; - if (!SDL_WaitAndAcquireGPUSwapchainTexture(cmdBuffer, m_window, &swapchainTexture, NULL, NULL)) + if (!SDL_WaitAndAcquireGPUSwapchainTexture(cmdBuffer, m_window, &swapchainTexture, nullptr, nullptr)) return; if (swapchainTexture) { @@ -307,7 +307,7 @@ namespace tgui colorTargetInfo.load_op = SDL_GPU_LOADOP_CLEAR; colorTargetInfo.store_op = SDL_GPU_STOREOP_STORE; - SDL_GPURenderPass* renderPass = SDL_BeginGPURenderPass(cmdBuffer, &colorTargetInfo, 1, NULL); + SDL_GPURenderPass* renderPass = SDL_BeginGPURenderPass(cmdBuffer, &colorTargetInfo, 1, nullptr); m_gui->draw(renderPass); SDL_EndGPURenderPass(renderPass); } diff --git a/tests/Tests.cpp b/tests/Tests.cpp index c67e57540..bad54af4b 100644 --- a/tests/Tests.cpp +++ b/tests/Tests.cpp @@ -355,7 +355,7 @@ void testDraw(tgui::BackendGui& gui, const char* filename, tgui::Vector2u imageS colorTargetInfo.load_op = SDL_GPU_LOADOP_CLEAR; colorTargetInfo.store_op = SDL_GPU_STOREOP_STORE; - SDL_GPURenderPass* renderPass = SDL_BeginGPURenderPass(cmdBuffer, &colorTargetInfo, 1, NULL); + SDL_GPURenderPass* renderPass = SDL_BeginGPURenderPass(cmdBuffer, &colorTargetInfo, 1, nullptr); dynamic_cast(gui).draw(renderPass); SDL_EndGPURenderPass(renderPass); From 8b56451bbfb80cce059b5d6848aa128765656d81 Mon Sep 17 00:00:00 2001 From: Jesper Juhl Date: Mon, 2 Feb 2026 17:10:37 +0100 Subject: [PATCH 3/3] Get rid of unneeded explicit dereference of this --- include/TGUI/Backend/Window/SFML/BackendGuiSFML.hpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/TGUI/Backend/Window/SFML/BackendGuiSFML.hpp b/include/TGUI/Backend/Window/SFML/BackendGuiSFML.hpp index 8ee660822..0663a92a0 100644 --- a/include/TGUI/Backend/Window/SFML/BackendGuiSFML.hpp +++ b/include/TGUI/Backend/Window/SFML/BackendGuiSFML.hpp @@ -177,7 +177,7 @@ namespace tgui // Let the gui handle the event bool eventHandledByGui = false; if (passEventToGui) - eventHandledByGui = this->handleEvent(std::forward(event)); + eventHandledByGui = handleEvent(std::forward(event)); // After the gui has handled the events, we call the handlers that return nothing. // These handlers can have an optional bool parameter that indicates whether the event was processed by the gui.