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

Popup zoekpagina #726

Closed
Wolfr opened this issue Feb 17, 2021 · 6 comments
Closed

Popup zoekpagina #726

Wolfr opened this issue Feb 17, 2021 · 6 comments
Assignees
Labels
css-refactor This code exists on the css-refactor branch good first issue Good for newcomers validated

Comments

@Wolfr
Copy link
Contributor

Wolfr commented Feb 17, 2021

image

Je kan hier geraken door op http://localhost:4200/zoeken/agendapunten te klikken op het info icoontje bij trefwoord.

In de popup van de zoekpagina staat er een tabel kolom te veel.

Ook zou de laatste lijn niet zichtbaar moeten zijn.

@Wolfr
Copy link
Contributor Author

Wolfr commented Feb 18, 2021

In het prototype ziet die popup er al beter uit.

Kunnen we de code van het prototype naar de front-end brengen?

https://kaleidos.mono.global/app/search/agendapunten

@Wolfr Wolfr added the good first issue Good for newcomers label Feb 18, 2021
@brenner-company
Copy link
Contributor

Aanpassingen op de volgende branch: DES/enhancement/#726-searchpage-popover

@brenner-company
Copy link
Contributor

@Wolfr Ik mis enkel nog een entry binnen nl-be.json voor 'Hulp bij zoeken'. Is dit iets wat ik zelf zou mogen toevoegen binnen die file? (desnoods vraag ik het eens aan iemand van de devs)

@brenner-company brenner-company added the question Further information is requested label Feb 22, 2021
@brenner-company brenner-company self-assigned this Feb 22, 2021
@Wolfr
Copy link
Contributor Author

Wolfr commented Feb 22, 2021

Ja, labels mag je gewoon zelf toevoegen.

@brenner-company brenner-company added css-refactor This code exists on the css-refactor branch and removed question Further information is requested labels Feb 23, 2021
@brenner-company
Copy link
Contributor

Momenteel wordt de popover nog getoond via een AttachPopover, maar binnen de prototypes van Mono zie ik dat er eerder voor een EmberPopover wordt gegaan. Komt er nog een specifiek popover component binnen app of kijk om tijdens #732 om gelijk te trekken met het prototype?

@brenner-company brenner-company added the question Further information is requested label Feb 23, 2021
@Wolfr
Copy link
Contributor Author

Wolfr commented Feb 23, 2021

Ik heb hier geen kennis over. Wat ik wel weet is dat we een zo modern mogelijke versie van Popper willen gebruiken (De 2.x branch op https://popper.js.org/docs/v2/ )

@brenner-company brenner-company added validated and removed question Further information is requested labels Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css-refactor This code exists on the css-refactor branch good first issue Good for newcomers validated
Development

No branches or pull requests

2 participants