Indicate loading in progress in source view

Author: vincentvanbush

app/assets/stylesheets/lit/application.css

@@ -51,3 +51,9 @@ padding: 5px 7px;
 .localization_row em{
   color: #bbb;
 }
+.loading {
+  display: inline;
+}
+.loaded {
+  display: none;
+}

app/controllers/lit/sources_controller.rb

@@ -20,6 +20,7 @@ def edit
 
     def synchronize
       @source = Source.find(params[:id])
+      @source.update_column(:sync_complete, false)
       if defined?(ActiveJob)
         SynchronizeSourceJob.perform_later(@source)
       else

app/helpers/lit/sources_helper.rb

@@ -1,4 +1,7 @@
 module Lit
   module SourcesHelper
+    def source_loading_class(source)
+      source.sync_complete ? 'loaded' : 'loading'
+    end
   end
 end

app/models/lit/source.rb

@@ -54,6 +54,7 @@ def synchronize
           last_change = get_last_change
           last_change = DateTime.parse(last_change) unless last_change.nil?
           touch_last_updated_at(last_change)
+          update_column(:sync_complete, true)
           save
         end
       end

app/views/lit/incomming_localizations/index.html.erb

@@ -1,15 +1,19 @@
 <h1>Incomming localizations</h1>
-<div class="row">
-  <span class="pull-right">
-    <%= link_to accept_all_source_incomming_localizations_path(@source), :class=>"btn btn-success btn-sm", :data=>{:confirm=>t('lit.common.you_sure', :default=>"Are you sure?")} do %>
-      <%= t('lit.common.accept_all', :default=>"Accept all") %>
-    <% end %>
-    <%= link_to reject_all_source_incomming_localizations_path(@source), :class=>"btn btn-danger btn-sm", :method=>:post, :data=>{:confirm=>t('lit.common.you_sure', :default=>"Are you sure?")} do %>
-      <%= t('lit.common.reject_all', :default=>"Reject all") %>
-    <% end %>
-  </span>
+<div class="container">
+  <div class="row inline-headers">
+    <h4 class="<%= source_loading_class(@source) %>">Please wait...</h2>
+    <span class="pull-right">
+      <%= link_to accept_all_source_incomming_localizations_path(@source), :class=>"btn btn-success btn-sm", :data=>{:confirm=>t('lit.common.you_sure', :default=>"Are you sure?")} do %>
+        <%= t('lit.common.accept_all', :default=>"Accept all") %>
+      <% end %>
+      <%= link_to reject_all_source_incomming_localizations_path(@source), :class=>"btn btn-danger btn-sm", :method=>:post, :data=>{:confirm=>t('lit.common.you_sure', :default=>"Are you sure?")} do %>
+        <%= t('lit.common.reject_all', :default=>"Reject all") %>
+      <% end %>
+    </span>
+  </div>
 </div>
 
+
 <table class="table table-bordered table-striped incomming-localizations-table">
   <tr>
     <th>Current value</th>

db/migrate/20160510125412_add_sync_complete_to_lit_sources.rb

@@ -0,0 +1,5 @@
+class AddSyncCompleteToLitSources < ActiveRecord::Migration
+  def change
+    add_column :lit_sources, :sync_complete, :boolean
+  end
+end