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

ffe tilbakemeldingskomponent #1730

Merged
merged 3 commits into from
Dec 5, 2023
Merged

Conversation

isakbjugn
Copy link
Contributor

@isakbjugn isakbjugn commented Nov 29, 2023

Beskrivelse

Tilbakemeldingskomponent for å la brukere gi vurdering og innspill til enkeltsider. Komponenten er implementert i React, med less-stilark i separat pakke.

Starttilstand til komponenten (tomler vises):
image

Tommel opp er klikket, og tekstfelt vises:
image

Tommel opp er klikket, og tekstfelt vises, og komponenten er tatt i bruk med en lenke som attributt (her til kundeservice):
image

Slutttilstand til komponenten (teksttilbakemelding er sendt inn):
image

Motivasjon og kontekst

Mange sider i Digitalbanken har en form for tilbakemeldingskomponent, med tommel opp/ned og et fritekstfelt. Disse må implementeres i hver enkelt app som bruker en slik komponent. Ved å ha en ffe-komponent for tilbakemeldinger, reduserer vi duplisert kode og gjør at vedlikehold kan gjøres på ett enkelt sted.

Testing

  • Har lagt til to varianter av komponenten i component-overiview, som bruker ulike farger, ulike språk og ulike attributter.
  • Har laget et eksempel hvor komponenten tas i bruk i appen kundefront-pm-boliglansdetaljer.

Til diskusjon

  • Er pakkene dratt inn på riktig måte i component-overview
  • Jeg har slitt med å eksportere Thumbs og ThumbValues fra ffe-feedback-react, og håper enda at dette kan gjøres på en ryddigere måte (f.eks. eksportere enum fra annen fil enn index.d.ts
  • Har inntil videre laget ett eksempel i component-overview, men denne kan deles opp i flere filer for å tydeliggjøre brukstilfellene

@isakbjugn isakbjugn requested a review from a team as a code owner November 29, 2023 12:35
@isakbjugn isakbjugn force-pushed the ffe-tilbakemeldingskomponent branch 2 times, most recently from bd345d9 to 176d188 Compare November 29, 2023 13:03
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-1730.westeurope.2.azurestaticapps.net

"@sb1/ffe-collapse-react": "^2.0.6",
"@sb1/ffe-core-react": "^7.1.4",
"@sb1/ffe-feedback": "^1.0.0",
"@sb1/ffe-form": "^25.0.2",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Du trenger ikke ha dependency til "ffe-form", fordi den er indrekte med gjennom at den er en dependency i ffe-form-react

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okei, det visste jeg ikke! Da skal jeg forenkle i package.json ☺️

Gjelder det for øvrig i appene der disse bibliotekene brukes også, at dersom man har avhengighet til react-pakken, så trenger man ikke eksplisitt avhengighet til less-pakken? (Men kan fremdeles importere less-filen ettersom less-pakken finnes i node-modules?)

@HeleneKassandra
Copy link
Contributor

Ser bra ut! Tenker kanskje som du også skriver at eksemplene burde deles opp, sånn at hvert eksempel kun viser ett eksempel på bruk.

Sånn til Team Designsystem: Vi må legge til støtte for theming, fordi sånn det er nå kan ikke komponenten brukes av f.eks partnerbanker.

@isakbjugn isakbjugn force-pushed the ffe-tilbakemeldingskomponent branch from 176d188 to df6a915 Compare November 30, 2023 11:53
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-1730.westeurope.2.azurestaticapps.net

@isakbjugn isakbjugn force-pushed the ffe-tilbakemeldingskomponent branch from df6a915 to 740f253 Compare November 30, 2023 12:59
@isakbjugn isakbjugn closed this Nov 30, 2023
@isakbjugn isakbjugn reopened this Nov 30, 2023
@isakbjugn
Copy link
Contributor Author

Ser bra ut! Tenker kanskje som du også skriver at eksemplene burde deles opp, sånn at hvert eksempel kun viser ett eksempel på bruk.

Har delt opp i fem eksempler nå, som viser:

  • Komponenten i enkleste form
  • Med satt bakgrunnsfarge
  • Med bølgen og bakgrunnsfarge fra GridRow
  • Med standard-lenke
  • Med egendefinert lenke

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-1730.westeurope.2.azurestaticapps.net

1 similar comment
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-1730.westeurope.2.azurestaticapps.net

@isakbjugn isakbjugn merged commit 5da9253 into develop Dec 5, 2023
6 checks passed
@isakbjugn isakbjugn deleted the ffe-tilbakemeldingskomponent branch December 5, 2023 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants