Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build(TA-439): update design language guidance regarding character limits #36

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions docs/assets/css/custom-full.css

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion docs/assets/css/custom-full.css.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion docs/assets/css/custom.css

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion docs/assets/css/custom.css.map

Large diffs are not rendered by default.

18 changes: 14 additions & 4 deletions docs/forms/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -352,8 +352,18 @@ <h2>Guidelines</h2>
<li>Labels always visible</li>
<li>A new row for each question</li>
<li>
Use a hint to explain any validation rules which have been applied to
the input
Where there is an external constraint that requires a character limit:
<ul>
<li>
Display how many characters the user has remaining as they type.
</li>
<li>
Display the remaining characters below the text area.
</li>
<li>
Allow the user to enter more characters than the maximum limit. The form validation will handle any excess.
</li>
</ul>
</li>
<li>
Use the appropriate <code>autocomplete</code> attribute for the field in
Expand All @@ -364,8 +374,8 @@ <h2>Guidelines</h2>
>
</li>
<li>
All fields required by default; explicitly mark those which are
optional.
All fields optional by default; explicitly mark those which are
required.
</li>
<li>
Use a <code>fieldset</code> to group multiple controls. A
Expand Down
76 changes: 38 additions & 38 deletions docs/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
/>
<script src="https://kit.fontawesome.com/1f723026cf.js" crossorigin="anonymous"></script>
<script src="https://cdn.jsdelivr.net/npm/[email protected]/dist/js/bootstrap.bundle.min.js" integrity="sha384-gtEjrD/SeCtmISkJkNUaaKMoLD0//ElJ19smozuHV6z3Iehds+3Ulb9Bn9Plx0x4" crossorigin="anonymous"></script>

<title>Bootstrap v5 Kitchen Sink | Talis</title>
</head>
<body>
Expand All @@ -34,34 +34,34 @@
</button>
<div class="collapse navbar-collapse" id="navbarSupportedContent">
<ul class="navbar-nav ms-auto mb-2 mb-lg-0">



Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's with all the whitespace changes?

Copy link
Contributor Author

@TomBirbeckTFS TomBirbeckTFS Feb 14, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not too sure, the line numbers seem to be the same for both the removed and added








<li class="nav-item">
<a class="nav-link " href="/bootstrap-theme/forms/">Forms</a>
</li>










<li class="nav-item">
<a class="nav-link " href="/bootstrap-theme/error/">Error</a>
</li>










<li class="nav-item">
<a class="nav-link " href="/bootstrap-theme/html-ipsum/">HTML Ipsum</a>
</li>

<li class="nav-item">
<button class="nav-link js-loader-toggle" type="button">
Toggle loader
Expand Down Expand Up @@ -97,9 +97,9 @@
</nav>

<div class="container">

<main id="content" tabindex="-1">




Expand Down Expand Up @@ -281,55 +281,55 @@ <h6>h6, .zeta</h6>
</ul>
</div>



<h2>Buttons</h2>
<h3>Normal</h3>

<button
class="btn btn-default "
type="button"
>Normal button</button>


<button
class="btn btn-primary "
type="button"
>Primary button</button>


<button
class="btn btn-success "
type="button"
>Success button</button>


<button
class="btn btn-warning "
type="button"
>Warning button</button>


<button
class="btn btn-info "
type="button"
>Info button</button>


<button
class="btn btn-danger "
type="button"
>Danger button</button>


<button
class="btn btn-link "
type="button"
>Link button</button>


<h3>Large</h3>

<button
class="btn btn-default btn-lg"
type="button"
Expand Down Expand Up @@ -403,51 +403,51 @@ <h3>XS</h3>

<h2>Buttons</h2>
<h3>Normal</h3>

<button
class="btn btn-default "
type="button"
>Normal button</button>


<button
class="btn btn-primary "
type="button"
>Primary button</button>


<button
class="btn btn-success "
type="button"
>Success button</button>


<button
class="btn btn-warning "
type="button"
>Warning button</button>


<button
class="btn btn-info "
type="button"
>Info button</button>


<button
class="btn btn-danger "
type="button"
>Danger button</button>


<button
class="btn btn-link "
type="button"
>Link button</button>


<h3>Large</h3>

<button
class="btn btn-default btn-lg"
type="button"
Expand Down Expand Up @@ -657,7 +657,7 @@ <h2>Tables</h2>
</tr>
</thead>
<tbody class="table-group-divider">

<tr>
<td>1</td>
<td>
Expand Down Expand Up @@ -688,7 +688,7 @@ <h2>Tables</h2>
</td>
</tr>


<tr>
<td>2</td>
<td>
Expand Down Expand Up @@ -719,7 +719,7 @@ <h2>Tables</h2>
</td>
</tr>


<tr>
<td>3</td>
<td>
Expand Down Expand Up @@ -750,7 +750,7 @@ <h2>Tables</h2>
</td>
</tr>


<tr>
<td>4</td>
<td>
Expand Down Expand Up @@ -781,7 +781,7 @@ <h2>Tables</h2>
</td>
</tr>


<tr>
<td>5</td>
<td>
Expand Down
Loading