@brightspace-ui/core 3.159.5 → 3.159.7

package/components/form/form-error-summary.js

@@ -10,7 +10,7 @@ class FormErrorSummary extends LocalizeCoreElement(LitElement) {
 		return {
 			errors: { type: Object, attribute: false },
 			_expanded: { type: Boolean, attribute: false },
-			_hasBottomMargin: { type: Boolean, attribute: '_has-bottom-margin', reflect: true },
+			_hasTopMargin: { type: Boolean, attribute: '_has-top-margin', reflect: true },
 			_hasErrors: { type: Boolean, attribute: '_has-errors', reflect: true },
 		};
 	}
@@ -24,8 +24,8 @@ class FormErrorSummary extends LocalizeCoreElement(LitElement) {
 			:host([hidden]) {
 				display: none;
 			}
-			:host([_has-bottom-margin][_has-errors]) {
-				margin-block-end: 1rem;
+			:host([_has-top-margin][_has-errors]) {
+				margin-block-start: 1rem;
 			}
 
 			.d2l-form-error-summary-header {
@@ -58,7 +58,7 @@ class FormErrorSummary extends LocalizeCoreElement(LitElement) {
 		super();
 		this.errors = [];
 		this._expanded = true;
-		this._hasBottomMargin = false;
+		this._hasTopMargin = false;
 		this._hasErrors = false;
 	}
 

package/components/progress/README.md

@@ -21,3 +21,4 @@ A progress bar communicates information relating to the progress of completion o
 | `label` | String | Label for the progress bar |
 | `label-hidden` | Boolean | Hide the bar's label |
 | `value-hidden` | Boolean | Hide the bar's value |
+<!-- docs: end hidden content -->

package/components/progress/progress.js

@@ -10,11 +10,35 @@ class Progress extends LitElement {
 
 	static get properties() {
 		return {
+			/**
+			 * The maximum value of the progress bar
+			 * @type {number}
+			 */
 			max: { type: Number, attribute: 'max' },
+			/**
+			 * The current value of the progress bar
+			 * @type {number}
+			 */
 			value: { type: Number },
+			/**
+			 * Label for the progress bar
+			 * @type {string}
+			 */
 			label: { type: String },
+			/**
+			 * Hide the bar's label
+			 * @type {boolean}
+			 */
 			labelHidden: { type: Boolean, attribute: 'label-hidden' },
+			/**
+			 * Hide the bar's value
+			 * @type {boolean}
+			 */
 			valueHidden: { type: Boolean, attribute: 'value-hidden' },
+			/**
+			 * Use d2l-body-small styles
+			 * @type {boolean}
+			 */
 			small: { type: Boolean, reflect: true }
 		};
 	}
@@ -89,6 +113,7 @@ class Progress extends LitElement {
 		this.max = 100;
 		this.value = 0;
 		this.valueHidden = false;
+		this.small = false;
 	}
 
 	render() {

package/custom-elements.json

@@ -11733,29 +11733,36 @@
       "attributes": [
         {
           "name": "label",
+          "description": "Label for the progress bar",
           "type": "string"
         },
-        {
-          "name": "small",
-          "type": "boolean"
-        },
         {
           "name": "label-hidden",
+          "description": "Hide the bar's label",
           "type": "boolean",
           "default": "false"
         },
         {
           "name": "max",
+          "description": "The maximum value of the progress bar",
           "type": "number",
           "default": "100"
         },
         {
           "name": "value",
+          "description": "The current value of the progress bar",
           "type": "number",
           "default": "0"
         },
         {
           "name": "value-hidden",
+          "description": "Hide the bar's value",
+          "type": "boolean",
+          "default": "false"
+        },
+        {
+          "name": "small",
+          "description": "Use d2l-body-small styles",
           "type": "boolean",
           "default": "false"
         }
@@ -11764,34 +11771,41 @@
         {
           "name": "label",
           "attribute": "label",
+          "description": "Label for the progress bar",
           "type": "string"
         },
-        {
-          "name": "small",
-          "attribute": "small",
-          "type": "boolean"
-        },
         {
           "name": "labelHidden",
           "attribute": "label-hidden",
+          "description": "Hide the bar's label",
           "type": "boolean",
           "default": "false"
         },
         {
           "name": "max",
           "attribute": "max",
+          "description": "The maximum value of the progress bar",
           "type": "number",
           "default": "100"
         },
         {
           "name": "value",
           "attribute": "value",
+          "description": "The current value of the progress bar",
           "type": "number",
           "default": "0"
         },
         {
           "name": "valueHidden",
           "attribute": "value-hidden",
+          "description": "Hide the bar's value",
+          "type": "boolean",
+          "default": "false"
+        },
+        {
+          "name": "small",
+          "attribute": "small",
+          "description": "Use d2l-body-small styles",
           "type": "boolean",
           "default": "false"
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brightspace-ui/core",
-  "version": "3.159.5",
+  "version": "3.159.7",
   "description": "A collection of accessible, free, open-source web components for building Brightspace applications",
   "type": "module",
   "repository": "https://github.com/BrightspaceUI/core.git",

package/templates/primary-secondary/demo/form.html

@@ -15,8 +15,8 @@
 			import '../primary-secondary.js';
 		</script>
 		<style>
-			d2l-template-primary-secondary {
-				--d2l-template-primary-secondary-primary-padding: 20px;
+			div[slot="primary"] {
+				padding: 0 20px;
 			}
 			div[slot="secondary"] {
 				padding: 10px;

package/templates/primary-secondary/primary-secondary.js

@@ -643,9 +643,12 @@ class TemplatePrimarySecondary extends LocalizeCoreElement(LitElement) {
 			main {
 				flex: 2 0 0;
 				overflow-x: hidden;
-				padding: var(--d2l-template-primary-secondary-primary-padding, 0);
 				transition: none;
 			}
+			main d2l-form-error-summary {
+				margin-inline: var(--d2l-template-primary-secondary-form-error-inline-margin, 20px);
+			}
+
 			:host([resizable]) main {
 				flex: 1 0 0;
 			}
@@ -1052,7 +1055,7 @@ class TemplatePrimarySecondary extends LocalizeCoreElement(LitElement) {
 			'd2l-template-scroll': isWindows
 		};
 		const primarySection = html`<main class="${classMap(scrollClasses)}">
-			${this.hasForm ? html`<d2l-form-error-summary _has-bottom-margin id="form-error-summary"></d2l-form-error-summary>` : nothing}
+			${this.hasForm ? html`<d2l-form-error-summary _has-top-margin id="form-error-summary"></d2l-form-error-summary>` : nothing}
 			<slot name="primary"></slot>
 		</main>`;
 		const secondarySection = html`