Skip to content

Commit

Permalink
Hide bottom bulk action buttons on mobile (#4212)
Browse files Browse the repository at this point in the history
  • Loading branch information
ThoWagen authored and demiankatz committed Jan 28, 2025
1 parent defa0d0 commit 5f1ba2d
Show file tree
Hide file tree
Showing 10 changed files with 20 additions and 19 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,14 @@ public function testBulkControls(int $windowWidth, int $windowHeight, array $con

// Test visibility of search result bulk items and checkbox:
$shouldBeVisible = $controlVisibility['bulk'];
$this->assertEquals($shouldBeVisible, $this->findCss($page, '.bulkActionButtons')->isVisible());
$this->assertEquals(
$shouldBeVisible,
$this->findCss($page, '#search-cart-form .bulkActionButtons')->isVisible()
);
$this->assertEquals(
$shouldBeVisible,
$this->findCss($page, '.mainbody > .bulkActionButtons')->isVisible()
);
$this->assertEquals($shouldBeVisible, $this->findCss($page, '.checkbox-select-item')->isVisible());

// Add a favorite:
Expand Down
2 changes: 1 addition & 1 deletion themes/bootprint3/css/compiled.css

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion themes/bootstrap3/css/compiled.css

Large diffs are not rendered by default.

6 changes: 2 additions & 4 deletions themes/bootstrap3/less/components/search.less
Original file line number Diff line number Diff line change
Expand Up @@ -54,10 +54,8 @@ $thumbnail-width-large: 160px !default;
#datevispublishDatexWrapper,
// Bulk action checkboxes:
#search-cart-form .result .checkbox,
// Search result bulk action buttons:
#search-cart-form .bulkActionButtons,
// Favorites bulk action buttons:
form[name="bulkActionForm"] .bulkActionButtons { display: none; }
// Search result and favorites bulk action buttons:
.bulkActionButtons { display: none; }
}

.searchHomeContent { .clearfix(); }
Expand Down
6 changes: 2 additions & 4 deletions themes/bootstrap3/scss/components/search.scss
Original file line number Diff line number Diff line change
Expand Up @@ -54,10 +54,8 @@ $thumbnail-width-large: 160px !default;
#datevispublishDatexWrapper,
// Bulk action checkboxes:
#search-cart-form .result .checkbox,
// Search result bulk action buttons:
#search-cart-form .bulkActionButtons,
// Favorites bulk action buttons:
form[name="bulkActionForm"] .bulkActionButtons { display: none; }
// Search result and favorites bulk action buttons:
.bulkActionButtons { display: none; }
}

.searchHomeContent { @include clearfix(); }
Expand Down
2 changes: 1 addition & 1 deletion themes/bootstrap5/css/compiled.css

Large diffs are not rendered by default.

6 changes: 2 additions & 4 deletions themes/bootstrap5/scss/components/search.scss
Original file line number Diff line number Diff line change
Expand Up @@ -54,10 +54,8 @@ $thumbnail-width-large: 160px !default;
#datevispublishDatexWrapper,
// Bulk action checkboxes:
#search-cart-form .result .checkbox,
// Search result bulk action buttons:
#search-cart-form .bulkActionButtons,
// Favorites bulk action buttons:
form[name="bulkActionForm"] .bulkActionButtons { display: none; }
// Search result and favorites bulk action buttons:
.bulkActionButtons { display: none; }
}

.searchHomeContent { @include clearfix(); }
Expand Down
2 changes: 1 addition & 1 deletion themes/local_theme_example/css/compiled.css

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion themes/sandal/css/compiled.css

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion themes/sandal5/css/compiled.css

Large diffs are not rendered by default.

0 comments on commit 5f1ba2d

Please sign in to comment.