Advies

Door Jop B op 05-05-2020 20:09

1.742 views

Voor mijn website ben ik bezig een AJAX add to cart functie te maken.
Alles functioneert maar wil het nog iet mooie maken qua layout.

Ik wil graag een popup die meld dat het is toegevoegd aan de winkelwagen.
Van de PHP file krijg ik JSON terug met success of already en de productnaam


if(data.status == 'success'){
//data.title
}

if(data.status == 'already'){
//data.title
}

Als success dan wil ik een groen balkje en bij already een grijs.
Dit zou ik kunnen doen met een addClass.

Nu mijn vraag

Zet ik de complete popup/modal al in de body en voeg ik met jQuery de info toe aan elementen of laat ik de PHP file een template retourneren?

Ivo P

05-05-2020 20:16

Ik zou voor A gaan.
Als je een modal opneemt voor allerlei messages dan ben je in 1x klaar.

misschien kun je een js functie scrijven zodat


showMessage('product is toegevoegd', 'success');

genoeg is.
Dan hoef je niet bij elke messsage opnieuw een modal toe te voegen aan de DOM.
Al helemaal niet door dan in JS een stuk html op te bouwen.

Vanuit PHP een hele template sturen is mi. not done: dan is je code ook maar precies 1x bruikbaar.
en je hebt dan layout in je gewone template staan EN in de php code die eigenlijk alleen maar iets doet als "voeg toe aan database" of iets dergelijks.

je

Jop B

05-05-2020 21:35

Hi Ivo,

Je bedoeld zoiets


function(msg, title){
var html_modal = '<div class="modal"><div class="message">+msg+</div><div class="title">+title+</div></div>';

$('body').append(html_modal);
}

De msg en title kun je dan vullen?

- Ariën -

05-05-2020 21:36

Geef de functie dan wel even een naam mee ;-)

Ivo P

05-05-2020 22:47

nee ik bedoel


<html> ....
<div class="modal" id="messagemodal">
  <div class="message" id="message"></div>
  <div class="title" id="title"></div>
</div>
</html>


function showMessage(tekst, titel, status)
{

   $('#message').text(tekst);
   $('#title').text(tekst);
   $('#messagemodal').show();
}

Ik zou namelijk niet html IN js verpakken...
[/code]

Reageren

Inloggen om te reageren