From 118af57845f5d3217e3566e158c85d12e3516d0a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pablo=20I=C3=B1igo=20Blasco?= <pablo.inigo@ibrobotics.com>
Date: Fri, 3 Apr 2026 01:29:54 +0200
Subject: [PATCH 1/2] fix(marketplace): correct category filter values to match
 manifest conventions

The category combo used "data_streamer" and "parser" which don't match
the actual manifest values "data_stream" and "message_parser". Also
remove unused "Bundle" category. Only "All" and "Data Loader" worked.
---
 pj_marketplace/src/ui/marketplace_window.cpp | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/pj_marketplace/src/ui/marketplace_window.cpp b/pj_marketplace/src/ui/marketplace_window.cpp
index ded8e092..9839a0d6 100644
--- a/pj_marketplace/src/ui/marketplace_window.cpp
+++ b/pj_marketplace/src/ui/marketplace_window.cpp
@@ -71,12 +71,11 @@ void MarketplaceWindow::setupUi() {
   ui_->update_all_btn_->setFixedWidth(90);
   ui_->update_all_btn_->setEnabled(false);
 
-  ui_->category_combo_->addItem("All categories", "");
-  ui_->category_combo_->addItem("Data Loader",    "data_loader");
-  ui_->category_combo_->addItem("Data Streamer",  "data_streamer");
-  ui_->category_combo_->addItem("Parser",         "parser");
-  ui_->category_combo_->addItem("Toolbox",        "toolbox");
-  ui_->category_combo_->addItem("Bundle",         "bundle");
+  ui_->category_combo_->addItem("All categories",  "");
+  ui_->category_combo_->addItem("Data Loader",     "data_loader");
+  ui_->category_combo_->addItem("Data Streamer",   "data_stream");
+  ui_->category_combo_->addItem("Message Parser",  "message_parser");
+  ui_->category_combo_->addItem("Toolbox",         "toolbox");
 
   connect(ui_->search_edit_, &QLineEdit::textChanged,
           this, &MarketplaceWindow::onSearchChanged);

From 9974ec0a193da487103641507d19a774bfd864b7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pablo=20I=C3=B1igo=20Blasco?= <pablo.inigo@ibrobotics.com>
Date: Fri, 3 Apr 2026 17:40:17 +0200
Subject: [PATCH 2/2] fix(marketplace): remove empty changelog section and
 rename GitHub button

Remove changelog header and text browser from extension detail dialog
since no extensions currently provide changelog data. Rename "View on
GitHub" button to "Visit Website" for platform-agnostic wording.
---
 .../src/ui/extension_detail_dialog.cpp        | 10 ------
 .../src/ui/extension_detail_dialog.ui         | 35 +------------------
 2 files changed, 1 insertion(+), 44 deletions(-)

diff --git a/pj_marketplace/src/ui/extension_detail_dialog.cpp b/pj_marketplace/src/ui/extension_detail_dialog.cpp
index caf37e70..baf59cbf 100644
--- a/pj_marketplace/src/ui/extension_detail_dialog.cpp
+++ b/pj_marketplace/src/ui/extension_detail_dialog.cpp
@@ -48,16 +48,6 @@ ExtensionDetailDialog::ExtensionDetailDialog(const Extension& ext, const QString
   // ── Description ────────────────────────────────────────────────────────────
   ui_->desc_lbl->setText(ext.description);
 
-  // ── Changelog ──────────────────────────────────────────────────────────────
-  QString changelog_html;
-  const auto keys = ext.changelog.keys();
-  for (int i = static_cast<int>(keys.size()) - 1; i >= 0; --i) {
-    const QString& ver = keys[i];
-    changelog_html += "<b>v" + ver + "</b><br/>";
-    changelog_html += ext.changelog[ver] + "<br/><br/>";
-  }
-  ui_->changelog_browser->setHtml(changelog_html);
-
   // ── Buttons ── state-dependent visibility and style ────────────────────────
   const bool installed  = !installed_version.isEmpty();
   const bool has_update = installed && installed_version != ext.version;
diff --git a/pj_marketplace/src/ui/extension_detail_dialog.ui b/pj_marketplace/src/ui/extension_detail_dialog.ui
index faca92bf..e9000906 100644
--- a/pj_marketplace/src/ui/extension_detail_dialog.ui
+++ b/pj_marketplace/src/ui/extension_detail_dialog.ui
@@ -76,39 +76,6 @@
     </widget>
    </item>
 
-   <!-- Separator -->
-   <item>
-    <widget class="QFrame" name="separator2">
-     <property name="frameShape"><enum>QFrame::HLine</enum></property>
-     <property name="frameShadow"><enum>QFrame::Sunken</enum></property>
-    </widget>
-   </item>
-
-   <!-- Changelog section -->
-   <item>
-    <widget class="QLabel" name="changelog_header">
-     <property name="text"><string>Changelog</string></property>
-     <property name="styleSheet">
-      <string notr="true">font-weight: bold; color: palette(mid);</string>
-     </property>
-    </widget>
-   </item>
-   <item>
-    <widget class="QTextBrowser" name="changelog_browser">
-     <property name="maximumSize">
-      <size><width>16777215</width><height>120</height></size>
-     </property>
-     <property name="frameShape"><enum>QFrame::NoFrame</enum></property>
-     <property name="openLinks"><bool>false</bool></property>
-     <property name="styleSheet">
-      <string notr="true">QTextBrowser { background: palette(alternate-base);
-  border: 1px solid palette(mid);
-  border-radius: 4px;
-  padding: 6px; }</string>
-     </property>
-    </widget>
-   </item>
-
    <!-- Vertical stretch -->
    <item>
     <spacer name="main_stretch">
@@ -122,7 +89,7 @@
     <layout class="QHBoxLayout" name="btn_layout">
      <item>
       <widget class="QPushButton" name="github_btn">
-       <property name="text"><string>View on GitHub</string></property>
+       <property name="text"><string>Visit Website</string></property>
        <property name="styleSheet">
         <string notr="true">QPushButton { border: 1px solid palette(mid); border-radius: 4px; padding: 4px 14px; }
 QPushButton:hover { background: palette(mid); }