Chore: Lee's code sweep (during Hacktoberfest '24) (#2450)

* Removed my outdated TODO comments

* Manifest Picker code tidy-up

* Attribute tidy-up

Things I'd picked up during development

* Corrects Storybook control type for `umb-localize` key

* Ran `npm run lint:fix`

Author: leekelleher

examples/manifest-picker/manifest-picker-dashboard.ts

@@ -1,11 +1,13 @@
 import { css, html, customElement, state, when, nothing } from '@umbraco-cms/backoffice/external/lit';
 import { umbExtensionsRegistry } from '@umbraco-cms/backoffice/extension-registry';
-import { UmbInputManifestElement } from '@umbraco-cms/backoffice/components';
+import type { UmbInputManifestElement } from '@umbraco-cms/backoffice/components';
 import { UmbLitElement } from '@umbraco-cms/backoffice/lit-element';
 import { UmbTextStyles } from '@umbraco-cms/backoffice/style';
-import { UUISelectEvent } from '@umbraco-cms/backoffice/external/uui';
+import type { UUISelectEvent } from '@umbraco-cms/backoffice/external/uui';
 
+// eslint-disable-next-line local-rules/enforce-umb-prefix-on-element-name
 @customElement('example-manifest-picker-dashboard')
+// eslint-disable-next-line local-rules/enforce-element-suffix-on-element-class-name, local-rules/umb-class-prefix
 export class ExampleManifestPickerDashboard extends UmbLitElement {
 	#options: Array<Option> = [];
 

src/apps/preview/preview.context.ts

@@ -149,7 +149,6 @@ export class UmbPreviewContext extends UmbContextBase<UmbPreviewContext> {
 		window.open(url, '_blank');
 	}
 
-	// TODO: [LK] Figure out how to make `iframe.contentDocument` works, as it's not from SameOrigin.
 	reloadIFrame(iframe: HTMLIFrameElement) {
 		const document = iframe.contentDocument;
 		if (!document) return;

src/assets/lang/ar.ts

@@ -26,10 +26,8 @@ export interface UmbCollectionConfiguration {
 export interface UmbCollectionColumnConfiguration {
 	alias: string;
 	header: string;
-	// TODO: [LK] Figure out why the server API needs an int (1|0) instead of a boolean.
 	isSystem: 1 | 0;
 	elementName?: string;
-	// TODO: [LK] Remove `nameTemplate`, to be replaced with `elementName`.
 	nameTemplate?: string;
 }
 

src/packages/core/collection/types.ts

@@ -260,5 +260,5 @@ export const manifests: Array<ManifestLocalization> = [
 			culture: 'zh-tw',
 		},
 		js: () => import('../../../assets/lang/zh-tw.js'),
-	}
+	},
 ];

src/packages/core/localization/manifests.ts

@@ -12,7 +12,7 @@ const meta: Meta<UmbLocalizeElement> = {
 	argTypes: {
 		args: {
 			control: {
-				type: 'array',
+				type: 'text',
 			},
 		},
 	},

src/packages/core/localization/stories/localize.element.stories.ts

@@ -1,10 +1,10 @@
+import type { ManifestMenu } from '../menu.extension.js';
 import type { ManifestSectionSidebarAppBaseMenu, ManifestSectionSidebarAppMenuKind } from './types.js';
 import { UmbTextStyles } from '@umbraco-cms/backoffice/style';
 import { css, html, customElement, property } from '@umbraco-cms/backoffice/external/lit';
 import type { UmbExtensionManifestKind } from '@umbraco-cms/backoffice/extension-registry';
 import { umbExtensionsRegistry } from '@umbraco-cms/backoffice/extension-registry';
 import { UmbLitElement } from '@umbraco-cms/backoffice/lit-element';
-import type { ManifestMenu } from '../menu.extension.js';
 
 // TODO: Move to separate file:
 const manifest: UmbExtensionManifestKind = {

src/packages/core/menu/section-sidebar-menu/section-sidebar-menu.element.ts

@@ -154,7 +154,7 @@ export class UmbSectionSidebarContextMenuElement extends UmbLitElement {
 				display: flex;
 				align-items: center;
 			}
-			#action-modal umb-entity-action-list{
+			#action-modal umb-entity-action-list {
 				--uui-menu-item-flat-structure: 0;
 			}
 		`,

src/packages/core/section/section-sidebar-context-menu/section-sidebar-context-menu.element.ts

@@ -91,7 +91,7 @@ export class UmbDocumentTypeWorkspaceEditorElement extends UmbLitElement {
 							.value=${this._name}
 							.alias=${this._alias}
 							?auto-generate-alias=${this._isNew}
-							@change="${this.#onNameAndAliasChange}"
+							@change=${this.#onNameAndAliasChange}
 							required
 							${umbBindToValidation(this, '$.name', this._name)}
 							${umbFocus()}>

src/packages/documents/document-types/workspace/document-type-workspace-editor.element.ts

@@ -91,20 +91,19 @@ export class UmbDocumentTypeWorkspaceViewTemplatesElement extends UmbLitElement
 
 	#renderTemplates() {
 		return html`
-			<uui-box headline="${this.localize.term('treeHeaders_templates')}">
+			<uui-box headline=${this.localize.term('treeHeaders_templates')}>
 				${when(
 					this.#workspaceContext?.createTemplateMode,
 					() => html`<p><em>The default template will be created once this document type has been saved.</em></p>`,
 				)}
-				<umb-property-layout
-					alias="Templates"
-					label="${this.localize.term('contentTypeEditor_allowedTemplatesHeading')}">
+				<umb-property-layout alias="Templates" label=${this.localize.term('contentTypeEditor_allowedTemplatesHeading')}>
 					<div slot="description">${this.localize.term('contentTypeEditor_allowedTemplatesDescription')}</div>
 					<div id="templates" slot="editor">
 						<umb-input-template
 							.defaultUnique=${this._defaultTemplateId ?? ''}
 							.selection=${this._allowedTemplateIds}
-							@change=${this.#templateInputChange}></umb-input-template>
+							@change=${this.#templateInputChange}>
+						</umb-input-template>
 					</div>
 				</umb-property-layout>
 			</uui-box>