Altinn / altinn-studio

Next generation open source Altinn platform and applications.
https://docs.altinn.studio
BSD 3-Clause "New" or "Revised" License
110 stars 71 forks source link

Create the default section for component config #12180

Closed nkylstad closed 5 months ago

nkylstad commented 5 months ago

Description

Create the default section for component config information, and move the following config into it:

Figma link: https://www.figma.com/file/VAvGOqkMhKM8HL8h4xBeDH7b/Re-design-Altinn-Studio?type=design&node-id=9905-69066&mode=design&t=V5WeYulqoU64hc3N-4

This issue also includes removing the same information from the "Instillinger" panel (both regular and beta) to avoid duplicating this information.

Updating the designs of the id and datamodelBinding component will be part of other issues.

TomasEng commented 5 months ago

Har testet denne og oppdaget noen ting jeg tror ikke er helt sklik det skal:

ghost commented 5 months ago

Har testet denne og oppdaget noen ting jeg tror ikke er helt sklik det skal:

  • Når jeg skal velge en datamodell, får jeg ikke opp noe i listen, selv om jeg har opprettet og generert en datamodell.
  • Kunne vi endret teksten "legg til datamodell" til "legg til datamodellknytning"? Det ville vært mer presist.
  • Designet ser litt rotete ut: image Størrelser og avstander er ikke konsistente, ting ligger ikke på linje og knappene avviker fra de andre knappene vi bruker. Jeg ser det er noe triksing med position og negativ margin her, det bør vi unngå nettopp fordi det gjør det vanskelig å få ting til å passe med hverandre i forskjellige situsjoner. Kunne vi fått det til å se mer ut som dette? image
  • Knapper som ikke har tekst bør ha title-attributt, men det mangler her.

Takk for tilbakemeldingen @TomasEng. Basert på det du skriver, så lurer jeg på om det er noe misforståelser om issuet. Jeg prøver å gå gjennom hvert at punktene dine:

  • Når jeg skal velge en datamodell, får jeg ikke opp noe i listen, selv om jeg har opprettet og generert en datamodell.

Merkelig, jeg får det når jeg tester hos meg selv: Image

  • Kunne vi endret teksten "legg til datamodell" til "legg til datamodellknytning"? Det ville vært mer presist.

Det kan vi helt sikkert 😄

  • Designet ser litt rotete ut:

Det er jeg helt enig med deg i, men som Nina skriver i issuet, så er dette issuet bare å flytte koden fra der den var til der den er nå. Oppdateringer på design kommer senere, og er definert i egne issues 😄 Screenshot av det som står i issuet: Image

nkylstad commented 5 months ago

Jeg opplever noe av det samme som @TomasEng her:

https://github.com/Altinn/altinn-studio/assets/1636323/383104fa-8528-4a4c-9708-905764a35a28

ghost commented 5 months ago

Jeg opplever noe av det samme som @TomasEng her:

  • Design på datamodellknytning har blitt rart 🤔 Enig i at dette issuet ikke handler om å fikse opp designet på de forskjellige komponentene, bare å flytte dem i denne omgang. Men her er det noe som har skjedd med visningen, det er ikke likt som det var da datamodellknytning lå under i "Innhold". Se video:

    Screen.Recording.2024-02-12.at.20.07.32.mov

  • ~Jeg får heller ikke opp noe i listen over datamodell-knytninger, selv om jeg tidligere har fått det på denne appen.~ Glem det, jeg fikk de opp etter at jeg trykket på generer modeller på nytt og lastet inn hele siden på nytt.

@nkylstad, takk for videoen! Det hjalp i forståelsen på hva som ikke var som det skulle være her. Jeg skal se på det 😄

ghost commented 5 months ago

@TomasEng @nkylstad Problemet som Nina viste i videoen er fikset og løst i denne PRen 😄

TomasEng commented 5 months ago

Når jeg skal velge en datamodell, får jeg ikke opp noe i listen, selv om jeg har opprettet og generert en datamodell.

Testet dette på nytt nå, og det fungerte. Kanskje gjorde jeg noe feil, vi får ta det videre hvis problemet dukker opp igjen.

nkylstad commented 5 months ago

Tested in dev, I'm happy with this now. We should update the ID- and datamodel-components as soon as possible, and potentially consider setting up a separate section for datamodel rather than keeping it in the "default" section. This will be especially important for components that can have multiple data model bindings. CC: @Annikenkbrathen

standeren commented 5 months ago

Ser bra ut for meg! 🎉