Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create the default section for component config #12180

Closed
Tracked by #12176
nkylstad opened this issue Jan 28, 2024 · 8 comments · Fixed by #12232 or #12295
Closed
Tracked by #12176

Create the default section for component config #12180

nkylstad opened this issue Jan 28, 2024 · 8 comments · Fixed by #12232 or #12295
Labels
area/ui-editor Area: Related to the designer tool for assembling app UI in Altinn Studio. status/ready-for-dev Status: Used for issues that are ready for development. Has been through grooming.

Comments

@nkylstad
Copy link
Member

nkylstad commented Jan 28, 2024

Description

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

  • Component name/type (as header of the section, with icon)
  • Component id
  • Component data model bindings (only for components that have dataModelBindings property in schema)

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.

@nkylstad nkylstad added area/ui-editor Area: Related to the designer tool for assembling app UI in Altinn Studio. status/ready-for-dev Status: Used for issues that are ready for development. Has been through grooming. area/ui-component labels Jan 28, 2024
@nkylstad nkylstad changed the title Create the default section Create the default section for component config Jan 28, 2024
@nkylstad nkylstad moved this to 📈 Todo in Team Studio Jan 29, 2024
@ghost ghost self-assigned this Jan 31, 2024
@ghost ghost moved this from 📈 Todo to 👷 In Progress in Team Studio Jan 31, 2024
@ghost ghost linked a pull request Feb 5, 2024 that will close this issue
3 tasks
@ghost ghost moved this from 👷 In Progress to 🔎 Review in Team Studio Feb 5, 2024
@ghost ghost removed their assignment Feb 5, 2024
@TomasEng TomasEng self-assigned this Feb 7, 2024
@ghost ghost closed this as completed in #12232 Feb 9, 2024
@github-project-automation github-project-automation bot moved this from 🔎 Review to 🧪 Test in Team Studio Feb 9, 2024
@ghost ghost unassigned TomasEng Feb 9, 2024
@TomasEng TomasEng self-assigned this Feb 12, 2024
@TomasEng
Copy link
Contributor

TomasEng commented Feb 12, 2024

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.

@TomasEng TomasEng reopened this Feb 12, 2024
@TomasEng TomasEng moved this from 🧪 Test to 👀 Test feedback in Team Studio Feb 12, 2024
@TomasEng TomasEng removed their assignment Feb 12, 2024
@ghost
Copy link

ghost commented Feb 12, 2024

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
Copy link
Member Author

nkylstad commented Feb 12, 2024

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.

@ghost
Copy link

ghost commented Feb 12, 2024

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 ghost self-assigned this Feb 12, 2024
@ghost ghost moved this from 👀 Test feedback to 👷 In Progress in Team Studio Feb 12, 2024
@ghost ghost mentioned this issue Feb 12, 2024
3 tasks
@ghost ghost linked a pull request Feb 12, 2024 that will close this issue
3 tasks
@ghost
Copy link

ghost commented Feb 12, 2024

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

@ghost ghost moved this from 👷 In Progress to 🔎 Review in Team Studio Feb 12, 2024
@ghost ghost removed their assignment Feb 12, 2024
@TomasEng
Copy link
Contributor

TomasEng commented Feb 13, 2024

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.

@ghost ghost closed this as completed in #12295 Feb 13, 2024
@github-project-automation github-project-automation bot moved this from 🔎 Review to 🧪 Test in Team Studio Feb 13, 2024
@nkylstad
Copy link
Member Author

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

@nkylstad nkylstad moved this from 🧪 Test to ✅ Done in Team Studio Feb 14, 2024
@standeren
Copy link
Contributor

Ser bra ut for meg! 🎉

@standeren standeren self-assigned this Feb 14, 2024
@ghost ghost assigned ghost and unassigned standeren Feb 14, 2024
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui-editor Area: Related to the designer tool for assembling app UI in Altinn Studio. status/ready-for-dev Status: Used for issues that are ready for development. Has been through grooming.
Projects
Archived in project
3 participants