Skip to content

Commit

Permalink
fix(core) Do not use static string bindings for boolean values (#3236)
Browse files Browse the repository at this point in the history
  • Loading branch information
salarenko committed Sep 11, 2020
1 parent b66ca4b commit df9e33d
Show file tree
Hide file tree
Showing 17 changed files with 37 additions and 37 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,9 @@ import { DIALOG_REF, DialogRef } from '@fundamental-ngx/core';
<fd-dialog-footer-button>
<button
fd-button
compact="true"
fdType="emphasized"
fd-dialog-decisive-button
[compact]="true"
(click)="this.dialogRef.close('Continue')"
>
Interesting
Expand All @@ -33,10 +33,10 @@ import { DIALOG_REF, DialogRef } from '@fundamental-ngx/core';
<fd-dialog-footer-button>
<button
fd-button
compact="true"
fdType="transparent"
fd-dialog-decisive-button
fd-initial-focus
[compact]="true"
(click)="this.dialogRef.dismiss('Cancel')"
>
Cancel
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,10 @@ <h1 fd-dialog-title>Backdrop Configuration</h1>
<fd-dialog-footer-button>
<button
fd-button
compact="true"
fd-initial-focus
fdType="emphasized"
fd-dialog-decisive-button
[compact]="true"
(click)="dialog.close()"
>
Close
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,11 @@ <h1 fd-dialog-title>Fresh Market</h1>
<fd-bar-element fullWidth="true">
<fd-input-group
fd-initial-focus
button="true"
compact="true"
glyph="search"
placement="after"
placeholder="Search"
[button]="true"
[compact]="true"
[(ngModel)]="searchedPhrase"
>
</fd-input-group>
Expand Down Expand Up @@ -60,7 +60,7 @@ <h1 fd-dialog-title>Fresh Market</h1>
fd-dialog-decisive-button
glyph="cart"
fdType="emphasized"
compact="true"
[compact]="true"
(click)="checkout()"
>
Checkout
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,10 @@ <h1 fd-dialog-title>Origin of the Pineapple Word</h1>
<fd-dialog-footer-button>
<button
fd-button
compact="true"
fdType="emphasized"
fd-initial-focus
fd-dialog-decisive-button
[compact]="true"
(click)="dialog.close()"
>
Interesting
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@
<div fd-bar-middle>
<fd-bar-element [fullWidth]="true">
<fd-input-group glyph="menu"
button="true"
compact="true">
[button]="true"
[compact]="true">
</fd-input-group>
</fd-bar-element>
</div>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,10 @@ <h1 fd-dialog-title>More Facts about the Pineapple</h1>
<fd-dialog-footer-button>
<button
fd-button
compact="true"
fdType="emphasized"
fd-initial-focus
fd-dialog-decisive-button
[compact]="true"
(click)="dialog.close()"
>
Interesting
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,10 @@ <h1 fd-dialog-title>Dialog states</h1>
<fd-dialog-footer-button>
<button
fd-button
compact="true"
fdType="emphasized"
fd-initial-focus
fd-dialog-decisive-button
[compact]="true"
(click)="dialog.close()">
Ok
</button>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,10 @@ import { SecondDialogExampleComponent } from './second-dialog-example.component'
<fd-dialog-footer-button>
<button
fd-button
compact="true"
fdType="emphasized"
fd-initial-focus
fd-dialog-decisive-button
[compact]="true"
(click)="openDialog()"
>
Open Second Dialog
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@ <h1 fd-dialog-title>The History of Pineapple</h1>
<fd-dialog-footer-button>
<button
fd-button
compact="true"
fdType="emphasized"
fd-dialog-decisive-button
[compact]="true"
(click)="dialog.close('Continue')">
Interested
</button>
Expand All @@ -24,10 +24,10 @@ <h1 fd-dialog-title>The History of Pineapple</h1>
<fd-dialog-footer-button>
<button
fd-button
compact="true"
fdType="transparent"
fd-initial-focus
fd-dialog-decisive-button
[compact]="true"
(click)="dialog.dismiss('Cancel')">
Cancel
</button>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,19 +1,19 @@
<div fd-form-item>
<label fd-form-label>Left Aligned Icon Button Add-on</label>
<fd-input-group button="true"
placement="before"
<fd-input-group placement="before"
glyph="accelerated"
placeholder="Amount">
placeholder="Amount"
[button]="true">
</fd-input-group>
</div>

<br/>

<div fd-form-item>
<label fd-form-label>Right Aligned Text Button Add-on</label>
<fd-input-group button="true"
placement="after"
<fd-input-group placement="after"
addOnText="Submit"
placeholder="Amount">
placeholder="Amount"
[button]="true">
</fd-input-group>
</div>
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,9 @@
<div fd-form-item>
<label fd-form-label>Search Input</label>
<fd-input-group glyph="decline"
button="true"
placeholder="Search"
formControlName="searchQuery">
formControlName="searchQuery"
[button]="true">
</fd-input-group>
</div>
<small>Search Query Entered: {{ customForm.controls.searchQuery.value }}</small>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
<div fd-form-item>
<label fd-form-label>Search input</label>
<fd-input-group
button="true"
glyph="decline"
placeholder="Search"
[button]="true"
[disabled]="false"
[(ngModel)]="searchTerm"
(addOnButtonClicked)="searchTerm = ''"
Expand All @@ -17,10 +17,10 @@
<div fd-form-item>
<label fd-form-label>Search input with search type</label>
<fd-input-group
button="true"
type="search"
glyph="search"
placeholder="Search"
[button]="true"
[disabled]="false"
[(ngModel)]="searchTermSecond"
>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@
<label fd-form-label>Information</label>
<fd-form-input-message-group>
<fd-input-group
button="true"
addOnText="Button"
state="information"
placeholder="Placeholder"
[button]="true"
></fd-input-group>
<fd-form-message type="information">
Pellentesque metus lacus commodo eget justo ut rutrum varius nunc
Expand All @@ -19,10 +19,10 @@
<label fd-form-label>Success</label>
<fd-form-input-message-group>
<fd-input-group
button="true"
state="success"
addOnText="Button"
placeholder="Placeholder"
[button]="true"
></fd-input-group>
<fd-form-message type="success">
Pellentesque metus lacus commodo eget justo ut rutrum varius nunc
Expand All @@ -36,10 +36,10 @@
<label fd-form-label>Warning</label>
<fd-form-input-message-group>
<fd-input-group
button="true"
state="warning"
addOnText="Button"
placeholder="Placeholder"
[button]="true"
></fd-input-group>
<fd-form-message type="warning">
Pellentesque metus lacus commodo eget justo ut rutrum varius nunc
Expand All @@ -53,10 +53,10 @@
<label fd-form-label>Error</label>
<fd-form-input-message-group>
<fd-input-group
button="true"
state="error"
addOnText="Button"
placeholder="Placeholder"
[button]="true"
></fd-input-group>
<fd-form-message type="error">
Pellentesque metus lacus commodo eget justo ut rutrum varius nunc
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
<div fd-form-item>
<label fd-form-label>Compact size</label>
<fd-input-group addOnText="$"
compact="true"
placement="before"
placeholder="Amount">
placeholder="Amount"
[compact]="true">
</fd-input-group>
</div>

Expand All @@ -12,9 +12,9 @@
<div fd-form-item>
<label fd-form-label>Compact size button</label>
<fd-input-group glyph="cart"
button="true"
compact="true"
placement="after"
placeholder="Amount">
placeholder="Amount"
[compact]="true"
[button]="true">
</fd-input-group>
</div>
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@
</label>
<fd-toolbar-spacer></fd-toolbar-spacer>
<fd-input-group
button="true"
compact="true"
glyph="decline"
placeholder="Search"
[button]="true"
[compact]="true"
[disabled]="false"
[(ngModel)]="searchTerm"
(ngModelChange)="handleSearchTermChange($event)"
Expand Down
2 changes: 1 addition & 1 deletion libs/core/src/lib/dialog/dialog.component.html
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
#dialogWindow
role="dialog"
tabindex="-1"
attr.aria-modal="true"
fdResize
fdResizeBoundary=".fd-dialog"
cdkDrag
Expand All @@ -25,6 +24,7 @@
[fdResizeDisabled]="!dialogConfig.resizable"
[@dialog-fade]
[attr.id]="dialogConfig.id"
[attr.aria-modal]="true"
[attr.aria-label]="dialogConfig.ariaLabel"
[attr.aria-labelledby]="dialogConfig.ariaLabelledBy"
[attr.aria-describedby]="dialogConfig.ariaDescribedBy"
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
<ng-template [ngIf]="items?.length > 0" [ngIfElse]="subHeader">

<button fd-button
fdMenu="true"
fdType="transparent"
class="fd-shellbar__button--menu"
[fdMenu]="true"
[disabled]="disabled"
[fdMenuTrigger]="menu">
<span class="fd-shellbar__title">{{ control }}</span>
Expand Down

0 comments on commit df9e33d

Please sign in to comment.