From 52c62c822eb0ff73eefe1a84bd2704d0a36cea09 Mon Sep 17 00:00:00 2001 From: Adolfo Fitoria Date: Thu, 16 Aug 2012 11:26:13 -0600 Subject: Fixed bugs on the widget creator and widget js code. Made it work with more than two widgets at the time --- askbot/migrations/0135_auto__add_askwidget.py | 2 +- askbot/models/widgets.py | 6 +++-- askbot/skins/default/media/style/askbot-modal.css | 25 ----------------- .../skins/default/templates/list_ask_widget.html | 2 +- .../default/templates/widgets/askbot_widget.css | 25 +++++++++++++++++ .../default/templates/widgets/askbot_widget.js | 31 +++++++++++++++------- askbot/urls.py | 6 +++++ askbot/views/widgets.py | 19 ++++++++++--- 8 files changed, 74 insertions(+), 42 deletions(-) delete mode 100755 askbot/skins/default/media/style/askbot-modal.css create mode 100755 askbot/skins/default/templates/widgets/askbot_widget.css diff --git a/askbot/migrations/0135_auto__add_askwidget.py b/askbot/migrations/0135_auto__add_askwidget.py index 11b287e8..ebf6f717 100644 --- a/askbot/migrations/0135_auto__add_askwidget.py +++ b/askbot/migrations/0135_auto__add_askwidget.py @@ -13,7 +13,7 @@ class Migration(SchemaMigration): ('id', self.gf('django.db.models.fields.AutoField')(primary_key=True)), ('title', self.gf('django.db.models.fields.CharField')(max_length=100)), ('group', self.gf('django.db.models.fields.related.ForeignKey')(blank=True, related_name='groups', null=True, to=orm['askbot.Tag'])), - ('default_group', self.gf('django.db.models.fields.related.ForeignKey')(to=orm['askbot.Tag'], null=True, blank=True)), + ('tag', self.gf('django.db.models.fields.related.ForeignKey')(to=orm['askbot.Tag'], null=True, blank=True)), ('include_text_field', self.gf('django.db.models.fields.BooleanField')(default=False, blank=True)), ('inner_style', self.gf('django.db.models.fields.TextField')(default='')), ('outer_style', self.gf('django.db.models.fields.TextField')(default='')), diff --git a/askbot/models/widgets.py b/askbot/models/widgets.py index 09b9a2d9..803b1293 100644 --- a/askbot/models/widgets.py +++ b/askbot/models/widgets.py @@ -40,8 +40,10 @@ class CreateAskWidgetForm(forms.ModelForm): initial=DEFAULT_OUTER_STYLE ) #these don't work for some reason - #group = forms.ModelChoiceField(query_set=get_groups()) - #tag = forms.ModelChoiceField(query_set=Tag.objects.get_content_tags()) + group = forms.ModelChoiceField(queryset=get_groups(), + required=False) + tag = forms.ModelChoiceField(queryset=Tag.objects.get_content_tags(), + required=False) class Meta: model = AskWidget diff --git a/askbot/skins/default/media/style/askbot-modal.css b/askbot/skins/default/media/style/askbot-modal.css deleted file mode 100755 index cec8bc6a..00000000 --- a/askbot/skins/default/media/style/askbot-modal.css +++ /dev/null @@ -1,25 +0,0 @@ -#AskbotAskWidget { - visibility: hidden; - position: absolute; - left: 0px; - top: 0px; - width:100%; - height:100%; - text-align:center; - z-index: 1000; -} - -#AskbotAskWidget div{ - width:600px; - height:400px; - margin: 100px auto; - background-color: #fff; - border:1px solid #000; - padding:15px; - text-align:right; -} - -#AskbotAskWidget iframe{ - width:600px; - height:400px; -} diff --git a/askbot/skins/default/templates/list_ask_widget.html b/askbot/skins/default/templates/list_ask_widget.html index d3ee6f3a..b895bde9 100644 --- a/askbot/skins/default/templates/list_ask_widget.html +++ b/askbot/skins/default/templates/list_ask_widget.html @@ -6,7 +6,7 @@ {%for widget in widgets%}

Widget: {{widget.title}} | Edit | Delete

-

Code: <script type="text/javascript" src=http://{{request.get_host()}}{% url render_ask_widget widget.id%} ><script> +

Code: <script type="text/javascript" src="http://{{request.get_host()}}{% url render_ask_widget widget.id%}" ></script> {%endfor%} diff --git a/askbot/skins/default/templates/widgets/askbot_widget.css b/askbot/skins/default/templates/widgets/askbot_widget.css new file mode 100755 index 00000000..573a07fd --- /dev/null +++ b/askbot/skins/default/templates/widgets/askbot_widget.css @@ -0,0 +1,25 @@ +#{{variable_name}} { + visibility: hidden; + position: absolute; + left: 0px; + top: 0px; + width:100%; + height:100%; + text-align:center; + z-index: 99999; +} + +#{{variable_name}} div{ + width:600px; + height:400px; + margin: 100px auto; + background-color: #fff; + border:1px solid #000; + padding:15px; + text-align:right; +} + +#{{variable_name}} iframe{ + width:600px; + height:400px; +} diff --git a/askbot/skins/default/templates/widgets/askbot_widget.js b/askbot/skins/default/templates/widgets/askbot_widget.js index 646d3dd5..57cead90 100755 --- a/askbot/skins/default/templates/widgets/askbot_widget.js +++ b/askbot/skins/default/templates/widgets/askbot_widget.js @@ -1,18 +1,20 @@ -var AskbotAskWidget = { - element_id: "AskbotAskWidget", +var {{variable_name}} = { + element_id: "{{variable_name}}", widgetToggle: function() { - element = document.getElementById(AskbotAskWidget.element_id); + element = document.getElementById({{variable_name}}.element_id); element.style.visibility = (element.style.visibility == "visible") ? "hidden" : "visible"; }, toHtml: function() { - var html = AskbotAskWidget.createButton(); + var html = {{variable_name}}.createButton(); var link = document.createElement('link'); link.setAttribute("rel", "stylesheet"); - link.setAttribute("href", 'http://{{host}}{{"/style/askbot-modal.css"|media}}'); + //link.setAttribute("href", 'http://{{host}}{{"/style/askbot-modal.css"|media}}'); + link.setAttribute("href", 'http://{{host}}{%url render_ask_widget_css widget.id%}'); //creating the div var motherDiv = document.createElement('div'); - motherDiv.setAttribute("id", AskbotAskWidget.element_id); + motherDiv.setAttribute("id", {{variable_name}}.element_id); + console.log(motherDiv); var containerDiv = document.createElement('div'); motherDiv.appendChild(containerDiv); @@ -26,7 +28,7 @@ var AskbotAskWidget = { var closeButton = document.createElement('a'); closeButton.setAttribute('href', '#'); closeButton.setAttribute('id', 'AskbotModalClose'); - closeButton.setAttribute('onClick', 'AskbotAskWidget.widgetToggle();'); + closeButton.setAttribute('onClick', '{{variable_name}}.widgetToggle();'); closeButton.innerText = 'Close'; containerDiv.appendChild(closeButton); @@ -38,6 +40,7 @@ var AskbotAskWidget = { var body = document.getElementsByTagName('body')[0]; if (body){ + console.log(body.firstChild); body.insertBefore(motherDiv, body.firstChild); body.insertBefore(link, body.firstChild); } @@ -48,7 +51,7 @@ var AskbotAskWidget = { buttonDiv.setAttribute('id', "AskbotAskButton"); var closeButton = document.createElement('button'); - closeButton.setAttribute('onClick', 'AskbotAskWidget.widgetToggle();'); + closeButton.setAttribute('onClick', '{{variable_name}}.widgetToggle();'); closeButton.innerText = label; buttonDiv.appendChild(closeButton); @@ -57,6 +60,14 @@ var AskbotAskWidget = { } }; +previous_function = window.onload; +var onload_functions = function(){ + if (previous_function){ + previous_function(); + } + {{variable_name}}.toHtml(); +} -window.onload = AskbotAskWidget.toHtml; -document.write(AskbotAskWidget.createButton().outerHTML); +console.log(onload_functions); +window.onload = onload_functions(); +document.write({{variable_name}}.createButton().outerHTML); diff --git a/askbot/urls.py b/askbot/urls.py index 8b0cb7cb..f6499845 100644 --- a/askbot/urls.py +++ b/askbot/urls.py @@ -397,6 +397,12 @@ urlpatterns = patterns('', views.widgets.render_ask_widget_js, name = 'render_ask_widget' ), + url( + r'^widgets/ask/(?P\d+).css$', + views.widgets.render_ask_widget_css, + name = 'render_ask_widget_css' + ), + url( r'^widgets/ask/complete/$', views.widgets.ask_widget_complete, diff --git a/askbot/views/widgets.py b/askbot/views/widgets.py index de1307ab..72aac74e 100644 --- a/askbot/views/widgets.py +++ b/askbot/views/widgets.py @@ -144,9 +144,22 @@ def delete_ask_widget(request, widget_id): #TODO: Add cache def render_ask_widget_js(request, widget_id): - widget = get_object_or_404(models.AskWidget) + widget = get_object_or_404(models.AskWidget, pk=widget_id) + variable_name = "AskbotAskWidget%d" % widget.id content_tpl = get_template('widgets/askbot_widget.js', request) - context_dict = {'widget': widget, 'host': request.get_host()} + context_dict = {'widget': widget, + 'host': request.get_host(), + 'variable_name': variable_name} content = content_tpl.render(Context(context_dict)) - print content return HttpResponse(content, mimetype='text/javascript') + +#TODO: Add cache +def render_ask_widget_css(request, widget_id): + widget = get_object_or_404(models.AskWidget, pk=widget_id) + variable_name = "AskbotAskWidget%d" % widget.id + content_tpl = get_template('widgets/askbot_widget.css', request) + context_dict = {'widget': widget, + 'host': request.get_host(), + 'variable_name': variable_name} + content = content_tpl.render(Context(context_dict)) + return HttpResponse(content, mimetype='text/css') -- cgit v1.2.3-1-g7c22