diff --git a/docs/components/progress-ring.md b/docs/components/progress-ring.md
index 6406d2142..fd7517b41 100644
--- a/docs/components/progress-ring.md
+++ b/docs/components/progress-ring.md
@@ -12,10 +12,10 @@ Progress rings are used to show the progress of a determinate operation in a cir
### Size
-Use the `size` attribute to set the diameter of the progress ring.
+Use the `--size` custom property to set the diameter of the progress ring.
```html preview
-
+
```
### Track Width
@@ -45,7 +45,7 @@ To change the color, use the `--track-color` and `--indicator-color` custom prop
Use the default slot to show a label.
```html preview
-50%
+50%
diff --git a/docs/resources/changelog.md b/docs/resources/changelog.md
index 446e8e9c6..72cc66816 100644
--- a/docs/resources/changelog.md
+++ b/docs/resources/changelog.md
@@ -21,6 +21,7 @@ _During the beta period, these restrictions may be relaxed in the event of a mis
- Added the undocumented custom properties `--thumb-size`, `--tooltip-offset`, `--track-height` on ``
- Changed the default `distance` in `` from `2` to `0` [#538](https://github.com/shoelace-style/shoelace/issues/538)
- Fixed a bug where `` would be larger than the viewport when it had lots of options [#544](https://github.com/shoelace-style/shoelace/issues/544)
+- Fixed a bug where `` wouldn't animate in Safari
- Updated the default height of `` from `16px` to `1rem` and added a subtle shadow to indicate depth
- Removed the `lit-html` dependency and moved corresponding imports to `lit` [#546](https://github.com/shoelace-style/shoelace/issues/546)
diff --git a/src/components/progress-ring/progress-ring.ts b/src/components/progress-ring/progress-ring.ts
index f7fc76317..29fe40189 100644
--- a/src/components/progress-ring/progress-ring.ts
+++ b/src/components/progress-ring/progress-ring.ts
@@ -1,5 +1,5 @@
import { LitElement, html } from 'lit';
-import { customElement, property, query } from 'lit/decorators.js';
+import { customElement, property, query, state } from 'lit/decorators.js';
import styles from './progress-ring.styles';
/**
@@ -22,9 +22,28 @@ export default class SlProgressRing extends LitElement {
@query('.progress-ring__indicator') indicator: SVGCircleElement;
+ @state() indicatorOffset: string;
+
/** The current progress percentage, 0 - 100. */
@property({ type: Number, reflect: true }) percentage: number;
+ updated(changedProps: Map) {
+ super.updated(changedProps);
+
+ //
+ // This block is only required for Safari because it doesn't transition the circle when the custom properties
+ // change, possibly because of a mix of pixel + unitless values in the calc() function. It seems like a Safari bug,
+ // but I couldn't pinpoint it so this works around the problem.
+ //
+ if (changedProps.has('percentage')) {
+ const radius = parseFloat(getComputedStyle(this.indicator).getPropertyValue('r'));
+ const circumference = 2 * Math.PI * radius;
+ const offset = circumference - (this.percentage / 100) * circumference;
+
+ this.indicatorOffset = String(offset) + 'px';
+ }
+ }
+
render() {
return html`