Make back to top link optional

WordPress · Jan 12, 2024 · 8bc9f39 · 8bc9f39
1 parent 78186e9
commit 8bc9f39
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 7 deletions.
diff --git a/mu-plugins/blocks/sidebar-container/index.php b/mu-plugins/blocks/sidebar-container/index.php
@@ -39,10 +39,12 @@ function init() {
  * @return string Returns the block markup.
  */
 function render( $attributes, $content, $block ) {
-	$back_to_top = sprintf(
-		'<p class="has-small-font-size is-link-to-top"><a href="#wp--skip-link--target">%s</a></p>',
-		esc_html__( '↑ Back to top', 'wporg' )
-	);
+	$back_to_top = $attributes['hasBackToTop']
+		? sprintf(
+			'<p class="has-small-font-size is-link-to-top"><a href="#wp--skip-link--target">%s</a></p>',
+			esc_html__( '↑ Back to top', 'wporg' )
+		)
+		: '';
 
 	$wrapper_attributes = get_block_wrapper_attributes();
 	return sprintf(

diff --git a/mu-plugins/blocks/sidebar-container/postcss/style.pcss b/mu-plugins/blocks/sidebar-container/postcss/style.pcss
@@ -18,7 +18,7 @@
 
 		width: var(--local--block-end-sidebar--width);
 		margin-bottom: 0 !important;
-		padding-top: var(--wp--preset--spacing--20);
+		padding: var(--wp--preset--spacing--20) 0;
 
 		main & {
 			position: absolute;
@@ -67,7 +67,6 @@
 
 		* + .is-link-to-top {
 			border-top: 1px solid var(--wp--preset--color--light-grey-1);
-			padding-bottom: var(--wp--preset--spacing--20);
 		}
 	}
 }

diff --git a/mu-plugins/blocks/sidebar-container/src/block.json b/mu-plugins/blocks/sidebar-container/src/block.json
@@ -7,7 +7,12 @@
 	"category": "layout",
 	"description": "A sticky container to be used in 2-column layouts.",
 	"textdomain": "wporg",
-	"attributes": {},
+	"attributes": {
+		"hasBackToTop": {
+			"type": "boolean",
+			"default": false
+		}
+	},
 	"supports": {
 		"inserter": false,
 		"__experimentalLayout": true,