Improved/tweaked the UI

skinkie / openkvk-migratie

De migratie pagina voor openkvk

Creative Commons Zero v1.0 Universal

0 stars 1 forks source link

Improved/tweaked the UI #1

Closed boekkooi closed 10 years ago

boekkooi commented 10 years ago

Extracted openkvk js widget
Removed form element and replaced it with input-group
Added loading bar/indicator
Removed some style attributes
Fixed a bug where the api was called with a empty string
Use bootstrap modal to show nice JSON

skinkie commented 10 years ago

Thanks!! Valt er ook een mapping te maken op dat inline stukje bestaande handelsnamen?

boekkooi commented 10 years ago

@skinkie is mogelijk maar kan je dan niet beter de rendering zelf doen via de renderResult optie?

skinkie commented 10 years ago

Ik vond de magie die je had gebruikt dat het automatisch ging sowieso al super spannend ;) Wat ik me voor zou stellen is dat de array wordt samengevoegd, en dat terecht komt in een class ".brands". Als bewerken alleen kan als je alles via renderResult doet, is dat wel jammer... maar so be it.

In de nieuwe api hebben we ook geo output enzo. Dus met dat in het achterhoofd... wat is dan de beste optie?

boekkooi commented 10 years ago

@skinkie Het kan zeker maar niet op een mooie manier (of je moet iets zoals twig.js gaan gebruiken maar das echt overkill). Wat beteft de nieuwe API kan ik niet veel zeggen maar het lijkt mij beter de rendering zelf te doen via renderResult aangezien je mogelijk nog een mapje ofzo iets wil laten zien.