Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
43117d4
feat(spinner): pfe-spinner 1-to-1
kylebuch8 Sep 13, 2022
55e2868
feat(spinner): adding the spinner component
kylebuch8 Sep 20, 2022
8533c8f
Merge branch 'main' into feat/pfe-spinner-1-to-1
kylebuch8 Sep 20, 2022
3fd5120
chore(spinner): updating package-lock.json
kylebuch8 Sep 20, 2022
cdf0ec9
chore(spinner): adding css properties for documentation
kylebuch8 Sep 21, 2022
f285745
chore(spinner): updating custom elements manifest config to include B…
kylebuch8 Sep 21, 2022
f4cb53d
Merge branch 'main' into feat/pfe-spinner-1-to-1
bennypowers Sep 22, 2022
1c5f0ba
Update elements/pfe-spinner/BaseSpinner.ts
kylebuch8 Sep 22, 2022
13ac507
Update elements/pfe-spinner/BaseSpinner.ts
kylebuch8 Sep 22, 2022
b682591
Update elements/pfe-spinner/BaseSpinner.ts
kylebuch8 Sep 22, 2022
4ee8cae
Update elements/pfe-spinner/BaseSpinner.scss
kylebuch8 Sep 22, 2022
3cecddf
Update elements/pfe-spinner/BaseSpinner.scss
kylebuch8 Sep 22, 2022
df62d26
Update elements/pfe-spinner/BaseSpinner.ts
kylebuch8 Sep 22, 2022
4c8daf4
test(spinner): removing check for progressbar role
kylebuch8 Sep 22, 2022
1c6ac7d
chore(spinner): adding a changeset for pfe-spinner
Sep 26, 2022
21552fe
fix(button): spinner html
bennypowers Sep 28, 2022
b768ee9
docs(spinner): update changelog
bennypowers Sep 28, 2022
af2aca8
test(spinner): test effects not settings
bennypowers Sep 28, 2022
d70f0d5
test(spinner): refactor tests
bennypowers Sep 28, 2022
2e4b59b
fix(spinner): move css vars to defaults
bennypowers Sep 28, 2022
128059b
docs(spinner): simplify demo page
bennypowers Sep 28, 2022
d679ef8
chore: update package lock
bennypowers Sep 28, 2022
b5c0127
Update elements/pfe-spinner/BaseSpinner.ts
kylebuch8 Sep 28, 2022
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
17 changes: 17 additions & 0 deletions .changeset/seven-phones-crash.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
---
"@patternfly/pfe-spinner": minor
"@patternfly/pfe-button": patch
"@patternfly/pfe-tools": patch
---

BREAKING CHANGE:

Renames `<pfe-progress-indicator>` to `<pfe-spinner>` and changes it's HTML and
CSS APIs to more closely match PatternFly v4 design specs.

```html
<!-- BEFORE -->
<pfe-progress-indicator indeterminate>Loading...</pfe-progress-indicator>
<!-- AFTER -->
<pfe-spinner>Loading...</pfe-spinner>
```
2 changes: 1 addition & 1 deletion elements/pfe-button/demo/pfe-button.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import '@patternfly/pfe-band';
import '@patternfly/pfe-button';
import '@patternfly/pfe-icon/font-awesome.js';
import '@patternfly/pfe-progress-indicator';
import '@patternfly/pfe-spinner';
import '@patternfly/pfe-select';

const root = document.querySelector('[data-demo="pfe-button"]')?.shadowRoot ?? document;
Expand Down
2 changes: 1 addition & 1 deletion elements/pfe-button/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@
"dependencies": {
"@patternfly/pfe-core": "^2.0.0-next.8",
"@patternfly/pfe-icon": "^2.0.0-next.4",
"@patternfly/pfe-progress-indicator": "^2.0.0-next.4",
"@patternfly/pfe-spinner": "^2.0.0-next.4",
"lit": "2.3.0"
}
}
7 changes: 3 additions & 4 deletions elements/pfe-button/pfe-button.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import { customElement, property } from 'lit/decorators.js';
import { BaseButton } from './BaseButton.js';

import '@patternfly/pfe-icon';
import '@patternfly/pfe-progress-indicator';
import '@patternfly/pfe-spinner';

import styles from './pfe-button.scss';

Expand Down Expand Up @@ -160,11 +160,10 @@ export class PfeButton extends BaseButton {
protected override renderDefaultIcon() {
return html`
<pfe-icon ?hidden=${!this.icon} icon=${this.icon} size="sm"></pfe-icon>
<pfe-progress-indicator
<pfe-spinner
?hidden=${!this.loading}
indeterminate
aria-label=${this.getAttribute('progress-label') ?? 'loading'}
></pfe-progress-indicator>
></pfe-spinner>
`;
}
}
Expand Down
2 changes: 1 addition & 1 deletion elements/pfe-button/tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
},
"references": [
{ "path": "../../core/pfe-core" },
{ "path": "../pfe-progress-indicator" },
{ "path": "../pfe-spinner" },
{ "path": "../pfe-icon" },
{ "path": "../../tools/pfe-tools" }
]
Expand Down
33 changes: 0 additions & 33 deletions elements/pfe-progress-indicator/README.md

This file was deleted.

22 changes: 0 additions & 22 deletions elements/pfe-progress-indicator/demo/demo.css

This file was deleted.

44 changes: 0 additions & 44 deletions elements/pfe-progress-indicator/demo/pfe-progress-indicator.html

This file was deleted.

This file was deleted.

41 changes: 0 additions & 41 deletions elements/pfe-progress-indicator/docs/pfe-progress-indicator.md

This file was deleted.

83 changes: 0 additions & 83 deletions elements/pfe-progress-indicator/pfe-progress-indicator.scss

This file was deleted.

72 changes: 0 additions & 72 deletions elements/pfe-progress-indicator/pfe-progress-indicator.ts

This file was deleted.

Loading