Refactor jquery #65
Refactor jquery #65
Conversation
This mostly includes selecting element, adding event listeners and class toggles
| import { Theme } from './Framework/Theme' | ||
| import { FileInput } from './Framework/FileInput' | ||
|
|
||
| window.bootstrap = bootstrap |
There was a problem hiding this comment.
Misschien een comment toevoegen waarom dit gebeurd? Iets als // Expose Bootstrap so we can use it directly
There was a problem hiding this comment.
@jonasdekeukelaere weet jij nog specifiek waarvoor dit nodig was?
There was a problem hiding this comment.
voor in andere imports aan bootstrap te kunnen, @Katrienvh gaat dat beter kunnen uitleggen
| static remove (toastId) { | ||
| $(`#${toastId}`).toast('hide') | ||
| } |
There was a problem hiding this comment.
Is die remove niet meer nodig?
There was a problem hiding this comment.
Geen idee. Een toast leeft sowieso maar voor 2 requests in Symfony, dus ik weet niet of we die ooit manueel moeten verwijderen. Misschien als we een SPA of volledige Vue applicatie maken wel.
There was a problem hiding this comment.
Good point, ik gebruikte wel vaak tijdelijke toasts voor confirmation messages of zo die na x seconden vanzelf verdwijnen.
There was a problem hiding this comment.
Ik ga het zo laten en het terug toevoegen als we het nodig hebben in een project (zoals bij veel zaken in deze PR). Goed?
Het is ruw, maar wel grotendeels functioneel. Ik gebruik deze branch om te testen in bestaand project.
PR in core bundle: sumocoders/FrameworkCoreBundle#86