Skip to content

Commit 039bf35

Browse files
authored
Merge pull request #433 from janfaracik/app-bar
Update 'App bar' guidance
2 parents f49a1bd + 4198f9c commit 039bf35

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

src/main/resources/io/jenkins/plugins/designlibrary/AppBar/index.jelly

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,9 @@
2828
<td>${%topAppBar.do.2}</td>
2929
<td>${%topAppBar.dont.2}</td>
3030
</tr>
31+
<tr>
32+
<td>${%topAppBar.do.3}</td>
33+
</tr>
3134
</s:dos-donts>
3235
</s:section>
3336

src/main/resources/io/jenkins/plugins/designlibrary/AppBar/index.properties

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
topAppBar=Top app bar
22
topAppBar.description=Top app bar provides the page heading as well as important actions.
3-
topAppBar.do.1=Place your most important actions in the top app bar.
4-
topAppBar.do.2=Show a count subtitle if it explicitly relates to your page.
3+
topAppBar.do.1=Use an app bar instead of a <code>h1</code> element as app bars have unique properties.
4+
topAppBar.do.2=Place your most important actions in the app bar.
5+
topAppBar.do.3=Show a count subtitle if it explicitly relates to your page.
56
topAppBar.dont.1=Avoid showing more than four actions at once, relegate less used actions to an overflow menu.
67
topAppBar.dont.2=Avoid showing a count subtitle if it could relate to multiple things on your page.
78

0 commit comments

Comments
 (0)