Add support for "None" transition type to scene item visibility action

plugins/base/utils/transition-selection.cpp

@@ -48,6 +48,8 @@ OBSWeakSource TransitionSelection::GetTransition() const
 		obs_source_release(source);
 		return weakSource;
 	}
+	case Type::NONE:
+		return nullptr;
 	default:
 		break;
 	}
@@ -63,6 +65,8 @@ std::string TransitionSelection::ToString() const
 		return obs_module_text("AdvSceneSwitcher.currentTransition");
 	case Type::ANY:
 		return obs_module_text("AdvSceneSwitcher.anyTransition");
+	case Type::NONE:
+		return obs_module_text("AdvSceneSwitcher.noneTransition");
 	default:
 		break;
 	}
@@ -70,14 +74,16 @@ std::string TransitionSelection::ToString() const
 }
 
 TransitionSelectionWidget::TransitionSelectionWidget(QWidget *parent,
-						     bool current, bool any)
+						     bool current, bool any,
+						     bool none)
 	: FilterComboBox(parent,
 			 obs_module_text("AdvSceneSwitcher.selectTransition")),
 	  _addCurrent(current),
-	  _addAny(any)
+	  _addAny(any),
+	  _addNone(none)
 {
 	setDuplicatesEnabled(true);
-	PopulateTransitionSelection(this, current, any, false);
+	PopulateTransitionSelection(this, current, any, false, none);
 
 	QWidget::connect(this, SIGNAL(currentTextChanged(const QString &)),
 			 this, SLOT(SelectionChanged(const QString &)));
@@ -86,8 +92,9 @@ TransitionSelectionWidget::TransitionSelectionWidget(QWidget *parent,
 void TransitionSelectionWidget::SetTransition(const TransitionSelection &t)
 {
 	// Order of entries
-	// 1. Any transition
-	// 2. Current transition
+	// 1. None transition
+	// 2. Any transition
+	// 3. Current transition
 	// 4. Transitions
 
 	switch (t.GetType()) {
@@ -102,6 +109,10 @@ void TransitionSelectionWidget::SetTransition(const TransitionSelection &t)
 		setCurrentIndex(findText(QString::fromStdString(
 			obs_module_text("AdvSceneSwitcher.anyTransition"))));
 		break;
+	case TransitionSelection::Type::NONE:
+		setCurrentIndex(findText(QString::fromStdString(
+			obs_module_text("AdvSceneSwitcher.noneTransition"))));
+		break;
 	default:
 		setCurrentIndex(-1);
 		break;
@@ -120,6 +131,12 @@ void TransitionSelectionWidget::EnableAnyEntry(bool enable)
 	Populate();
 }
 
+void TransitionSelectionWidget::EnableNoneEntry(bool enable)
+{
+	_addNone = enable;
+	Populate();
+}
+
 void TransitionSelectionWidget::showEvent(QShowEvent *event)
 {
 	FilterComboBox::showEvent(event);
@@ -133,7 +150,7 @@ void TransitionSelectionWidget::Populate()
 {
 	const QSignalBlocker blocker(this);
 	clear();
-	PopulateTransitionSelection(this, _addCurrent, _addAny);
+	PopulateTransitionSelection(this, _addCurrent, _addAny, true, _addNone);
 }
 
 static bool isFirstEntry(const QComboBox *l, QString name, int idx)
@@ -163,6 +180,9 @@ TransitionSelection TransitionSelectionWidget::GetCurrentSelection() const
 		if (IsAnyTransitionSelected(text)) {
 			result._type = TransitionSelection::Type::ANY;
 		}
+		if (IsNoneTransitionSelected(text)) {
+			result._type = TransitionSelection::Type::NONE;
+		}
 	}
 	return result;
 }
@@ -187,6 +207,16 @@ bool TransitionSelectionWidget::IsAnyTransitionSelected(
 	return false;
 }
 
+bool TransitionSelectionWidget::IsNoneTransitionSelected(
+	const QString &name) const
+{
+	if (name == QString::fromStdString((obs_module_text(
+			    "AdvSceneSwitcher.noneTransition")))) {
+		return isFirstEntry(this, name, currentIndex());
+	}
+	return false;
+}
+
 void TransitionSelectionWidget::SelectionChanged(const QString &)
 {
 	emit TransitionChanged(GetCurrentSelection());

plugins/base/utils/transition-selection.hpp

@@ -16,6 +16,7 @@ public:
 		TRANSITION,
 		CURRENT,
 		ANY,
+		NONE,
 	};
 
 	Type GetType() const { return _type; }
@@ -33,10 +34,11 @@ class TransitionSelectionWidget : public FilterComboBox {
 
 public:
 	TransitionSelectionWidget(QWidget *parent, bool current = true,
-				  bool any = false);
+				  bool any = false, bool none = false);
 	void SetTransition(const TransitionSelection &);
 	void EnableCurrentEntry(bool enable);
 	void EnableAnyEntry(bool enable);
+	void EnableNoneEntry(bool enable);
 
 protected:
 	void showEvent(QShowEvent *event) override;
@@ -52,9 +54,11 @@ private:
 	TransitionSelection GetCurrentSelection() const;
 	bool IsCurrentTransitionSelected(const QString &name) const;
 	bool IsAnyTransitionSelected(const QString &name) const;
+	bool IsNoneTransitionSelected(const QString &name) const;
 
 	bool _addCurrent;
 	bool _addAny;
+	bool _addNone;
 };
 
 } // namespace advss