fix(ui-library): comments removed

deven-org · Mar 15, 2024 · f40db60 · f40db60
1 parent cd52cc9
commit f40db60
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 20 deletions.
diff --git a/packages/ui-library/src/components/input-field-text/index.stories.ts b/packages/ui-library/src/components/input-field-text/index.stories.ts
@@ -111,7 +111,7 @@ export default {
       },
       if: { arg: 'hasLabel', eq: true },
     },
-    // todo it has to be renamed? Checke this
+
     hasIcon: {
       description: 'Choose if component has an icon.',
       options: [undefined, ...PureIconKeys],
@@ -120,8 +120,7 @@ export default {
         category: 'Content / Settings',
       },
     },
-    // todo it has to be renamed? Checke this
-    inputIcon: {
+    icon: {
       description: 'Select an icon which is displayed inside of the input.',
       options: [undefined, ...PureIconKeys],
       control: { type: 'select' },
@@ -130,8 +129,6 @@ export default {
         category: 'Content / Settings',
       },
     },
-
-    // todo renaming from hintText to hasHint in the blrFormRenderFunction, partially solution with name overwriting
     hasHint: {
       name: 'hasHint',
       description: ' Choose if component has a hint message. ',
@@ -143,7 +140,6 @@ export default {
         category: 'Content / Settings',
       },
     },
-    // todo renaming from hintMessageIcon to hintMessageIcon in the blrFormRenderFunction, partially solution with name overwriting
     hintMessageIcon: {
       name: 'hintMessageIcon',
       description: 'Select an icon which is displayed in front of the hint message.',
@@ -154,7 +150,6 @@ export default {
         category: 'Content / Settings',
       },
     },
-    // todo renaming from hintText to hasHint in the blrFormRenderFunction, partially solution with name overwriting
     hintMessage: {
       name: 'hintMessage',
       description: 'Enter string used used as hint message.',
@@ -198,7 +193,6 @@ export default {
         category: 'Validation',
       },
     },
-    // todo renaming from errorMessageIcon to errorMessageIcon, partially solution with name overwriting
     errorMessageIcon: {
       name: 'errorMessageIcon',
       description: 'Select an icon which is displayed in front of the error message.',
@@ -325,30 +319,29 @@ BlrInputFieldText.storyName = 'Input Field Text';
 
 // Default parameters for Input Field Text component
 const defaultParams: BlrInputFieldTextType = {
+  theme: 'Light',
   sizeVariant: 'md',
+  type: 'text',
   placeholder: 'Placeholder-text',
   value: '',
   maxLength: 140,
+  hasLabel: true,
   label: 'Label-text',
   labelAppendix: '(Appendix)',
+  hasIcon: true,
+  icon: 'blr360',
   hasHint: false,
   hintMessage: 'This is a small hint message',
   hintMessageIcon: 'blrInfo',
-  arialabel: 'InputFieldText',
-  name: 'InputFieldText',
-  theme: 'Light',
-  inputFieldTextId: 'Input Id',
-  hasLabel: true,
-  required: false,
   disabled: false,
   readonly: false,
+  required: false,
   hasError: false,
   errorMessage: '',
   errorMessageIcon: 'blrInfo',
-
-  type: 'text',
-  icon: 'blr360',
-  hasIcon: true,
+  arialabel: 'InputFieldText',
+  name: 'InputFieldText',
+  inputFieldTextId: 'Input Id',
 };
 
 BlrInputFieldText.args = defaultParams;
@@ -357,6 +350,7 @@ BlrInputFieldText.args = defaultParams;
 const argTypesToDisable = [
   'theme',
   'sizeVariant',
+  'type',
   'isResizeable',
   'placeholder',
   'value',
@@ -368,7 +362,6 @@ const argTypesToDisable = [
   'hasHint',
   'hintText',
   'hintMessageIcon',
-  'type',
   'hasIcon',
   'inputFieldTextId',
   'disabled',

diff --git a/packages/ui-library/src/components/input-field-text/index.ts b/packages/ui-library/src/components/input-field-text/index.ts
@@ -43,6 +43,7 @@ export class BlrInputFieldText extends LitElement {
   @property() inputFieldTextId!: string;
   @property() type: InputTypes = 'text';
   @property() arialabel!: string;
+  @property() hasLabel!: boolean;
   @property() label!: string;
   @property() labelAppendix?: string;
   @property() value!: string;
@@ -61,7 +62,7 @@ export class BlrInputFieldText extends LitElement {
   @property() hintMessage?: string;
   @property() hintMessageIcon?: SizelessIconType;
   @property() errorMessageIcon?: SizelessIconType;
-  @property() hasLabel!: boolean;
+
   @property() name!: string;
   @property() theme: ThemeType = 'Light';