Qt: Post processing UI

src/duckstation-qt/CMakeLists.txt

@@ -54,6 +54,11 @@ set(SRCS
   mainwindow.ui
   memorycardsettingswidget.cpp
   memorycardsettingswidget.h
+  postprocessingchainconfigwidget.cpp
+  postprocessingchainconfigwidget.h
+  postprocessingchainconfigwidget.ui
+  postprocessingshaderconfigwidget.cpp
+  postprocessingshaderconfigwidget.h
   qtdisplaywidget.cpp
   qtdisplaywidget.h
   qthostinterface.cpp

src/duckstation-qt/displaysettingswidget.cpp

@@ -1,9 +1,11 @@
 #include "displaysettingswidget.h"
 #include "core/gpu.h"
 #include "core/settings.h"
+#include "postprocessingchainconfigwidget.h"
 #include "qtutils.h"
 #include "settingsdialog.h"
 #include "settingwidgetbinder.h"
+#include <QtWidgets/QMessageBox>
 
 // For enumerating adapters.
 #include "frontend-common/vulkan_host_display.h"
@@ -45,6 +47,25 @@ DisplaySettingsWidget::DisplaySettingsWidget(QtHostInterface* host_interface, QW
           &DisplaySettingsWidget::onGPUAdapterIndexChanged);
   populateGPUAdapters();
 
+  {
+    std::string post_chain = g_host_interface->GetStringSettingValue("Display", "PostProcessChain");
+    if (!post_chain.empty() && !m_ui.postChain->setConfigString(post_chain))
+    {
+      QMessageBox::critical(this, tr("Error"),
+                            tr("The current post-processing chain is invalid, it has been reset. Any changes made will "
+                               "overwrite the existing config."));
+    }
+  }
+  connect(m_ui.postChain, &PostProcessingChainConfigWidget::chainConfigStringChanged,
+          [this](const std::string& new_config) {
+            if (new_config.empty())
+              m_host_interface->RemoveSettingValue("Display", "PostProcessChain");
+            else
+              m_host_interface->SetStringSettingValue("Display", "PostProcessChain", new_config.c_str());
+
+            m_host_interface->applySettings();
+          });
+
   dialog->registerWidgetHelp(
     m_ui.renderer, tr("Renderer"), Settings::GetRendererDisplayName(Settings::DEFAULT_GPU_RENDERER),
     tr("Chooses the backend to use for rendering the console/game visuals. <br>Depending on your system and hardware, "

src/duckstation-qt/displaysettingswidget.h

@@ -5,6 +5,7 @@
 #include "ui_displaysettingswidget.h"
 
 class QtHostInterface;
+class PostProcessingChainConfigWidget;
 class SettingsDialog;
 
 class DisplaySettingsWidget : public QWidget

src/duckstation-qt/displaysettingswidget.ui

@@ -52,6 +52,13 @@
       <item row="1" column="1">
        <widget class="QComboBox" name="adapter"/>
       </item>
+      <item row="2" column="0" colspan="2">
+       <widget class="QCheckBox" name="vsync">
+        <property name="text">
+         <string>VSync</string>
+        </property>
+       </widget>
+      </item>
      </layout>
     </widget>
    </item>
@@ -60,7 +67,7 @@
      <property name="title">
       <string>Screen Display</string>
      </property>
-      <layout class="QFormLayout" name="formLayout_4">
+     <layout class="QFormLayout" name="formLayout">
       <item row="0" column="0">
        <widget class="QLabel" name="label_4">
         <property name="text">
@@ -82,25 +89,22 @@
        <widget class="QComboBox" name="displayCropMode"/>
       </item>
       <item row="2" column="0" colspan="2">
+       <layout class="QGridLayout" name="gridLayout_2">
+        <item row="0" column="0">
          <widget class="QCheckBox" name="displayLinearFiltering">
           <property name="text">
            <string>Linear Upscaling</string>
           </property>
          </widget>
         </item>
-      <item row="3" column="0" colspan="2">
+        <item row="0" column="1">
          <widget class="QCheckBox" name="displayIntegerScaling">
           <property name="text">
            <string>Integer Upscaling</string>
           </property>
          </widget>
         </item>
-      <item row="4" column="0" colspan="2">
+       </layout>
-        <widget class="QCheckBox" name="vsync">
-        <property name="text">
-          <string>VSync</string>
-        </property>
-        </widget>
       </item>
      </layout>
     </widget>
@@ -149,6 +153,30 @@
      </layout>
     </widget>
    </item>
+   <item>
+    <widget class="QGroupBox" name="groupBox_2">
+     <property name="title">
+      <string>Post-Processing Chain</string>
+     </property>
+     <layout class="QGridLayout" name="gridLayout">
+      <property name="leftMargin">
+       <number>0</number>
+      </property>
+      <property name="topMargin">
+       <number>0</number>
+      </property>
+      <property name="rightMargin">
+       <number>0</number>
+      </property>
+      <property name="bottomMargin">
+       <number>0</number>
+      </property>
+      <item row="0" column="0">
+       <widget class="PostProcessingChainConfigWidget" name="postChain" native="true"/>
+      </item>
+     </layout>
+    </widget>
+   </item>
    <item>
     <spacer name="verticalSpacer">
      <property name="orientation">
@@ -164,6 +192,14 @@
    </item>
   </layout>
  </widget>
+ <customwidgets>
+  <customwidget>
+   <class>PostProcessingChainConfigWidget</class>
+   <extends>QWidget</extends>
+   <header>postprocessingchainconfigwidget.h</header>
+   <container>1</container>
+  </customwidget>
+ </customwidgets>
  <resources/>
  <connections/>
 </ui>

src/duckstation-qt/duckstation-qt.vcxproj

@@ -56,6 +56,8 @@
     <ClCompile Include="mainwindow.cpp" />
     <ClCompile Include="controllersettingswidget.cpp" />
     <ClCompile Include="memorycardsettingswidget.cpp" />
+    <ClCompile Include="postprocessingchainconfigwidget.cpp" />
+    <ClCompile Include="postprocessingshaderconfigwidget.cpp" />
     <ClCompile Include="qthostinterface.cpp" />
     <ClCompile Include="qtprogresscallback.cpp" />
     <ClCompile Include="qtutils.cpp" />
@@ -84,6 +86,8 @@
     <QtMoc Include="gamelistsettingswidget.h" />
     <QtMoc Include="gamelistwidget.h" />
     <QtMoc Include="gamepropertiesdialog.h" />
+    <QtMoc Include="postprocessingchainconfigwidget.h" />
+    <QtMoc Include="postprocessingshaderconfigwidget.h" />
     <QtMoc Include="mainwindow.h" />
     <QtMoc Include="qthostinterface.h" />
     <ClInclude Include="qtutils.h" />
@@ -143,6 +147,9 @@
     <QtUi Include="gamepropertiesdialog.ui">
       <FileType>Document</FileType>
     </QtUi>
+    <QtUi Include="postprocessingchainconfigwidget.ui">
+      <FileType>Document</FileType>
+    </QtUi>
   </ItemGroup>
   <ItemGroup>
     <QtResource Include="resources\resources.qrc">
@@ -169,6 +176,8 @@
     <ClCompile Include="$(IntDir)moc_inputbindingwidgets.cpp" />
     <ClCompile Include="$(IntDir)moc_mainwindow.cpp" />
     <ClCompile Include="$(IntDir)moc_memorycardsettingswidget.cpp" />
+    <ClCompile Include="$(IntDir)moc_postprocessingchainconfigwidget.cpp" />
+    <ClCompile Include="$(IntDir)moc_postprocessingshaderconfigwidget.cpp" />
     <ClCompile Include="$(IntDir)moc_qtdisplaywidget.cpp" />
     <ClCompile Include="$(IntDir)moc_qthostinterface.cpp" />
     <ClCompile Include="$(IntDir)moc_qtprogresscallback.cpp" />

src/duckstation-qt/postprocessingchainconfigwidget.cpp

@@ -0,0 +1,174 @@
+#include "postprocessingchainconfigwidget.h"
+#include "frontend-common/postprocessing_chain.h"
+#include <QtGui/QCursor>
+#include <QtWidgets/QMenu>
+#include <QtWidgets/QMessageBox>
+
+PostProcessingChainConfigWidget::PostProcessingChainConfigWidget(QWidget* parent) : QWidget(parent)
+{
+  m_ui.setupUi(this);
+  connectUi();
+  updateButtonStates();
+}
+
+PostProcessingChainConfigWidget::~PostProcessingChainConfigWidget() = default;
+
+void PostProcessingChainConfigWidget::connectUi()
+{
+  connect(m_ui.add, &QPushButton::clicked, this, &PostProcessingChainConfigWidget::onAddButtonClicked);
+  connect(m_ui.remove, &QPushButton::clicked, this, &PostProcessingChainConfigWidget::onRemoveButtonClicked);
+  connect(m_ui.clear, &QPushButton::clicked, this, &PostProcessingChainConfigWidget::onClearButtonClicked);
+  connect(m_ui.moveUp, &QPushButton::clicked, this, &PostProcessingChainConfigWidget::onMoveUpButtonClicked);
+  connect(m_ui.moveDown, &QPushButton::clicked, this, &PostProcessingChainConfigWidget::onMoveDownButtonClicked);
+  connect(m_ui.shaderSettings, &QPushButton::clicked, this,
+          &PostProcessingChainConfigWidget::onShaderConfigButtonClicked);
+  connect(m_ui.shaders, &QListWidget::itemSelectionChanged, this, &PostProcessingChainConfigWidget::updateButtonStates);
+
+  m_ui.loadPreset->setEnabled(false);
+  m_ui.savePreset->setEnabled(false);
+}
+
+bool PostProcessingChainConfigWidget::setConfigString(const std::string_view& config_string)
+{
+  if (!m_chain.CreateFromString(config_string))
+    return false;
+
+  updateList();
+  return true;
+}
+
+std::optional<u32> PostProcessingChainConfigWidget::getSelectedIndex() const
+{
+  QList<QListWidgetItem*> selected_items = m_ui.shaders->selectedItems();
+  return selected_items.empty() ? std::nullopt :
+                                  std::optional<u32>(selected_items.first()->data(Qt::UserRole).toUInt());
+}
+
+void PostProcessingChainConfigWidget::updateList()
+{
+  m_ui.shaders->clear();
+
+  for (u32 i = 0; i < m_chain.GetStageCount(); i++)
+  {
+    const FrontendCommon::PostProcessingShader& shader = m_chain.GetShaderStage(i);
+
+    QListWidgetItem* item = new QListWidgetItem(QString::fromStdString(shader.GetName()), m_ui.shaders);
+    item->setData(Qt::UserRole, QVariant(i));
+  }
+
+  updateButtonStates();
+}
+
+void PostProcessingChainConfigWidget::configChanged()
+{
+  if (m_chain.IsEmpty())
+    chainConfigStringChanged(std::string());
+  else
+    chainConfigStringChanged(m_chain.GetConfigString());
+}
+
+void PostProcessingChainConfigWidget::updateButtonStates()
+{
+  std::optional<u32> index = getSelectedIndex();
+  m_ui.remove->setEnabled(index.has_value());
+  m_ui.clear->setEnabled(!m_chain.IsEmpty());
+  m_ui.shaderSettings->setEnabled(index.has_value());
+
+  if (index.has_value())
+  {
+    m_ui.moveUp->setEnabled(index.value() > 0);
+    m_ui.moveDown->setEnabled(index.value() < (m_chain.GetStageCount() - 1u));
+  }
+  else
+  {
+    m_ui.moveUp->setEnabled(false);
+    m_ui.moveDown->setEnabled(false);
+  }
+}
+
+void PostProcessingChainConfigWidget::onAddButtonClicked()
+{
+  QMenu menu;
+
+  const std::vector<std::string> shaders(FrontendCommon::PostProcessingChain::GetAvailableShaderNames());
+  if (shaders.empty())
+  {
+    menu.addAction(tr("No Shaders Available"))->setEnabled(false);
+  }
+  else
+  {
+    for (const std::string& shader : shaders)
+    {
+      QAction* action = menu.addAction(QString::fromStdString(shader));
+      connect(action, &QAction::triggered, [this, &shader]() {
+        if (!m_chain.AddStage(shader))
+        {
+          QMessageBox::critical(this, tr("Error"), tr("Failed to add shader. The log may contain more information."));
+          return;
+        }
+        
+        updateList();
+        configChanged();
+      });
+    }
+  }
+
+  menu.exec(QCursor::pos());
+}
+
+void PostProcessingChainConfigWidget::onRemoveButtonClicked()
+{
+  QList<QListWidgetItem*> selected_items = m_ui.shaders->selectedItems();
+  if (selected_items.empty())
+    return;
+
+  QListWidgetItem* item = selected_items.first();
+  u32 index = item->data(Qt::UserRole).toUInt();
+  if (index < m_chain.GetStageCount())
+  {
+    m_chain.RemoveStage(index);
+    updateList();
+    configChanged();
+  }
+}
+
+void PostProcessingChainConfigWidget::onClearButtonClicked()
+{
+  if (QMessageBox::question(this, tr("Question"), tr("Are you sure you want to clear all shader stages?"),
+                            QMessageBox::Yes, QMessageBox::No) == QMessageBox::Yes)
+  {
+    m_chain.ClearStages();
+    updateList();
+    configChanged();
+  }
+}
+
+void PostProcessingChainConfigWidget::onMoveUpButtonClicked()
+{
+  std::optional<u32> index = getSelectedIndex();
+  if (index.has_value())
+  {
+    m_chain.MoveStageUp(index.value());
+    updateList();
+    configChanged();
+  }
+}
+
+void PostProcessingChainConfigWidget::onMoveDownButtonClicked()
+{
+  std::optional<u32> index = getSelectedIndex();
+  if (index.has_value())
+  {
+    m_chain.MoveStageDown(index.value());
+    updateList();
+    configChanged();
+  }
+}
+
+void PostProcessingChainConfigWidget::onShaderConfigButtonClicked()
+{
+  std::optional<u32> index = getSelectedIndex();
+  if (index.has_value())
+  {
+  }
+}

src/duckstation-qt/postprocessingchainconfigwidget.h

@@ -0,0 +1,45 @@
+#pragma once
+#include "common/types.h"
+#include "ui_postprocessingchainconfigwidget.h"
+#include "frontend-common/postprocessing_chain.h"
+#include <QtWidgets/QWidget>
+#include <optional>
+#include <memory>
+#include <string_view>
+
+namespace FrontendCommon {
+class PostProcessingChain;
+}
+
+class PostProcessingChainConfigWidget : public QWidget
+{
+  Q_OBJECT
+
+public:
+  PostProcessingChainConfigWidget(QWidget* parent);
+  ~PostProcessingChainConfigWidget();
+
+  bool setConfigString(const std::string_view& config_string);
+
+Q_SIGNALS:
+  void chainConfigStringChanged(const std::string& new_config_string);
+
+private Q_SLOTS:
+  void onAddButtonClicked();
+  void onRemoveButtonClicked();
+  void onClearButtonClicked();
+  void onMoveUpButtonClicked();
+  void onMoveDownButtonClicked();
+  void onShaderConfigButtonClicked();
+  void updateButtonStates();
+
+private:
+  void connectUi();
+  std::optional<u32> getSelectedIndex() const;
+  void updateList();
+  void configChanged();
+
+  Ui::PostProcessingChainConfigWidget m_ui;
+
+  FrontendCommon::PostProcessingChain m_chain;
+};

src/duckstation-qt/postprocessingchainconfigwidget.ui

@@ -0,0 +1,130 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<ui version="4.0">
+ <class>PostProcessingChainConfigWidget</class>
+ <widget class="QWidget" name="PostProcessingChainConfigWidget">
+  <property name="geometry">
+   <rect>
+    <x>0</x>
+    <y>0</y>
+    <width>497</width>
+    <height>151</height>
+   </rect>
+  </property>
+  <property name="windowTitle">
+   <string>Form</string>
+  </property>
+  <layout class="QGridLayout" name="gridLayout">
+   <item row="1" column="0">
+    <layout class="QHBoxLayout" name="horizontalLayout">
+     <item>
+      <widget class="QToolButton" name="add">
+       <property name="text">
+        <string>Add</string>
+       </property>
+       <property name="icon">
+        <iconset resource="resources/resources.qrc">
+         <normaloff>:/icons/list-add.png</normaloff>:/icons/list-add.png</iconset>
+       </property>
+      </widget>
+     </item>
+     <item>
+      <widget class="QToolButton" name="remove">
+       <property name="text">
+        <string>Remove</string>
+       </property>
+       <property name="icon">
+        <iconset resource="resources/resources.qrc">
+         <normaloff>:/icons/list-remove.png</normaloff>:/icons/list-remove.png</iconset>
+       </property>
+      </widget>
+     </item>
+     <item>
+      <widget class="QToolButton" name="clear">
+       <property name="text">
+        <string>Clear</string>
+       </property>
+       <property name="icon">
+        <iconset resource="resources/resources.qrc">
+         <normaloff>:/icons/edit-clear-16.png</normaloff>:/icons/edit-clear-16.png</iconset>
+       </property>
+      </widget>
+     </item>
+     <item>
+      <widget class="QToolButton" name="moveUp">
+       <property name="text">
+        <string>Move Up</string>
+       </property>
+       <property name="icon">
+        <iconset resource="resources/resources.qrc">
+         <normaloff>:/icons/go-up-16.png</normaloff>:/icons/go-up-16.png</iconset>
+       </property>
+      </widget>
+     </item>
+     <item>
+      <widget class="QToolButton" name="moveDown">
+       <property name="text">
+        <string>Move Down</string>
+       </property>
+       <property name="icon">
+        <iconset resource="resources/resources.qrc">
+         <normaloff>:/icons/go-down-16.png</normaloff>:/icons/go-down-16.png</iconset>
+       </property>
+      </widget>
+     </item>
+     <item>
+      <widget class="QToolButton" name="shaderSettings">
+       <property name="text">
+        <string>Shader Settings...</string>
+       </property>
+       <property name="icon">
+        <iconset resource="resources/resources.qrc">
+         <normaloff>:/icons/preferences-system@2x.png</normaloff>:/icons/preferences-system@2x.png</iconset>
+       </property>
+      </widget>
+     </item>
+     <item>
+      <spacer name="horizontalSpacer">
+       <property name="orientation">
+        <enum>Qt::Horizontal</enum>
+       </property>
+       <property name="sizeHint" stdset="0">
+        <size>
+         <width>40</width>
+         <height>20</height>
+        </size>
+       </property>
+      </spacer>
+     </item>
+     <item>
+      <widget class="QPushButton" name="loadPreset">
+       <property name="text">
+        <string>Load Preset</string>
+       </property>
+      </widget>
+     </item>
+     <item>
+      <widget class="QPushButton" name="savePreset">
+       <property name="text">
+        <string>Save Preset</string>
+       </property>
+      </widget>
+     </item>
+    </layout>
+   </item>
+   <item row="0" column="0">
+    <widget class="QListWidget" name="shaders">
+     <property name="minimumSize">
+      <size>
+       <width>0</width>
+       <height>50</height>
+      </size>
+     </property>
+    </widget>
+   </item>
+  </layout>
+ </widget>
+ <resources>
+  <include location="resources/resources.qrc"/>
+ </resources>
+ <connections/>
+</ui>

src/duckstation-qt/resources/resources.qrc

@@ -32,6 +32,8 @@
     <file>icons/duck.png</file>
     <file>icons/duck_128.png</file>
     <file>icons/duck_64.png</file>
+    <file>icons/edit-clear-16.png</file>
+    <file>icons/edit-clear-16@2x.png</file>
     <file>icons/edit-find.png</file>
     <file>icons/flag-eu.png</file>
     <file>icons/flag-eu@2x.png</file>
@@ -43,6 +45,10 @@
     <file>icons/flag-us@2x.png</file>
     <file>icons/folder-open.png</file>
     <file>icons/folder-open@2x.png</file>
+    <file>icons/go-down-16.png</file>
+    <file>icons/go-down-16@2x.png</file>
+    <file>icons/go-up-16.png</file>
+    <file>icons/go-up-16@2x.png</file>
     <file>icons/input-gaming.png</file>
     <file>icons/input-gaming@2x.png</file>
     <file>icons/list-add.png</file>