search

Geonovum / pdokkaart

API and the wizard for PDOKKaart
http://geonovum.github.com/pdokkaart/
14 stars 18 forks source link

Foutjes in code? #137

Open JanKK opened 11 years ago

JanKK commented 11 years ago

Collega krijgt indruk dat volgende niet klopt in pagina’s (index en help):

<body> Moet zijn: <body id="content">

<div id="content"> Moet zijn: <div class="content">

Dan werkt ook de footer, die nu ontbreekt/wellicht daarom is weggelaten:

<div class="footer"> <div class="ref"></div> <span></span> </div>

JanKK commented 11 years ago

Aanvulling: http://dl.dropbox.com/u/4652903/thommaas.txt (in een extern bestandje omdat Github soms wat raar omgaat met tonen van code)

rduivenvoorde commented 11 years ago

over welke code hebben we het?

de wizard code of de api code?

want de laatst heeft helemaal geen doen met styles van body's etc?

JanKK commented 11 years ago

Collega heeft de hele wizard in eigen omgeving gezet, wizard code dus

rduivenvoorde commented 11 years ago

beetje moeilijk om hier verder op in te gaan.

Wij hebben inderdaad de footer weggelaten (bewust). Als ik bovenstaande toevoeg, krijg ik niet een werkende footer.

Wat 'klopt' er dan verder niet?

JanKK commented 11 years ago

Moet ik even navragen, wordt vervolgd...

ThomMaas commented 11 years ago

Zie zowel de Wizard: http://83.83.18.63/pdokkaart/ Als een gefatsoeneerde Help: http://83.83.18.63/pdokkaart/help.html

Mijn Wizard: regels 29, 60 (nieuwe class: wizard), 329 t/m 332

Mijn Help: regels 13, (t.v.v. de style: jullie 13 t/m 25), 15, 38 (nieuwe id help) + veel tags gefatsoeneerd, inhoudelijk niet aangepast.

In main.css toevoegen: .wizard { min-height: 600px; padding: 10px 0 0; top: 0px; }

help {

padding: 24px; }

Groet, Thom Maas (collega van JanKK)

LuukS commented 11 years ago

De bovenstaande aanpassingen geprobeerd maar weer verwijderd omdat de wizard er lelijker op werd. De kaart en de stappen waren niet meer boven uitgelijnd op dezelfde hoogte. Richard en ik vonden dat dat niet de bedoeling was.

ThomMaas commented 11 years ago

Zie: http://83.83.18.63/pdokkaart/

In main.css onder .stepwrapper toevoegen (zoals in template.css): .top { margin-top: 0; padding-top: 0; }

div id="search" class="stepwrapper top"

Curieus om het issue te sluiten: het probleem (geen footer) negeer je dan dus. Als je dat niet aanpakt, krijg je volgens mij problemen met de rijkshuisstijl-webpolitie.

Met vriendelijke groet, Thom Maas

rduivenvoorde commented 11 years ago

Misschien ten onrechte helemaal afgesloten vanwege de druk om v1.0 op te leveren. Hierbij reopened om bij een volgende versie te bekijken