misc: adapt popups with no form to work with gadjo dialog (#84831) #944
vdeniaud
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/84831-popup-with-no-form"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -92,3 +92,3 @@response.reason_phrase = self.quixote_response.reason_phraseelif request.headers.get('X-Popup') == 'true':response = HttpResponse('<div class="popup-content">%s</div>' % context['body'])response = HttpResponse('<div><div class="popup-content">%s</div></div>' % context['body'])pour permettre au sélecteur d'attraper le .div-content il faut une balise supplémentaire autour.
@ -9,3 +9,3 @@<ul class="extra-actions-menu">{% block actions-menu-extra-links %}<li><a href="public-url" rel="popup">{% trans "Display public URL" %}</a></li><li><a href="public-url" rel="popup" data-selector=".popup-content div">{% trans "Display public URL" %}</a></li>C'est moche, heureusement les popups sans formulaire sont très rares (et j'espère vraiment qu'il n'y a que ces deux-ci). (spoiler : il y en aura d'autres).
WIP: misc: adapt popups with no form to work with gadjo dialog (#84831)to misc: adapt popups with no form to work with gadjo dialog (#84831)@ -149,3 +150,4 @@raise KeyErrorexcept KeyError:r += htmltext('<h2>%s</h2>') % _('Tracking code not available')r += htmltext('<div class="tracking-code-popup-content">')Le code de gadjo a une gestion basique pour les titres, il faut isoler le contenu pour qu'ils ne se trouvent pas répétés dans la popup.
@ -101,2 +101,4 @@if script_name == 'popup.js':self.add_javascript(['../../i18n.js', 'jquery.js', 'jquery-ui.js'])if not get_request().is_in_backoffice():self.add_javascript(['gadjo.js'])En front il faut être explicite et charger gadjo si on veut une popup.