Visuell oppgradering av "siste prosjekter" webdel

[Remi749]

Pull request (PR)

Sørg for at du ber om PR for din branch (høyre side). Sørg for at du gjør en PR mot riktig release-branch (venstre side). Sjekk commits og alle commit-meldingene.

Sjekklisten din

Alle sjekkpunktene under må være sjekket av og godkjent for at vi skal kunne merge branchen din mot dev.

• Sjekk at din branch ikke feiler på linting.
• Legg ved beskrivelse i CHANGELOG, markert med ID av issue knyttet til PR-en
• Angi korrekt Milestone på PR-en og issuet, samt tilegn deg selv PR-en og legg til labels

Beskrivelse

Webdelen har fått en helomvendig og bruker nå Fluent UI 9 og er tilpasset UU.

Endring å merke seg:

• Det er mulig å skru av og på prosjektlogo (default: på)
• Hovereffekter og mer feedback
• Lenker åpnes i ny fane som default og man kan anvende (musehjul klikk)

Før	Etter

Med bilder slått på	Interaktivitet

Hvordan teste

Punkter for testing av det visuelle som gjennomføres i 1.9 kommer som en egen PR senere.

Relevante issues (hvis aktuelt)

• Visuell oppgradering 1/2 #1132

Sjekkliste for godkjenner

Alle sjekkpunktene under må være sjekket av og godkjent av reviewers for at vi skal kunne merge branchen din mot dev.

• Sjekk at det er fylt ut testpunkter
• Sjekk om det er nødvendig å nevne denne PR i release notes
• Sjekk om det er nødvendig å oppdatere dokumentasjon for hjelpeinnhold
  • Ta en titt på Brukermanual for Prosjektportalen 365 og vurder behovet for oppdateringer.

[olemp]

@Remi749 Jeg synes det blir for mye med prosjektlogo her også når prosjektutlistingswebdelen baserer seg på prosjektloger. Hva tenker dere @pzljanb og @tarjeieo?

Jeg synes personlig det blir ryddigere og holde den "tekstbasert" (som det er nå i 1.9):

[Remi749]

@Remi749 Jeg synes det blir for mye med prosjektlogo her også når prosjektutlistingswebdelen baserer seg på prosjektloger. Hva tenker dere @pzljanb og @tarjeieo?

Jeg synes personlig det blir ryddigere og holde den "tekstbasert" (som det er nå i 1.9):

Her er det helhetlig.

Vi kan eventuelt ha det av default?

[olemp]

@Remi749 Jeg synes det blir for mye med prosjektlogo her også når prosjektutlistingswebdelen baserer seg på prosjektloger. Hva tenker dere @pzljanb og @tarjeieo?
Jeg synes personlig det blir ryddigere og holde den "tekstbasert" (som det er nå i 1.9):

Her er det helhetlig.

Vi kan eventuelt ha det av default?

Jeg foretrekker denne:

Tanken med webdelen er en enkel utlisting. Jeg synes ikke vi skal gjøre den mer kompleks.

@tarjeieo og @pzljanb får siste ordet <3

[Remi749]

Oppdatert PR og kode, bilder er default av

[olemp]

Skjønner at vi bør ha lenkene mer tydelig klikkbar i forhold til UU, men synes det er alt for mye endringer og komplisering av kode for små endringer. Det er f.eks. over 200 linjer styling og kode for å vise logo som er default av, og de fleste ikke vil ta i bruk.

[Remi749]

Skjønner at vi bør ha lenkene mer tydelig klikkbar i forhold til UU, men synes det er alt for mye endringer og komplisering av kode for små endringer. Det er f.eks. over 200 linjer styling og kode for å vise logo som er default av, og de fleste ikke vil ta i bruk.

Tenkte å gjøre en forbedringer her, skrev følgende i PR

Gjøre avatar/bilde innlastningen til en delt komponent?

[olemp]

Skjønner at vi bør ha lenkene mer tydelig klikkbar i forhold til UU, men synes det er alt for mye endringer og komplisering av kode for små endringer. Det er f.eks. over 200 linjer styling og kode for å vise logo som er default av, og de fleste ikke vil ta i bruk.

Tenkte å gjøre en forbedringer her, skrev følgende i PR

Gjøre avatar/bilde innlastningen til en delt komponent?

Kan ikke den gjøres som en del av denne PRen? Så unngår vi at det blir liggende for mye teknisk gjeld i 1.9 branchen? Eller er det en god grunn til å merge det as-is og gjøre forbedringen senere?

[Remi749]

Skjønner at vi bør ha lenkene mer tydelig klikkbar i forhold til UU, men synes det er alt for mye endringer og komplisering av kode for små endringer. Det er f.eks. over 200 linjer styling og kode for å vise logo som er default av, og de fleste ikke vil ta i bruk.

Tenkte å gjøre en forbedringer her, skrev følgende i PR

Gjøre avatar/bilde innlastningen til en delt komponent?

Kan ikke den gjøres som en del av denne PRen? Så unngår vi at det blir liggende for mye teknisk gjeld i 1.9 branchen? Eller er det en god grunn til å merge det as-is og gjøre forbedringen senere?

Tenkte egentlig å ta det som en egen PR da dette vil påvirke Prosjektutlisting også

[Remi749]

@olemp , laget ProjectLogo til en delt komponent. Gjerne sjekk over :) Lager en egen PR for Prosjektutlisting senere.

[olemp]

This is added even when logo is not enabled, which causes some wierd UI. Will fix in next commit @Remi749