-
Notifications
You must be signed in to change notification settings - Fork 132
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
wet-366 Create Working Examples for Utilities/Helpers: Forms (GCWeb) #2234
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some additional change required
"fr": "Les éléments HTML utilisé tel quel." | ||
}, | ||
"modified": "2023-02-21", | ||
"componentName": "Utilities", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"componentName": "Utilities", | |
"componentName": "utilities", |
{ | ||
"title": "Forms", | ||
"language": "en", | ||
"path": "forms.html" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"path": "forms.html" | |
"path": "helpers/forms.html" |
{ | ||
"title": "Formulaires", | ||
"language": "fr", | ||
"path": "formulaires.html" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"path": "formulaires.html" | |
"path": "helpers/formulaires.html" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This index.json-ld need to be one level up, Here: common/utilities/
common/utilities/helpers/forms.html
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing the following modifier as a static example and a few more. Check the project card for all the CSS that need to be demoed:
- has-success
…ilities/index.json-ld
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Remove the merge commit
- Fix the content structure as commented inline
- Fix the lint spacing issue
common/utilities/forms.html
Outdated
<code class="language-html" data-lang="html"> | ||
<input class="form-control input-lg" type="datetime" placeholder=".form-control .input-lg" /> | ||
<input class="form-control" type="datetime-local" placeholder="Default input" /> | ||
<input class="form-control input-sm" type="date" placeholder=".form-control .input-sm" /> | ||
<input class="form-control input-lg" type="month" placeholder=".form-control .input-lg" /> | ||
<input class="form-control" type="datetime-time" placeholder="Default input" /> | ||
<input class="form-control input-sm" type="week" placeholder=".form-control .input-sm" /> | ||
</code> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<code class="language-html" data-lang="html"> | |
<input class="form-control input-lg" type="datetime" placeholder=".form-control .input-lg" /> | |
<input class="form-control" type="datetime-local" placeholder="Default input" /> | |
<input class="form-control input-sm" type="date" placeholder=".form-control .input-sm" /> | |
<input class="form-control input-lg" type="month" placeholder=".form-control .input-lg" /> | |
<input class="form-control" type="datetime-time" placeholder="Default input" /> | |
<input class="form-control input-sm" type="week" placeholder=".form-control .input-sm" /> | |
</code> | |
<pre><code class="language-html" data-lang="html"> | |
<input class="form-control input-lg" type="datetime" placeholder=".form-control .input-lg" /> | |
<input class="form-control" type="datetime-local" placeholder="Default input" /> | |
<input class="form-control input-sm" type="date" placeholder=".form-control .input-sm" /> | |
<input class="form-control input-lg" type="month" placeholder=".form-control .input-lg" /> | |
<input class="form-control" type="datetime-time" placeholder="Default input" /> | |
<input class="form-control input-sm" type="week" placeholder=".form-control .input-sm" /> | |
</code></pre> |
common/utilities/forms.html
Outdated
<input class="form-control input-sm" type="week" placeholder=".form-control .input-sm" /> | ||
</div> | ||
|
||
<br/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<br/> |
common/utilities/forms.html
Outdated
<br/> | ||
|
||
|
||
<strong>input type: datetime</strong> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<strong>input type: datetime</strong> | |
<h4>input type: datetime</h4> |
common/utilities/forms.html
Outdated
</label> | ||
</div> | ||
|
||
<div class="prettyprint prettyprinted"><pre><code class="language-html" data-lang="html"><div class="checkbox"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<div class="prettyprint prettyprinted"><pre><code class="language-html" data-lang="html"><div class="checkbox"> | |
<pre class="prettyprint prettyprinted"><code class="language-html" data-lang="html"><div class="checkbox"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AtlakM
Missing the example for the following CSS style:
.form-control-static
https://github.com/twbs/bootstrap-sass/blob/6b0ca5025c64f2e5696c8719cfce7148c78d0a50/assets/stylesheets/bootstrap/_forms.scss#L298-L311.form-group-sm
- https://github.com/twbs/bootstrap-sass/blob/6b0ca5025c64f2e5696c8719cfce7148c78d0a50/assets/stylesheets/bootstrap/_forms.scss#L314-L372.form-group-lg
- https://github.com/twbs/bootstrap-sass/blob/6b0ca5025c64f2e5696c8719cfce7148c78d0a50/assets/stylesheets/bootstrap/_forms.scss#L314-L372- Checkbox and radio options - with [data-toggle="buttons"] - https://github.com/twbs/bootstrap-sass/blob/6b0ca5025c64f2e5696c8719cfce7148c78d0a50/assets/stylesheets/bootstrap/_button-groups.scss#L222
- Missing a few
input-group
example (input-group-sm input-group-lg) - See the CSS here - https://github.com/twbs/bootstrap-sass/blob/6b0ca5025c64f2e5696c8719cfce7148c78d0a50/assets/stylesheets/bootstrap/_input-groups.scss - btn-group-xs btn-group-sm btn-group-lg - https://github.com/twbs/bootstrap-sass/blob/6b0ca5025c64f2e5696c8719cfce7148c78d0a50/assets/stylesheets/bootstrap/_button-groups.scss#L96-L98
- btn-group-justified - https://github.com/twbs/bootstrap-sass/blob/6b0ca5025c64f2e5696c8719cfce7148c78d0a50/assets/stylesheets/bootstrap/_button-groups.scss#L96-L98
- an example that show the use of gc-font-2019 that wrap a whole form -
GCWeb/sites/baseline/_forms.scss
Line 42 in 8e3a4f0
// Apply the new font size to form, for early adopter
No description provided.