Replace inline document.ready with dispatcher

Dmitriy Zaporozhets
1 parent 4f5aae1d
Showing 15 changed files with 27 additions and 28 deletions Show diff stats
app/assets/javascripts/dispatcher.js.coffee
app/helpers/application_helper.rb
app/views/dashboard/show.html.haml
app/views/groups/show.html.haml
app/views/issues/index.html.haml
app/views/layouts/admin.html.haml
app/views/layouts/application.html.haml
app/views/layouts/group.html.haml
app/views/layouts/profile.html.haml
app/views/layouts/project_resource.html.haml
app/views/layouts/user_team.html.haml
app/views/projects/edit.html.haml
app/views/projects/new.html.haml
app/views/projects/show.html.haml
app/views/teams/show.html.haml
@@ -0,0 +1,17 @@
+$ ->
+  new Dispatcher()
+  
+class Dispatcher
+  constructor: () ->
+    page = $('body').attr('data-page')
+
+    console.log(page)
+
+    switch page
+      when 'issues:index' then Issues.init()
+      when 'dashboard:show' then dashboardPage()
+      when 'groups:show' then Pager.init(20, true)
+      when 'teams:show' then Pager.init(20, true)
+      when 'projects:show' then Pager.init(20, true)
+      when 'projects:new' then new Projects()
+      when 'projects:edit' then new Projects()
@@ -189,4 +189,8 @@ module ApplicationHelper
     css_class << " multiselect" if opts[:multiple]
     hidden_field_tag(id, '', class: css_class)
   end
+
+  def body_data_page
+    controller.controller_name + ":" + controller.action_name
+  end
 end
@@ -7,6 +7,3 @@
 - else
   = render "zero_authorized_projects"
-
-:javascript
-  dashboardPage();
@@ -24,6 +24,3 @@
     %hr
     = render 'shared/promo'
-
-:javascript
-  $(function(){ Pager.init(20, true); });
@@ -23,8 +23,3 @@
     = render 'filter', entity: 'issue'
   .span9.issues-holder
     = render "issues"
-
-:javascript
-  $(function(){
-    Issues.init();
-  })
 !!! 5
 %html{ lang: "en"}
   = render "layouts/head", title: "Admin area"
-  %body{class: "#{app_theme} admin"}
+  %body{class: "#{app_theme} admin", :'data-page' => body_data_page}
     = render "layouts/head_panel", title: "Admin area"
     = render "layouts/flash"
     %nav.main-nav
 !!! 5
 %html{ lang: "en"}
   = render "layouts/head", title: "Dashboard"
-  %body{class: "#{app_theme} application"}
+  %body{class: "#{app_theme} application", :'data-page' => body_data_page }
     = render "layouts/head_panel", title: "Dashboard"
     = render "layouts/flash"
     %nav.main-nav
 !!! 5
 %html{ lang: "en"}
   = render "layouts/head", title: "#{@group.name}"
-  %body{class: "#{app_theme} application"}
+  %body{class: "#{app_theme} application", :'data-page' => body_data_page}
     = render "layouts/head_panel", title: "group: #{@group.name}"
     = render "layouts/flash"
     %nav.main-nav
 !!! 5
 %html{ lang: "en"}
   = render "layouts/head", title: "Profile"
-  %body{class: "#{app_theme} profile"}
+  %body{class: "#{app_theme} profile", :'data-page' => body_data_page}
     = render "layouts/head_panel", title: "Profile"
     = render "layouts/flash"
     %nav.main-nav
 !!! 5
 %html{ lang: "en"}
   = render "layouts/head", title: @project.name_with_namespace
-  %body{class: "#{app_theme} project"}
+  %body{class: "#{app_theme} project", :'data-page' => body_data_page}
     = render "layouts/head_panel", title: project_title(@project)
     = render "layouts/flash"
     - if can?(current_user, :download_code, @project)
 !!! 5
 %html{ lang: "en"}
   = render "layouts/head", title: "#{@team.name}"
-  %body{class: "#{app_theme} application"}
+  %body{class: "#{app_theme} application", :'data-page' => body_data_page}
     = render "layouts/head_panel", title: "team: #{@team.name}"
     = render "layouts/flash"
     %nav.main-nav
@@ -8,6 +8,3 @@
   %center
     = image_tag "ajax_loader.gif"
     %h3 Saving project. Please wait a moment, this page will automatically refresh when ready.
-
-:javascript
-  $(function(){ new Projects(); });
@@ -7,6 +7,3 @@
   %center
     = image_tag "ajax_loader.gif"
     %h3 Creating project &amp; repository. Please wait a moment, this page will automatically refresh when ready.
-
-:javascript
-  $(function(){ new Projects(); });
@@ -17,5 +17,3 @@
       %p Repo Size: #{@project.repository.size} MB
       %p Created at: #{@project.created_at.stamp('Aug 22, 2013')}
       %p Owner: #{link_to @project.owner_name, @project.owner}
-:javascript
-  $(function(){ Pager.init(20); });
@@ -23,6 +23,3 @@
     %hr
     = render 'shared/promo'
-
-:javascript
-  $(function(){ Pager.init(20, true); });
@@ -0,0 +1,17 @@		@@ -0,0 +1,17 @@
	1	+$ ->
	2	+ new Dispatcher()
	3	+
	4	+class Dispatcher
	5	+ constructor: () ->
	6	+ page = $('body').attr('data-page')
	7	+
	8	+ console.log(page)
	9	+
	10	+ switch page
	11	+ when 'issues:index' then Issues.init()
	12	+ when 'dashboard:show' then dashboardPage()
	13	+ when 'groups:show' then Pager.init(20, true)
	14	+ when 'teams:show' then Pager.init(20, true)
	15	+ when 'projects:show' then Pager.init(20, true)
	16	+ when 'projects:new' then new Projects()
	17	+ when 'projects:edit' then new Projects()
	@@ -189,4 +189,8 @@ module ApplicationHelper		@@ -189,4 +189,8 @@ module ApplicationHelper
189	css_class << " multiselect" if opts[:multiple]	189	css_class << " multiselect" if opts[:multiple]
190	hidden_field_tag(id, '', class: css_class)	190	hidden_field_tag(id, '', class: css_class)
191	end	191	end
		192	+
		193	+ def body_data_page
		194	+ controller.controller_name + ":" + controller.action_name
		195	+ end
192	end	196	end
	@@ -7,6 +7,3 @@		@@ -7,6 +7,3 @@
7		7
8	- else	8	- else
9	= render "zero_authorized_projects"	9	= render "zero_authorized_projects"
10	-
11	-:javascript
12	- dashboardPage();
	@@ -24,6 +24,3 @@		@@ -24,6 +24,3 @@
24		24
25	%hr	25	%hr
26	= render 'shared/promo'	26	= render 'shared/promo'
27	-
28	-:javascript
29	- $(function(){ Pager.init(20, true); });
	@@ -23,8 +23,3 @@		@@ -23,8 +23,3 @@
23	= render 'filter', entity: 'issue'	23	= render 'filter', entity: 'issue'
24	.span9.issues-holder	24	.span9.issues-holder
25	= render "issues"	25	= render "issues"
26	-
27	-:javascript
28	- $(function(){
29	- Issues.init();
30	- })
1	!!! 5	1	!!! 5
2	%html{ lang: "en"}	2	%html{ lang: "en"}
3	= render "layouts/head", title: "Admin area"	3	= render "layouts/head", title: "Admin area"
4	- %body{class: "#{app_theme} admin"}	4	+ %body{class: "#{app_theme} admin", :'data-page' => body_data_page}
5	= render "layouts/head_panel", title: "Admin area"	5	= render "layouts/head_panel", title: "Admin area"
6	= render "layouts/flash"	6	= render "layouts/flash"
7	%nav.main-nav	7	%nav.main-nav
	@@ -8,6 +8,3 @@		@@ -8,6 +8,3 @@
8	%center	8	%center
9	= image_tag "ajax_loader.gif"	9	= image_tag "ajax_loader.gif"
10	%h3 Saving project. Please wait a moment, this page will automatically refresh when ready.	10	%h3 Saving project. Please wait a moment, this page will automatically refresh when ready.
11	-
12	-:javascript
13	- $(function(){ new Projects(); });
	@@ -17,5 +17,3 @@		@@ -17,5 +17,3 @@
17	%p Repo Size: #{@project.repository.size} MB	17	%p Repo Size: #{@project.repository.size} MB
18	%p Created at: #{@project.created_at.stamp('Aug 22, 2013')}	18	%p Created at: #{@project.created_at.stamp('Aug 22, 2013')}
19	%p Owner: #{link_to @project.owner_name, @project.owner}	19	%p Owner: #{link_to @project.owner_name, @project.owner}
20	-:javascript
21	- $(function(){ Pager.init(20); });
	@@ -23,6 +23,3 @@		@@ -23,6 +23,3 @@
23		23
24	%hr	24	%hr
25	= render 'shared/promo'	25	= render 'shared/promo'
26	-
27	-:javascript
28	- $(function(){ Pager.init(20, true); });