Skip to content

Commit 8dd6830

Browse files
committed
Polish section header breadcrumb styling
Center the slash separator in section header breadcrumbs and remove link underlines in the shared stylesheets. Add a regression test to keep the header breadcrumb spacing consistent across both published CSS bundles.
1 parent 0198b12 commit 8dd6830

3 files changed

Lines changed: 73 additions & 2 deletions

File tree

public/assets/style.css

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2928,6 +2928,20 @@ pre {
29282928
.section .section-header .section-header-breadcrumb .breadcrumb-item {
29292929
font-size: 12px;
29302930
}
2931+
.section .section-header .section-header-breadcrumb .breadcrumb-item + .breadcrumb-item {
2932+
padding-left: 0;
2933+
}
2934+
.section .section-header .section-header-breadcrumb .breadcrumb-item + .breadcrumb-item::before {
2935+
content: "/";
2936+
float: none;
2937+
display: inline-block;
2938+
padding: 0 6px;
2939+
}
2940+
.section .section-header .section-header-breadcrumb .breadcrumb-item a,
2941+
.section .section-header .section-header-breadcrumb .breadcrumb-item a:hover,
2942+
.section .section-header .section-header-breadcrumb .breadcrumb-item a:focus {
2943+
text-decoration: none;
2944+
}
29312945
.section .section-header .btn {
29322946
font-size: 12px;
29332947
}
@@ -3650,4 +3664,4 @@ body.layout-3 .main-wrapper {
36503664
-webkit-transform: scale(1.2, 1.2);
36513665
opacity: 0;
36523666
}
3653-
}
3667+
}

public/css/style.css

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2928,6 +2928,20 @@ pre {
29282928
.section .section-header .section-header-breadcrumb .breadcrumb-item {
29292929
font-size: 12px;
29302930
}
2931+
.section .section-header .section-header-breadcrumb .breadcrumb-item + .breadcrumb-item {
2932+
padding-left: 0;
2933+
}
2934+
.section .section-header .section-header-breadcrumb .breadcrumb-item + .breadcrumb-item::before {
2935+
content: "/";
2936+
float: none;
2937+
display: inline-block;
2938+
padding: 0 6px;
2939+
}
2940+
.section .section-header .section-header-breadcrumb .breadcrumb-item a,
2941+
.section .section-header .section-header-breadcrumb .breadcrumb-item a:hover,
2942+
.section .section-header .section-header-breadcrumb .breadcrumb-item a:focus {
2943+
text-decoration: none;
2944+
}
29312945
.section .section-header .btn {
29322946
font-size: 12px;
29332947
}
@@ -3650,4 +3664,4 @@ body.layout-3 .main-wrapper {
36503664
-webkit-transform: scale(1.2, 1.2);
36513665
opacity: 0;
36523666
}
3653-
}
3667+
}
Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,43 @@
1+
<?php
2+
3+
namespace Tests\Feature\Styling;
4+
5+
use Tests\TestCase;
6+
7+
class BreadcrumbHeaderStyleTest extends TestCase
8+
{
9+
public function test_section_header_breadcrumb_styles_render_a_centered_slash_separator(): void
10+
{
11+
$primaryCss = file_get_contents(public_path('css/style.css'));
12+
$mirroredCss = file_get_contents(public_path('assets/style.css'));
13+
14+
$linkSelector = '.section .section-header .section-header-breadcrumb .breadcrumb-item a';
15+
$adjacentItemSelector = '.section .section-header .section-header-breadcrumb .breadcrumb-item + .breadcrumb-item';
16+
$separatorSelector = '.section .section-header .section-header-breadcrumb .breadcrumb-item + .breadcrumb-item::before';
17+
18+
$this->assertIsString($primaryCss);
19+
$this->assertIsString($mirroredCss);
20+
21+
$this->assertStringContainsString($linkSelector, $primaryCss);
22+
$this->assertStringContainsString($adjacentItemSelector, $primaryCss);
23+
$this->assertStringContainsString($separatorSelector, $primaryCss);
24+
$this->assertStringContainsString('padding-left: 0;', $primaryCss);
25+
$this->assertStringContainsString('content: "/";', $primaryCss);
26+
$this->assertStringContainsString('display: inline-block;', $primaryCss);
27+
$this->assertStringContainsString('padding: 0 6px;', $primaryCss);
28+
$this->assertStringContainsString('text-decoration: none;', $primaryCss);
29+
$this->assertStringNotContainsString('margin-left: 8px;', $primaryCss);
30+
$this->assertStringNotContainsString('content: " / ";', $primaryCss);
31+
32+
$this->assertStringContainsString($linkSelector, $mirroredCss);
33+
$this->assertStringContainsString($adjacentItemSelector, $mirroredCss);
34+
$this->assertStringContainsString($separatorSelector, $mirroredCss);
35+
$this->assertStringContainsString('padding-left: 0;', $mirroredCss);
36+
$this->assertStringContainsString('content: "/";', $mirroredCss);
37+
$this->assertStringContainsString('display: inline-block;', $mirroredCss);
38+
$this->assertStringContainsString('padding: 0 6px;', $mirroredCss);
39+
$this->assertStringContainsString('text-decoration: none;', $mirroredCss);
40+
$this->assertStringNotContainsString('margin-left: 8px;', $mirroredCss);
41+
$this->assertStringNotContainsString('content: " / ";', $mirroredCss);
42+
}
43+
}

0 commit comments

Comments
 (0)