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

feat(edit): submit empty platform select to show all platforms #1831

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

SelmaBergstrand
Copy link
Collaborator

Tom plattform-select tilsvarer "vis alle"


Motivasjon

Redusere antall klikk med 2 for folk som skal lage tavle for et stoppested.

Endringer

  • når platform-select er tom tilsvarer dette "vis alle"

Sjekkliste for Review

  • Sjekk at det fortsatt går an å legge til, endre og slette både stop-places og quays som vanlig

obs: annen mulig løsning er å sette dropdownen til "vis alle"-verdien med en gang, men det viste seg å være vanskelig (kan ikke sette default-valg enkelt i dropdown-komponenten)

obsobs: hele tileselector-komponenten er jo HELT krise, alt av form handling skjer på en annen måte enn det vi gjør ellers

Copy link
Collaborator

@emilielr emilielr left a comment

Choose a reason for hiding this comment

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

Hvis vi er keen på å ha "Vis alle", så kan man legge til dette på quay-selectoren:

selectedItem={
                        selectedQuay ??
                        ({
                            value: selectedStopPlace?.value,
                            label: 'Vis alle',
                        } as NormalizedDropdownItemType)
                    }

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.

2 participants