]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-21067 Fixes from validation
authorJeremy Davis <jeremy.davis@sonarsource.com>
Mon, 4 Dec 2023 17:05:23 +0000 (18:05 +0100)
committersonartech <sonartech@sonarsource.com>
Mon, 4 Dec 2023 20:03:22 +0000 (20:03 +0000)
server/sonar-web/src/main/js/apps/settings/components/inputs/PropertySetInput.tsx
server/sonar-web/src/main/js/apps/settings/components/pullRequestDecorationBinding/AlmSpecificForm.tsx

index 8c9bd9d96434637b0ed4de34a98be2a768ff5087..5dbaccb068d8ff0f1df298afd184a287cbb843d9 100644 (file)
@@ -77,25 +77,27 @@ export default class PropertySetInput extends React.PureComponent<DefaultSpecial
                   name={getUniqueName(definition, field.key)}
                   onChange={(value) => this.handleInputChange(index, field.key, value)}
                   setting={newSetting}
-                  size="auto"
+                  size="full"
                   value={fieldValues[field.key]}
                 />
               </ContentCell>
             );
           })}
         <ActionCell className="sw-border-0">
-          {!isLast && (
-            <DestructiveIcon
-              Icon={TrashIcon}
-              aria-label={translateWithParameters(
-                'settings.definitions.delete_fields',
-                getPropertyName(setting.definition),
-                index,
-              )}
-              className="js-remove-value"
-              onClick={() => this.handleDeleteValue(index)}
-            />
-          )}
+          <div className="sw-w-9">
+            {!isLast && (
+              <DestructiveIcon
+                Icon={TrashIcon}
+                aria-label={translateWithParameters(
+                  'settings.definitions.delete_fields',
+                  getPropertyName(setting.definition),
+                  index,
+                )}
+                className="js-remove-value"
+                onClick={() => this.handleDeleteValue(index)}
+              />
+            )}
+          </div>
         </ActionCell>
       </TableRow>
     );
@@ -106,7 +108,7 @@ export default class PropertySetInput extends React.PureComponent<DefaultSpecial
     const displayedValue = [...this.ensureValue(), ...getEmptyValue(definition)];
 
     const columnWidths = (isCategoryDefinition(definition) ? definition.fields : [])
-      .map(() => 'auto')
+      .map(() => '50%')
       .concat('1px');
 
     return (
index de956ccca47bf544605fb101d3372be1b85b473f..59d2ca83c057567f01f201716e0669d66a4837ee 100644 (file)
@@ -23,7 +23,7 @@ import { FormattedMessage } from 'react-intl';
 import withAvailableFeatures, {
   WithAvailableFeaturesProps,
 } from '../../../../app/components/available-features/withAvailableFeatures';
-import DocLink from '../../../../components/common/DocLink';
+import DocumentationLink from '../../../../components/common/DocumentationLink';
 import MandatoryFieldMarker from '../../../../components/ui/MandatoryFieldMarker';
 import { ALM_DOCUMENTATION_PATHS } from '../../../../helpers/constants';
 import { translate } from '../../../../helpers/l10n';
@@ -297,7 +297,9 @@ export function AlmSpecificForm(props: AlmSpecificFormProps) {
           help: true,
           helpParams: {
             doc_link: (
-              <DocLink to={ALM_DOCUMENTATION_PATHS[alm]}>{translate('learn_more')}</DocLink>
+              <DocumentationLink to={ALM_DOCUMENTATION_PATHS[alm]}>
+                {translate('learn_more')}
+              </DocumentationLink>
             ),
           },
           id: 'monorepo',