ui: Improved look of the about page

lib/libimhex/source/ui/imgui_imhex_extensions.cpp

@@ -108,6 +108,7 @@ namespace ImGui {
         ImVec2 size = CalcItemSize(size_arg, label_size.x, label_size.y) + ImVec2(g.FontSize + style.FramePadding.x * 2, 0.0f);
 
         const ImRect bb(pos, pos + size);
+        ItemSize(size, 0);
         if (!ItemAdd(bb, id))
             return false;
 
@@ -119,10 +120,9 @@ namespace ImGui {
         // Render
         const ImU32 col = hovered ? GetColorU32(ImGuiCol_ButtonHovered) : GetColorU32(ImGuiCol_ButtonActive);
         PushStyleColor(ImGuiCol_Text, ImU32(col));
-        RenderBullet(window->DrawList, bb.Min + ImVec2(style.FramePadding.x + g.FontSize * 0.5f, g.FontSize * 0.5f), col);
+        RenderBullet(window->DrawList, bb.Min + ImVec2(style.FramePadding.x, g.FontSize * 0.5f), col);
-        RenderText(bb.Min + ImVec2(g.FontSize + style.FramePadding.x * 2, 0.0f), label, nullptr, false);
+        RenderText(bb.Min + ImVec2(g.FontSize * 0.5 + style.FramePadding.x, 0.0f), label, nullptr, false);
-        GetWindowDrawList()->AddLine(bb.Min + ImVec2(style.FramePadding.x, size.y), pos + size, ImU32(col));
+        GetWindowDrawList()->AddLine(bb.Min + ImVec2(g.FontSize * 0.5 + style.FramePadding.x, size.y), pos + size - ImVec2(g.FontSize * 0.5 + style.FramePadding.x, 0), ImU32(col));
-        ImGui::NewLine();
         PopStyleColor();
 
         IMGUI_TEST_ENGINE_ITEM_INFO(id, label, window->DC.LastItemStatusFlags);

plugins/builtin/CMakeLists.txt

@@ -29,7 +29,7 @@ add_library(${PROJECT_NAME} SHARED
         source/content/views/view_pattern_data.cpp
         source/content/views/view_hashes.cpp
         source/content/views/view_information.cpp
-        source/content/views/view_help.cpp
+        source/content/views/view_about.cpp
         source/content/views/view_tools.cpp
         source/content/views/view_strings.cpp
         source/content/views/view_data_inspector.cpp

plugins/builtin/include/content/views/view_about.hpp

@@ -15,10 +15,10 @@ namespace hex::plugin::builtin {
         class Provider;
     }
 
-    class ViewHelp : public View {
+    class ViewAbout : public View {
     public:
-        ViewHelp();
+        ViewAbout();
-        ~ViewHelp() override;
+        ~ViewAbout() override;
 
         void drawContent() override;
 

plugins/builtin/source/content/views.cpp

@@ -3,7 +3,7 @@
 #include "content/views/view_pattern_data.hpp"
 #include "content/views/view_hashes.hpp"
 #include "content/views/view_information.hpp"
-#include "content/views/view_help.hpp"
+#include "content/views/view_about.hpp"
 #include "content/views/view_tools.hpp"
 #include "content/views/view_strings.hpp"
 #include "content/views/view_data_inspector.hpp"
@@ -34,7 +34,7 @@ namespace hex::plugin::builtin {
         ContentRegistry::Views::add<ViewPatches>();
         ContentRegistry::Views::add<ViewTools>();
         ContentRegistry::Views::add<ViewCommandPalette>();
-        ContentRegistry::Views::add<ViewHelp>();
+        ContentRegistry::Views::add<ViewAbout>();
         ContentRegistry::Views::add<ViewSettings>();
         ContentRegistry::Views::add<ViewDataProcessor>();
         ContentRegistry::Views::add<ViewYara>();

plugins/builtin/source/content/views/view_about.cpp

@@ -1,9 +1,8 @@
-#include "content/views/view_help.hpp"
+#include "content/views/view_about.hpp"
 
 #include <hex/api/content_registry.hpp>
 
 #include <hex/helpers/fmt.hpp>
-#include <hex/helpers/logger.hpp>
 #include <hex/helpers/fs.hpp>
 #include <hex/helpers/utils.hpp>
 
@@ -11,7 +10,7 @@
 
 namespace hex::plugin::builtin {
 
-    ViewHelp::ViewHelp() : View("hex.builtin.view.help.about.name") {
+    ViewAbout::ViewAbout() : View("hex.builtin.view.help.about.name") {
 
         ContentRegistry::Interface::addMenuItem("hex.builtin.menu.help", 1000, [&, this] {
             if (ImGui::MenuItem("hex.builtin.view.help.about.name"_lang, "")) {
@@ -26,16 +25,27 @@ namespace hex::plugin::builtin {
         });
     }
 
-    ViewHelp::~ViewHelp() {
+    ViewAbout::~ViewAbout() {
         ImGui::UnloadImage(this->m_logoTexture);
     }
 
-    static void link(const std::string &label, const std::string &url) {
+    static void link(const std::string &name, const std::string &author, const std::string &url) {
-        if (ImGui::BulletHyperlink(label.data()))
+        if (ImGui::BulletHyperlink(name.c_str()))
             hex::openWebpage(url);
+
+        if (ImGui::IsItemHovered()) {
+            ImGui::BeginTooltip();
+            ImGui::TextFormatted("{}", url);
+            ImGui::EndTooltip();
         }
 
-    void ViewHelp::drawAboutMainPage() {
+        if (!author.empty()) {
+            ImGui::SameLine(0, 0);
+            ImGui::TextFormatted("by {}", author);
+        }
+    }
+
+    void ViewAbout::drawAboutMainPage() {
         if (ImGui::BeginTable("about_table", 2, ImGuiTableFlags_SizingFixedFit)) {
             ImGui::TableNextRow();
             ImGui::TableNextColumn();
@@ -83,37 +93,55 @@ namespace hex::plugin::builtin {
         }
     }
 
-    void ViewHelp::drawContributorPage() {
+    void ViewAbout::drawContributorPage() {
-        link("Mary for porting ImHex to MacOS", "https://github.com/Thog");
+        ImGui::TextFormattedWrapped("These amazing people have contributed to ImHex in the past. If you'd like to become part of them, please submit a PR to the GitHub Repository!");
-        link("Roblabla for adding the MSI Windows installer", "https://github.com/roblabla");
+        ImGui::NewLine();
-        link("jam1garner for adding support for Rust plugins", "https://github.com/jam1garner");
+
+        link("Mary for porting ImHex to MacOS", "", "https://github.com/Thog");
+        link("Roblabla for adding the MSI Windows installer", "", "https://github.com/roblabla");
+        link("jam1garner for adding support for Rust plugins", "", "https://github.com/jam1garner");
+
+        ImGui::NewLine();
+
+        link("All other amazing contributors", "", "https://github.com/WerWolv/ImHex/graphs/contributors/");
     }
 
-    void ViewHelp::drawLibraryCreditsPage() {
+    void ViewAbout::drawLibraryCreditsPage() {
         ImGui::PushStyleColor(ImGuiCol_ChildBg, ImVec4(0.2F, 0.2F, 0.2F, 0.3F));
 
-        link("ImGui by ocornut", "https://github.com/ocornut/imgui");
+        link("ImGui", "ocornut", "https://github.com/ocornut/imgui/");
-        link("imgui_club by ocornut", "https://github.com/ocornut/imgui_club");
+        link("imgui_club", "ocornut", "https://github.com/ocornut/imgui_club/");
-        link("imnodes by Nelarius", "https://github.com/Nelarius/imnodes");
+        link("imnodes", "Nelarius", "https://github.com/Nelarius/imnodes/");
-        link("ImGuiColorTextEdit by BalazsJako", "https://github.com/BalazsJako/ImGuiColorTextEdit");
+        link("ImGuiColorTextEdit", "BalazsJako", "https://github.com/BalazsJako/ImGuiColorTextEdit/");
-        link("ImPlot by epezent", "https://github.com/epezent/implot");
+        link("ImPlot", "epezent", "https://github.com/epezent/implot/");
-        link("capstone by aquynh", "https://github.com/aquynh/capstone");
+
-        link("JSON for Modern C++ by nlohmann", "https://github.com/nlohmann/json");
-        link("YARA by VirusTotal", "https://github.com/VirusTotal/yara");
-        link("Native File Dialog Extended by btzy and mlabbe", "https://github.com/btzy/nativefiledialog-extended");
-        link("Native File Dialog Extended by btzy and mlabbe", "https://github.com/btzy/nativefiledialog-extended");
         ImGui::NewLine();
-        link("GNU libmagic", "http://www.darwinsys.com/file/");
+
-        link("GLFW3", "https://github.com/glfw/glfw");
+        link("capstone", "aquynh", "https://github.com/aquynh/capstone/");
-        link("LLVM", "https://github.com/llvm/llvm-project");
+        link("JSON for Modern C++", "nlohmann", "https://github.com/nlohmann/json/");
-        link("Python 3", "https://github.com/python/cpython");
+        link("YARA", "VirusTotal", "https://github.com/VirusTotal/yara/");
-        link("FreeType", "https://gitlab.freedesktop.org/freetype/freetype");
+        link("Native File Dialog Extended", "btzy and mlabbe", "https://github.com/btzy/nativefiledialog-extended/");
-        link("Mbed TLS", "https://github.com/ARMmbed/mbedtls");
+        link("libromfs", "WerWolv", "https://github.com/WerWolv/libromfs/");
+        link("microtar", "rxi", "https://github.com/rxi/microtar/");
+        link("xdgpp", "danyspin97", "https://sr.ht/~danyspin97/xdgpp/");
+        link("FreeType", "David Turner", "https://gitlab.freedesktop.org/freetype/freetype/");
+        link("Mbed TLS", "ARM", "https://github.com/ARMmbed/mbedtls/");
+        link("libcurl", "Daniel Stenberg", "https://curl.se/");
+        link("libfmt", "vitaut", "https://fmt.dev/");
+
+        ImGui::NewLine();
+
+        link("GNU libmagic", "", "https://www.darwinsys.com/file/");
+        link("GLFW3", "", "https://github.com/glfw/glfw/");
+        link("LLVM", "", "https://github.com/llvm/llvm-project/");
+        link("Python 3", "", "https://github.com/python/cpython/");
 
         ImGui::PopStyleColor();
+
+        ImGui::NewLine();
     }
 
-    void ViewHelp::drawPathsPage() {
+    void ViewAbout::drawPathsPage() {
         if (ImGui::BeginTable("##imhex_paths", 2, ImGuiTableFlags_ScrollY | ImGuiTableFlags_Borders | ImGuiTableFlags_RowBg | ImGuiTableFlags_SizingFixedFit)) {
             ImGui::TableSetupScrollFreeze(0, 1);
             ImGui::TableSetupColumn("Type");
@@ -145,11 +173,11 @@ namespace hex::plugin::builtin {
         }
     }
 
-    void ViewHelp::drawLicensePage() {
+    void ViewAbout::drawLicensePage() {
         ImGui::TextFormattedWrapped("{}", romfs::get("LICENSE").string());
     }
 
-    void ViewHelp::drawAboutPopup() {
+    void ViewAbout::drawAboutPopup() {
         ImGui::SetNextWindowSize(scaled(ImVec2(600, 350)), ImGuiCond_Always);
         if (ImGui::BeginPopupModal(View::toWindowName("hex.builtin.view.help.about.name").c_str(), &this->m_aboutWindowOpen, ImGuiWindowFlags_NoResize)) {
 
@@ -209,7 +237,7 @@ namespace hex::plugin::builtin {
         }
     }
 
-    void ViewHelp::drawContent() {
+    void ViewAbout::drawContent() {
         if (!this->m_aboutWindowOpen)
             this->getWindowOpenState() = false;