Feature/several small UI tweaks

docker-compose-local.yml

@@ -1,4 +1,3 @@
-version: '3.8'
 services:
   oidc-provider:
     ports:

docker-compose.override.yml

@@ -1,5 +1,3 @@
-version: '3.8'
-
 services:
   oidc-provider:
     build:

docker-compose.yml

@@ -1,4 +1,3 @@
-version: '3.8'
 services:
   oidc-provider:
     build:

e2e/docker-compose.ci.yml

@@ -1,4 +1,3 @@
-version: '3.8'
 services:
   frontend:
     volumes:

e2e/docker-compose.yml

@@ -1,4 +1,3 @@
-version: '3.8'
 services:
   frontend:
     build:

e2e/test/login.test.ts

@@ -5,5 +5,5 @@ import { DEV_USER } from '@utils/users.js';
 test('Login', async ({ browser }) => {
   const { page } = await login(browser, DEV_USER);
 
-  await expect(page).toHaveURL('https://localhost:1443/');
+  await expect(page).toHaveURL(/^https:\/\/localhost:1443.*/);
 });

frontend/src/components/SplitButton.tsx

@@ -20,6 +20,7 @@ interface Props {
   variant?: 'contained' | 'outlined';
   hideSelectedOptionFromList?: boolean;
   disableButtonSelection?: boolean;
+  hideButtonSelection?: boolean;
   /** Functions to be triggered immediately when a menu item is clicked. Overrides default menu item selection logic. */
   directOptionFunctions?: ((() => void) | null)[];
 }
@@ -31,6 +32,7 @@ export function SplitButton({
   variant = 'outlined',
   hideSelectedOptionFromList,
   disableButtonSelection,
+  hideButtonSelection,
   directOptionFunctions,
 }: Props) {
   const [open, setOpen] = useState(false);
@@ -66,9 +68,11 @@ export function SplitButton({
     <>
       <ButtonGroup {...(cssProp && { css: cssProp })} variant={variant} ref={anchorRef}>
         {renderButton(options[selectedIndex], selectedIndex)}
-        <Button disabled={disableButtonSelection} size="small" onClick={handleToggle}>
-          <ArrowDropDown />
-        </Button>
+        {!hideButtonSelection && (
+          <Button disabled={disableButtonSelection} size="small" onClick={handleToggle}>
+            <ArrowDropDown />
+          </Button>
+        )}
       </ButtonGroup>
       <Popper
         css={css`

frontend/src/views/Project/ProjectViewWrapper/ModifyButton.tsx

@@ -82,12 +82,13 @@ export function ModifyButton({
       <>
         <SplitButton
           hideSelectedOptionFromList
-          disableButtonSelection={editing || splitButtonDisabled}
+          hideButtonSelection={editing}
+          disableButtonSelection={splitButtonDisabled}
           cssProp={css`
             margin-left: auto;
           `}
           variant={editing ? 'outlined' : 'contained'}
-          options={[tr('projectObjectView.modify'), tr('projectObjectView.moveToProject')]}
+          options={[tr('projectObjectView.modify'), tr('projectObjectView.moveToProjectButton')]}
           directOptionFunctions={[null, () => setPopupOpen(true)]}
           renderButton={(label, idx) => (
             <Button
@@ -133,7 +134,8 @@ export function ModifyButton({
     <>
       <SplitButton
         hideSelectedOptionFromList
-        disableButtonSelection={editing || splitButtonDisabled}
+        hideButtonSelection={editing}
+        disableButtonSelection={splitButtonDisabled}
         cssProp={css`
           margin-left: auto;
         `}

frontend/src/views/Project/ProjectViewWrapper/ProjectObjectMoveView.tsx

@@ -87,7 +87,12 @@ export function ProjectObjectMoveView(props: Props) {
           padding-bottom: 1rem;
         `}
       >
-        <DialogContentText>{tr('projectObjectView.moveToProject.contentText')}</DialogContentText>
+        <DialogContentText>
+          {tr('projectObjectView.moveToProject.contentText')}{' '}
+          {newProjectCandidates.data &&
+            newProjectCandidates.data.length > 0 &&
+            tr('projectObjectView.moveToProject.contentTextSap')}
+        </DialogContentText>
       </DialogContent>
       <DialogActions
         css={css`
@@ -136,19 +141,31 @@ export function ProjectObjectMoveView(props: Props) {
                 )}
               />
             ) : (
-              <Typography>{tr('projectObjectView.moveToProject.noProjects')}</Typography>
+              <Typography color="error">
+                {tr('projectObjectView.moveToProject.noProjects')}
+              </Typography>
             )}
             <Box
               css={css`
                 margin-left: auto;
+                display: flex;
+                gap: 10px;
               `}
             >
-              <Button onClick={props.handleClose} color="primary">
-                {tr('cancel')}
-              </Button>
-              <Button disabled={!isValid} type="submit" variant="contained">
-                {tr('projectObjectView.move')}
-              </Button>
+              {!newProjectCandidates.isLoading && newProjectCandidates.data?.length === 0 ? (
+                <Button onClick={props.handleClose} color="primary" variant="contained">
+                  {tr('ok')}
+                </Button>
+              ) : (
+                <>
+                  <Button onClick={props.handleClose} color="primary">
+                    {tr('cancel')}
+                  </Button>
+                  <Button disabled={!isValid} type="submit" variant="contained">
+                    {tr('projectObjectView.move')}
+                  </Button>
+                </>
+              )}
             </Box>
           </form>
         </FormProvider>

frontend/src/views/Project/ProjectViewWrapper/ProjectShiftView.tsx

@@ -139,14 +139,16 @@ export function ProjectShiftView(props: Props) {
             />
             <Box
               css={css`
+                display: flex;
                 margin-left: auto;
+                gap: 10px;
               `}
             >
               <Button onClick={props.handleClose} color="primary">
                 {tr('cancel')}
               </Button>
               <Button disabled={!isValid} type="submit" color="primary" variant="contained">
-                {tr('ok')}
+                {tr('move')}
               </Button>
             </Box>
           </form>

shared/src/language/fi.ts

@@ -59,11 +59,14 @@ export const fi = {
   'projectSearch.generateReport': 'Lataa raportti',
   'projectSearch.reportFailed': 'Raportin luonti epäonnistui.',
   'projectObjectView.modify': 'Muokkaa kohdetta',
+  'projectObjectView.moveToProjectButton': 'Siirrä kohde toiselle hankkeelle',
   'projectObjectView.moveToProject': 'Siirretään kohde toiselle hankkeelle',
   'projectObjectView.moveToProject.label': 'Kohteen uusi hanke',
   'projectObjectView.moveToProject.contentText':
-    'Kohteen pystyy siirtämään toiselle investointihankkeelle, jonka toteutusväli sekä lautakuntavalinta mahdollistavat kohteen siirron. Kohteen mahdollinen SAP-rakenneosan tunniste poistetaan siirron yhteydessä.',
-  'projectObjectView.moveToProject.noProjects': 'Ei sopivia hankkeita.',
+    'Kohteen voi siirtää investointihankkeelle, jolla on sopiva toteutusväli sekä oikea lautakunta.',
+  'projectObjectView.moveToProject.contentTextSap':
+    'Kohteen mahdollinen SAP-rakenneosan tunniste poistetaan siirron yhteydessä.',
+  'projectObjectView.moveToProject.noProjects': 'Sopivia hankkeita ei ole.',
   'projectObjectView.moveToProject.formError': 'Uusi hanke puuttuu.',
   'projectObjectView.moveToProject.success': 'Kohde siirretty onnistuneesti.',
   'projectObjectView.moveToProject.failed': 'Kohteen siirto epäonnistui.',
@@ -316,7 +319,7 @@ export const fi = {
   'dateRange.endDate': 'Loppupäivä',
   'budgetTable.year': 'Vuosi',
   'budgetTable.yearHelp':
-    'Talousluvut osoitetaan vuosikohtaisina. Näkymässä tarjolla olevat vuodet johdetaan suoraan käyttäjän kirjaamasta toteutusvälistä (alku- ja loppupäivämäärä).',
+    'Talousluvut osoitetaan vuosikohtaisina. Näkymässä tarjolla olevat vuodet johdetaan suoraan käyttäjän kirjaamasta toteutusvälistä (alku- ja lopetusajankohta).',
   'budgetTable.yearHelpOngoing':
     'Toistaiseksi jatkuvan hankkeen talouslukuja on mahdollista raportoida viideksi vuodeksi eteenpäin.',
   'budgetTable.yearHelpOngoingObject':
@@ -378,10 +381,10 @@ export const fi = {
   'projectObject.objectUsageLabelShort': 'Käyttötarkoitus',
   'projectObject.objectUsageTooltip': 'Valitse kohteen toiminnallinen käyttötarkoitus.',
   'projectObject.committeeLabel': 'Lautakunta',
-  'projectObject.startDateLabel': 'Alkupäivämäärä',
-  'projectObject.startDateTooltip': 'Valitse kohteelle aloituspäivämäärä.',
-  'projectObject.endDateLabel': 'Loppupäivämäärä',
-  'projectObject.endDateTooltip': 'Valitse kohteen lopetuspäivämäärä.',
+  'projectObject.startDateLabel': 'Alkuajankohta',
+  'projectObject.startDateTooltip': 'Valitse kohteelle aloitusajankohta.',
+  'projectObject.endDateLabel': 'Loppuajankohta',
+  'projectObject.endDateTooltip': 'Valitse kohteen lopetusajankohta.',
   'projectObject.sapWBSIdLabel': 'SAP-rakenneosa',
   'projectObject.sapWBSIdTooltip': 'Valitse SAP-rakenneosa.',
   'projectObject.sapWBSIdNoSAPProjectIdTooltip': 'Hankkeelle ei ole määritetty SAP-projektia.',
@@ -631,10 +634,10 @@ export const fi = {
   'workTable.export.projectName': 'Hanke',
   'workTable.export.objectName': 'Kohde',
   'workTable.export.lifecycleState': 'Tila',
-  'workTable.export.projectStartDate': 'Hankkeen aloituspäivämäärä',
-  'workTable.export.projectEndDate': 'Hankkeen lopetuspäivämäärä',
-  'workTable.export.objectStartDate': 'Kohteen aloituspäivämäärä',
-  'workTable.export.objectEndDate': 'Kohteen lopetuspäivämäärä',
+  'workTable.export.projectStartDate': 'Hankkeen aloitusajankohta',
+  'workTable.export.projectEndDate': 'Hankkeen lopetusajankohta',
+  'workTable.export.objectStartDate': 'Kohteen aloitusajankohta',
+  'workTable.export.objectEndDate': 'Kohteen lopetusajankohta',
   'workTable.export.objectType': 'Tyyppi',
   'workTable.export.objectCategory': 'Omaisuusluokka',
   'workTable.export.objectUsage': 'Käyttötarkoitus',
@@ -681,6 +684,7 @@ export const fi = {
   'wholeMunicipalityMapInfoBoxButton.tooltip': 'Koko kunnan aluetta koskevat hankkeet',
   'wholeMunicipalityMapInfoBoxButton.singleTooltip': 'Koko kunnan aluetta koskeva hanke',
   'email.subject': 'Aihe',
+  move: 'Siirrä',
   send: 'Lähetä',
   clear: 'Tyhjennä',
   ok: 'Ok',