From 4bb58c18cb677e4af03346952c1d847cb2108340 Mon Sep 17 00:00:00 2001
From: "mohammed.razzaq" <mohammed.razzaq@get10up.com>
Date: Thu, 1 Feb 2024 19:02:02 +0530
Subject: [PATCH 1/3] Move playlist notice outside the controls bar and add
 check for other notices

---
 includes/admin/class-bc-templates.php | 65 ++++++++++++++++-----------
 1 file changed, 40 insertions(+), 25 deletions(-)

diff --git a/includes/admin/class-bc-templates.php b/includes/admin/class-bc-templates.php
index 2210404b..46e3d437 100644
--- a/includes/admin/class-bc-templates.php
+++ b/includes/admin/class-bc-templates.php
@@ -21,11 +21,23 @@ public function __construct() {
 	 * Adds all templates for Backbone application
 	 */
 	public function add_templates() {
-		?>
+		global $pagenow; ?>
 
 		<?php /* Used by views/media-manager.js */ ?>
 		<script type="text/html" id="tmpl-brightcove-media">
-			<div id="brightcove-media-frame-router" class="brightcove media-frame-router"></div>
+			<?php if ( 'admin.php' === $pagenow ) : ?>
+				<div class="brightcove brightcove-notices">
+					<# if( data.mediaType === 'playlists' ) { #>
+						<div class="notice notice-warning">
+							<p>
+								<?php esc_html_e( 'Please note that you can create new playlists only from Brightcove.', 'brightcove' ); ?>
+							</p>
+						</div>
+					<# } #>
+				</div>
+			<?php endif; ?>
+			<div id="brightcove-media-frame-router" class="brightcove media-frame-router">
+			</div>
 			<div class="brightcove-message message hidden"></div>
 			<div id="brightcove-media-frame-content" class="brightcove media-frame-content">
 				<span id="js-media-loading" class="spinner"></span>
@@ -1275,34 +1287,37 @@ class="brightcove-datetime brightcove-end-date"
 							<# }); #>
 					</select>
 					<# }#>
+					<?php if ( in_array( $pagenow, array( 'post.php', 'post-new.php' ), true ) ) : ?>
+						<# if( data.mediaType === 'videoexperience' ) { #>
+						<div class="notice notice-warning">
+							<p>
+								<?php esc_html_e( 'Please note that you can create new Experiences only from Brightcove.', 'brightcove' ); ?>
+							</p>
+						</div>
+						<div class="notice notice-warning">
+							<p>
+								<?php esc_html_e( 'Leave videos unselected for default Experience behavior.', 'brightcove' ); ?>
+							</p>
+						</div>
+						<# } #>
 
-					<# if( data.mediaType === 'videoexperience' ) { #>
-					<div class="notice notice-warning">
-						<p>
-							<?php esc_html_e( 'Please note that you can create new Experiences only from Brightcove.', 'brightcove' ); ?>
-						</p>
-					</div>
-					<div class="notice notice-warning">
-						<p>
-							<?php esc_html_e( 'Leave videos unselected for default Experience behavior.', 'brightcove' ); ?>
-						</p>
-					</div>
-					<# } #>
-
-					<# if ( data.mediaType === 'playlistexperience' ) { #>
-					<div class="notice notice-warning">
-						<p>
-							<?php esc_html_e( 'Please note that you can create new Experiences only from Brightcove.', 'brightcove' ); ?>
-						</p>
-					</div>
-					<# } #>
-
-					<# if( data.mediaType === 'playlists' || data.mediaType === 'playlistexperience' ) { #>
+						<# if ( data.mediaType === 'playlistexperience' ) { #>
 						<div class="notice notice-warning">
 							<p>
-								<?php esc_html_e( 'Please note that you can create new playlists only from Brightcove.', 'brightcove' ); ?>
+								<?php esc_html_e( 'Please note that you can create new Experiences only from Brightcove.', 'brightcove' ); ?>
 							</p>
 						</div>
+						<# } #>
+
+						<# if( data.mediaType === 'playlists' || data.mediaType === 'playlistexperience' ) { #>
+							<div class="notice notice-warning">
+								<p>
+									<?php esc_html_e( 'Please note that you can create new playlists only from Brightcove.', 'brightcove' ); ?>
+								</p>
+							</div>
+						<# } #>
+					<?php endif; ?>
+					<# if( data.mediaType === 'playlists' || data.mediaType === 'playlistexperience' ) { #>
 						<p>
 							<input type="checkbox" name="brightcove-empty-playlists" id="brightcove-empty-playlists" class="brightcove-empty-playlists attachment-filters">
 							<label for="brightcove-empty-playlists"><?php esc_html_e( 'Hide Empty Playlists', 'brightcove' ); ?></label>

From ccd400653533d9ec2cdddea22c8def9879f4a554 Mon Sep 17 00:00:00 2001
From: "mohammed.razzaq" <mohammed.razzaq@get10up.com>
Date: Thu, 15 Feb 2024 17:46:46 +0530
Subject: [PATCH 2/3] Feedback added

---
 includes/admin/class-bc-templates.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/includes/admin/class-bc-templates.php b/includes/admin/class-bc-templates.php
index 46e3d437..cb3b6ec8 100644
--- a/includes/admin/class-bc-templates.php
+++ b/includes/admin/class-bc-templates.php
@@ -21,7 +21,8 @@ public function __construct() {
 	 * Adds all templates for Backbone application
 	 */
 	public function add_templates() {
-		global $pagenow; ?>
+		global $pagenow;
+		?>
 
 		<?php /* Used by views/media-manager.js */ ?>
 		<script type="text/html" id="tmpl-brightcove-media">

From f429f15d872dbe2308e941330cf357806ccd5eb1 Mon Sep 17 00:00:00 2001
From: "mohammed.razzaq" <mohammed.razzaq@get10up.com>
Date: Tue, 20 Feb 2024 11:38:39 +0530
Subject: [PATCH 3/3] Remove extra warning for Playlist experience

---
 includes/admin/class-bc-templates.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/includes/admin/class-bc-templates.php b/includes/admin/class-bc-templates.php
index cb3b6ec8..77af0eba 100644
--- a/includes/admin/class-bc-templates.php
+++ b/includes/admin/class-bc-templates.php
@@ -1310,7 +1310,7 @@ class="brightcove-datetime brightcove-end-date"
 						</div>
 						<# } #>
 
-						<# if( data.mediaType === 'playlists' || data.mediaType === 'playlistexperience' ) { #>
+						<# if( data.mediaType === 'playlists' ) { #>
 							<div class="notice notice-warning">
 								<p>
 									<?php esc_html_e( 'Please note that you can create new playlists only from Brightcove.', 'brightcove' ); ?>