Skip to content

Commit f224a16

Browse files
authored
Merge pull request #420 from lewisbirks/use-app-bar
Use app bar for page title
2 parents 58c4929 + d7f10d0 commit f224a16

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

src/main/resources/bootstrap5/page.jelly

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
Owner of the page.
88
</st:attribute>
99
<st:attribute name="notitle" use="optional">
10-
Hide the title on top of the page.
10+
Hide the title on top of the page. This option can also be used if you wish to add a more complex app bar.
1111
</st:attribute>
1212
<st:attribute name="class" use="optional">
1313
Additional classes to be applied to the page body.
@@ -25,14 +25,12 @@
2525

2626
<st:adjunct includes="io.jenkins.plugins.bootstrap5"/>
2727

28-
<div class="container-fluid ${class}">
29-
30-
<j:if test="${notitle == null}">
31-
<h1>${title}</h1>
32-
</j:if>
28+
<j:if test="${notitle == null}">
29+
<l:app-bar title="${title}"/>
30+
</j:if>
3331

32+
<div class="${class}">
3433
<d:invokeBody/>
35-
3634
</div>
3735
</l:main-panel>
3836
</l:layout>

0 commit comments

Comments
 (0)