Poenai / webwinkel

Een Javascript en XML Webwinkel
1 stars 1 forks source link

contact formulier altijd in beeld #3

Closed barld closed 10 years ago

barld commented 10 years ago

lijkt een chrome specefieke bug die nog niet eerder optrad verschillende browsers getest krijg alleen in chrome altijd het contact formulier onder in beeld

Poenai commented 10 years ago

Hoi Barld, morgen krijg je nadere instructies van me. Ik zag inderdaad al sinds de overgang naar XML dat in de verschillende browsers de webwinkel anders getoond werd (in chrome zag ik bijv. geen producten meer op de voorpagina, in IE wel) Tot morgen

Poenai commented 10 years ago

BTW contactformulier kan eventueel weg als dat teveel tijd kost om te fixen, aangezien er nog andere prioriteiten liggen en het formulier eigenlijk vrij overbodig is. Er kan contact opgenomen worden via de hoofdsite. Ik zal dat morgen even beoordelen.

barld commented 10 years ago
<?php
$xml = simplexml_load_file("xml/producten.xml");
?>

<script>
        var Catalog = <?= json_encode($xml); ?>.product;
</script>

overigens als er PHP beschikbaar is kan je op deze manier het xmltojson in javascript er weer uithalen. Dit scheelt twee requists en maakt de pagina daarmee sneller. 1. het js framework hoeft niet ingeladen te worden uit een apart bestand. Er hoeft niet meer een requist te worden gedaan naar het XML bestand en dit ook nog een omzetten. Er is nu gewoon direct een json string aanwezig in het html document die als bron nogsteeds het xml bestand heeft waar wijzigingen in gedaan kunnen worden die direct doorgevoerd woren

Poenai commented 10 years ago

Ok goed plan, kun je tevens kijken of dat het browser probleem oplost? Er is PHP beschikbaar.

barld commented 10 years ago

dan ga ik dat morgen toepassen heb al een klein beetje ergens anders getest zou moeten werken

barld commented 10 years ago

XML to JSON verwijdert en functie ervan vervangen met PHP pull requist gemaakt zie #7 lost ook deze bug met het contact formulier in Chrome op (als u de requist goed vind dan kan dit probleem als opgelost worden gezien)

barld commented 10 years ago

7 doorgevoerd