SONAR-18776 Branch selector scroll fix and resolving code smells

diff --git a/server/sonar-web/src/main/js/app/components/nav/component/__tests__/Header-test.tsx b/server/sonar-web/src/main/js/app/components/nav/component/__tests__/Header-test.tsx
index 5add2af3e0e2608ff6bf367763da021e9e9b41ca..04286b17cd1a586545af9bdb0f7279db2e121946 100644 (file)
--- a/server/sonar-web/src/main/js/app/components/nav/component/__tests__/Header-test.tsx
+++ b/server/sonar-web/src/main/js/app/components/nav/component/__tests__/Header-test.tsx
@@ -29,6 +29,7 @@ import { mockComponent } from '../../../../../helpers/mocks/component';
 import { mockCurrentUser, mockLoggedInUser } from '../../../../../helpers/testMocks';
 import { renderApp } from '../../../../../helpers/testReactTestingUtils';
 import { AlmKeys } from '../../../../../types/alm-settings';
+import { ComponentQualifier } from '../../../../../types/component';
 import { Feature } from '../../../../../types/features';
 import { BranchStatusContext } from '../../../branch-status/BranchStatusContext';
 import { Header, HeaderProps } from '../Header';
@@ -74,7 +75,7 @@ it('should show manage branch and pull request button for admin', async () => {
     currentUser: mockLoggedInUser(),
     component: mockComponent({
       configuration: { showSettings: true },
-      breadcrumbs: [{ name: 'project', key: 'project', qualifier: 'TRK' }],
+      breadcrumbs: [{ name: 'project', key: 'project', qualifier: ComponentQualifier.Project }],
     }),
   });
   await user.click(screen.getByRole('button', { name: 'branch-1 overview.quality_gate_x.OK' }));
@@ -131,7 +132,7 @@ it('should show the correct help tooltip for applications', () => {
     currentUser: mockLoggedInUser(),
     component: mockComponent({
       configuration: { showSettings: true },
-      breadcrumbs: [{ name: 'project', key: 'project', qualifier: 'APP' }],
+      breadcrumbs: [{ name: 'project', key: 'project', qualifier: ComponentQualifier.Application }],
       qualifier: 'APP',
     }),
     branchLikes: [mockMainBranch()],
@@ -182,7 +183,7 @@ function renderHeader(props?: Partial<HeaderProps>, featureList = [Feature.Branc
       <Header
         branchLikes={branchLikes}
         component={mockComponent({
-          breadcrumbs: [{ name: 'project', key: 'project', qualifier: 'TRK' }],
+          breadcrumbs: [{ name: 'project', key: 'project', qualifier: ComponentQualifier.Project }],
         })}
         currentBranchLike={branchLikes[0]}
         currentUser={mockCurrentUser()}

diff --git a/server/sonar-web/src/main/js/app/components/nav/component/branch-like/MenuItemList.tsx b/server/sonar-web/src/main/js/app/components/nav/component/branch-like/MenuItemList.tsx
index ee21a7a07bc69f423610e4e6764b0a9270b238dc..a8c6f9b78f8c653b31655ceff6a2d907136716a7 100644 (file)
--- a/server/sonar-web/src/main/js/app/components/nav/component/branch-like/MenuItemList.tsx
+++ b/server/sonar-web/src/main/js/app/components/nav/component/branch-like/MenuItemList.tsx
@@ -59,7 +59,7 @@ export function MenuItemList(props: MenuItemListProps) {
   );
 
   return (
-    <ul className="item-list sw-overflow-scroll">
+    <ul className="item-list sw-overflow-y-auto sw-overflow-x-hidden">
       {!hasResults && (
         <div className="sw-px-3 sw-py-2">
           <span>{translate('no_results')}</span>

diff --git a/server/sonar-web/src/main/js/app/components/nav/component/branch-like/QualityGateStatus.tsx b/server/sonar-web/src/main/js/app/components/nav/component/branch-like/QualityGateStatus.tsx
index 66daeb578423419bad81643ce857a57530c772f7..492299f49c019fc0656498738e26325329aa37c3 100644 (file)
--- a/server/sonar-web/src/main/js/app/components/nav/component/branch-like/QualityGateStatus.tsx
+++ b/server/sonar-web/src/main/js/app/components/nav/component/branch-like/QualityGateStatus.tsx
@@ -24,6 +24,7 @@ import { getBranchStatusByBranchLike } from '../../../../../helpers/branch-like'
 import { translateWithParameters } from '../../../../../helpers/l10n';
 import { formatMeasure } from '../../../../../helpers/measures';
 import { BranchLike } from '../../../../../types/branch-like';
+import { MetricType } from '../../../../../types/metrics';
 import { Component } from '../../../../../types/types';
 import { BranchStatusContext } from '../../../branch-status/BranchStatusContext';
 
@@ -52,7 +53,7 @@ export default function QualityGateStatus({
     return null;
   }
   const { status } = branchStatus;
-  const formatted = formatMeasure(status, 'LEVEL');
+  const formatted = formatMeasure(status, MetricType.Level);
   const ariaLabel = translateWithParameters('overview.quality_gate_x', formatted);
   return (
     <div className={classNames(className, `it__level-${status}`)}>