upjs-rails 0.2.1 → 0.2.2

data/lib/assets/stylesheets/up/tooltip.css.sass

@@ -1,17 +1,42 @@
 $stratum: 30000
 $distance-from-trigger: 6px
+$corner-size: 8px
+$background: #666
 
 .up-tooltip
   z-index: $stratum
   position: fixed
-  background-color: rgba(30, 30, 30, 0.8)
+  background-color: $background
   color: white
-  padding: 6px 8px
-  box-shadow: 0 0 4px rgba(0, 0, 0, 0.2)
+  padding: 6px 9px
+  //box-shadow: 0 0 4px rgba(0, 0, 0, 0.2)
   
   &[up-origin=top]
     margin-top: -$distance-from-trigger
+    &:after
+      content: ''
+      position: absolute
+      border-style: solid
+      border-width: $corner-size $corner-size 0
+      border-color: $background transparent
+      display: block
+      width: 0
+      z-index: 1
+      bottom: -$corner-size
+      left: 50%
+      margin-left: -$corner-size
   
   &[up-origin=bottom]
-    margin-bottom: -$distance-from-trigger
-    
+    margin-top: $distance-from-trigger
+    &:after
+      content: ''
+      position: absolute
+      border-style: solid
+      border-width: 0 $corner-size $corner-size
+      border-color: $background transparent
+      display: block
+      width: 0
+      z-index: 1
+      top: -$corner-size
+      left: 50%
+      margin-left: -$corner-size

data/lib/upjs/rails/current_location.rb

@@ -0,0 +1,19 @@
+module Upjs
+  module Rails
+    module CurrentLocation
+
+      def self.included(base)
+        base.before_filter :set_header_for_current_location
+      end
+
+      private
+      
+      def set_header_for_current_location
+        headers['X-Up-Current-Location'] = request.fullpath
+      end
+
+      ActionController::Base.include(self)
+
+    end
+  end
+end

data/lib/upjs/rails/version.rb

@@ -1,5 +1,5 @@
 module Upjs
   module Rails
-    VERSION = "0.2.1"
+    VERSION = "0.2.2"
   end
 end

data/lib/upjs-rails.rb

@@ -1,7 +1,4 @@
 require "upjs/rails/version"
 require "upjs/rails/engine"
-require "upjs/rails/redirection"
+require "upjs/rails/current_location"
 require "upjs/rails/request"
-
-
-

data/spec_app/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    upjs-rails (0.1.0)
+    upjs-rails (0.2.1)
       rails (>= 3)
 
 GEM
@@ -105,32 +105,32 @@ GEM
       hpricot (~> 0.8.6)
       ruby_parser (~> 3.1.1)
     i18n (0.7.0)
-    jasmine-core (2.1.3)
-    jasmine-rails (0.10.6)
+    jasmine-core (2.2.0)
+    jasmine-rails (0.10.7)
       jasmine-core (>= 1.3, < 3.0)
-      phantomjs
+      phantomjs (< 2.0)
       railties (>= 3.1.0)
       sprockets-rails
     jquery-rails (4.0.2)
       rails-dom-testing (~> 1.0)
       railties (>= 4.2.0)
       thor (>= 0.14, < 2.0)
-    json (1.8.1)
+    json (1.8.2)
     libv8 (3.16.14.3)
     loofah (2.0.1)
       nokogiri (>= 1.5.9)
     mail (2.6.3)
       mime-types (>= 1.16, < 3)
     mime-types (2.4.3)
-    mini_portile (0.6.1)
-    minitest (5.5.0)
+    mini_portile (0.6.2)
+    minitest (5.5.1)
     multi_json (1.10.1)
     multi_test (0.1.1)
-    nokogiri (1.6.5)
+    nokogiri (1.6.6.2)
       mini_portile (~> 0.6.0)
-    phantomjs (1.9.7.1)
+    phantomjs (1.9.8.0)
     rack (1.6.0)
-    rack-test (0.6.2)
+    rack-test (0.6.3)
       rack (>= 1.0)
     rails (4.2.0)
       actionmailer (= 4.2.0)
@@ -200,7 +200,7 @@ GEM
       multi_json (~> 1.0)
       rack (~> 1.0)
       tilt (~> 1.1, != 1.3.0)
-    sprockets-rails (2.2.2)
+    sprockets-rails (2.2.4)
       actionpack (>= 3.0)
       activesupport (>= 3.0)
       sprockets (>= 2.8, < 4.0)

data/spec_app/app/assets/stylesheets/application.css

@@ -10,6 +10,8 @@
  * defined in the other CSS/SCSS files in this directory. It is generally better to create a new
  * file per style scope.
  *
+ *= require up
  *= require_tree .
  *= require_self
  */
+

data/spec_app/app/assets/stylesheets/jasmine_specs.css

@@ -0,0 +1,5 @@
+/*
+ *= require up
+ *= require_self
+ */
+

data/spec_app/app/views/pages/home.html.haml

@@ -1,5 +1,12 @@
 .menu
+  
+  %p
+    fullpath:
+    = request.fullpath
+  
+  %p
+    url:
+    = request.url 
 
   = link_to 'Cards panel', cards_path, class: 'menu__item'
-  = link_to 'Modal test', modal_test_path, class: 'menu__item'
   = link_to 'Another screen', '#', class: 'menu__item'

data/spec_app/spec/javascripts/support/jasmine.yml

@@ -22,6 +22,7 @@ src_files:
 # list of file expressions to include as css files
 # relative path from css_dir
 css_files:
+ - "application.{css.sass,css,sass}"
 
 # path to parent directory of spec_files
 # relative path from Rails.root

data/spec_app/spec/javascripts/up/bus_spec.js.coffee

@@ -0,0 +1,12 @@
+describe 'up.bus', ->
+  
+  describe 'Javascript functions', ->
+    
+    describe 'up.bus.on', ->
+
+      it 'should have tests'
+    
+    describe 'up.bus.emit', ->
+
+      it 'should have tests'
+      

data/spec_app/spec/javascripts/up/flow_spec.js.coffee

@@ -1,75 +1,99 @@
 describe 'up.flow', ->
   
-  describe '.replace', ->
-    
-    beforeEach ->
-      jasmine.Ajax.install()
-
-      affix('.before').text('old-before')
-      affix('.middle').text('old-middle')
-      affix('.after').text('old-after')
-
-      @promise = up.replace('.middle', '/path')
-
-      jasmine.Ajax.requests.mostRecent().respondWith
-        status: 200
-        contentType: '/text/html'
-        responseText:
-          """
-          <div class="before">new-before</div>
-          <div class="middle">new-middle</div>
-          <div class="after">new-after</div>
-          """      
-    
-    it 'replaces the given selector with the same selector from a freshly fetched page', (done) ->
-      @promise.then ->
-        expect($('.before')).toHaveText('old-before')
-        expect($('.middle')).toHaveText('new-middle')
-        expect($('.after')).toHaveText('old-after')
-        done()      
+  describe 'Javascript functions', -> 
+  
+    describe 'up.replace', ->
+      
+      beforeEach ->
+        jasmine.Ajax.install()
+  
+        affix('.before').text('old-before')
+        affix('.middle').text('old-middle')
+        affix('.after').text('old-after')
+  
+        @respond = ->
+          jasmine.Ajax.requests.mostRecent().respondWith
+            status: 200
+            contentType: 'text/html'
+            responseText:
+              """
+              <div class="before">new-before</div>
+              <div class="middle">new-middle</div>
+              <div class="after">new-after</div>
+              """      
       
-    it 'should set the browser location to the given URL', (done) ->
-      @promise.then ->
-        expect(window.location.pathname).toBe('/path')
-        done()
+      it 'replaces the given selector with the same selector from a freshly fetched page', (done) ->
+        @request = up.replace('.middle', '/path')
+        @respond()
+        @request.then ->
+          expect($('.before')).toHaveText('old-before')
+          expect($('.middle')).toHaveText('new-middle')
+          expect($('.after')).toHaveText('old-after')
+          done()      
         
-    it 'marks the element with the URL from which it was retrieved', (done) ->
-      @promise.then ->
-        expect($('.middle').attr('up-source')).toMatch(/\/path$/)
-        done()
-
-  describe '.destroy', ->
-    
-    it 'removes the element with the given selector', ->
-      affix('.element')
-      up.destroy('.element')
-      expect($('.element')).not.toExist()
+      it 'should set the browser location to the given URL', (done) ->
+        @request = up.replace('.middle', '/path')
+        @respond()
+        @request.then ->
+          expect(window.location.pathname).toBe('/path')
+          done()
+          
+      it 'marks the element with the URL from which it was retrieved', (done) ->
+        @request = up.replace('.middle', '/path')
+        @respond()
+        @request.then ->
+          expect($('.middle').attr('up-source')).toMatch(/\/path$/)
+          done()
+          
+      it 'replaces multiple selectors separated with a comma', (done) ->
+        @request = up.replace('.middle, .after', '/path')
+        @respond()
+        @request.then ->
+          expect($('.before')).toHaveText('old-before')
+          expect($('.middle')).toHaveText('new-middle')
+          expect($('.after')).toHaveText('new-after')
+          done()
+          
+    describe 'up.implant', ->
+      
+      it 'should have tests'
       
-    it 'calls destructors for custom elements', ->
-      up.awaken('.element', ($element) -> destructor)
-      destructor = jasmine.createSpy('destructor')
-      up.ready(affix('.element'))
-      up.destroy('.element')
-      expect(destructor).toHaveBeenCalled()
+    describe 'up.destroy', ->
       
-  describe '.reload', ->
-    
-    it 'reloads the given selector from the closest known source URL', (done) ->
-      affix('.container[up-source="/source"] .element').find('.element').text('old text')
-
-      up.reload('.element').then ->
-        expect($('.element')).toHaveText('new text')
-        done()
+      it 'removes the element with the given selector', ->
+        affix('.element')
+        up.destroy('.element')
+        expect($('.element')).not.toExist()
+        
+      it 'calls destructors for custom elements', ->
+        up.awaken('.element', ($element) -> destructor)
+        destructor = jasmine.createSpy('destructor')
+        up.ready(affix('.element'))
+        up.destroy('.element')
+        expect(destructor).toHaveBeenCalled()
         
-      request = jasmine.Ajax.requests.mostRecent()
-      expect(request.url).toMatch(/\/source$/)
-
-      request.respondWith
-        status: 200
-        contentType: '/text/html'
-        responseText:
-          """
-          <div class="container">
-            <div class="element">new text</div>
-          </div>
-          """
+    describe 'up.reload', ->
+      
+      it 'reloads the given selector from the closest known source URL', (done) ->
+        affix('.container[up-source="/source"] .element').find('.element').text('old text')
+  
+        up.reload('.element').then ->
+          expect($('.element')).toHaveText('new text')
+          done()
+          
+        request = jasmine.Ajax.requests.mostRecent()
+        expect(request.url).toMatch(/\/source$/)
+  
+        request.respondWith
+          status: 200
+          contentType: '/text/html'
+          responseText:
+            """
+            <div class="container">
+              <div class="element">new text</div>
+            </div>
+            """
+  
+    describe 'up.reset', ->
+  
+      it 'should have tests'

data/spec_app/spec/javascripts/up/form_spec.js.coffee

@@ -0,0 +1,100 @@
+describe 'up.form', ->
+  
+  describe 'Javascript functions', ->
+    
+    describe 'up.observe', ->
+
+      it 'should have tests'
+
+    describe 'up.submit', ->
+      
+      beforeEach ->
+        jasmine.Ajax.install()
+  
+        $form = affix('form[action="/path/to"][method="put"][up-target=".response"]')
+        $form.append('<input name="field1" value="value1">')
+        $form.append('<input name="field2" value="value2">')
+  
+        affix('.response').text('old-text')
+  
+        @promise = up.submit($form)
+  
+        @request = jasmine.Ajax.requests.mostRecent()
+        expect(@request.url).toMatch /\/path\/to$/
+        expect(@request.method).toBe 'PUT'
+        expect(@request.data()).toEqual
+          field1: ['value1']
+          field2: ['value2']
+      
+      it 'submits the given form and replaces the target with the response', (done) ->
+  
+        @request.respondWith
+          status: 200
+          contentType: 'text/html'
+          responseText:
+            """
+            text-before
+  
+            <div class="response">
+              new-text
+            </div>
+  
+            text-after
+            """
+  
+        @promise.then ->
+          expect($('.response')).toHaveText('new-text')
+          expect($('body')).not.toHaveText('text-before')
+          expect($('body')).not.toHaveText('text-after')
+          done()
+      
+      it 'places the response into the form if the submission returns a 5xx status code', (done) ->
+        @request.respondWith
+          status: 500
+          contentType: 'text/html'
+          responseText:
+            """
+            text-before
+  
+            <form>
+              error-messages
+            </form>
+  
+            text-after
+            """
+  
+        @promise.always ->
+          expect($('.response')).toHaveText('old-text')
+          expect($('form')).toHaveText('error-messages')
+          expect($('body')).not.toHaveText('text-before')
+          expect($('body')).not.toHaveText('text-after')
+          done()
+      
+      it 'respects a X-Up-Current-Location header that the server sends in case of a redirect', (done) ->
+  
+        @request.respondWith
+          status: 200
+          contentType: 'text/html'
+          responseHeaders: { 'X-Up-Current-Location': '/other/path' }
+          responseText:
+            """
+            <div class="response">
+              new-text
+            </div>
+            """
+  
+        @promise.then ->
+          expect(up.browser.url()).toMatch(/\/other\/path$/)
+          done()
+
+  describe 'unobtrusive behavior', ->
+    
+    describe 'form[up-target]', ->
+
+      it 'should have tests'
+      
+    describe 'input[up-observe]', ->
+
+      it 'should have tests'
+      
+     

data/spec_app/spec/javascripts/up/history_spec.js.coffee

@@ -0,0 +1,12 @@
+describe 'up.history', ->
+  
+  describe 'Javascript functions', ->
+    
+    describe 'up.history.replace', ->
+
+      it 'should have tests'
+      
+    describe 'up.history.push', ->
+
+      it 'should have tests'
+      

data/spec_app/spec/javascripts/up/link_spec.js.coffee

@@ -0,0 +1,47 @@
+describe 'up.link', ->
+  
+  describe 'Javascript functions', ->
+  
+    describe 'up.follow', ->
+      
+      it 'loads the given link via AJAX and replaces the response in the given target', (done) ->
+        jasmine.Ajax.install()
+  
+        affix('.before').text('old-before')
+        affix('.middle').text('old-middle')
+        affix('.after').text('old-after')
+        $link = affix('a[href="/path"][up-target=".middle"]')
+  
+        promise = up.follow($link)
+  
+        jasmine.Ajax.requests.mostRecent().respondWith
+          status: 200
+          contentType: 'text/html'
+          responseText:
+            """
+            <div class="before">new-before</div>
+            <div class="middle">new-middle</div>
+            <div class="after">new-after</div>
+            """      
+        
+        promise.then ->
+          expect($('.before')).toHaveText('old-before')
+          expect($('.middle')).toHaveText('new-middle')
+          expect($('.after')).toHaveText('old-after')
+          done()
+
+    describe 'up.visit', ->
+      
+      it 'should have tests'
+          
+  describe 'unobtrusive behavior', ->
+
+    describe 'a[up-target]', ->
+
+      it 'should have tests'
+
+    describe '[up-follow]', ->
+
+      it 'should have tests'
+      
+      

data/spec_app/spec/javascripts/up/magic_spec.js.coffee

@@ -0,0 +1,49 @@
+describe 'up.magic', ->
+  
+  describe 'Javascript functions', ->
+  
+    describe 'up.on', ->
+      
+      it 'registers a delagating event listener to the document body', ->
+        
+        affix('.container .child')
+        observeClass = jasmine.createSpy()
+        up.on 'click', '.child', (event, $element) ->
+          observeClass($element.attr('class'))
+   
+        $('.container').click()
+        $('.child').click()
+   
+        expect(observeClass).not.toHaveBeenCalledWith('container')
+        expect(observeClass).toHaveBeenCalledWith('child')           
+      
+    describe 'up.awaken', ->
+      
+      it 'applies an event initializer whenever a matching fragment is inserted', ->
+  
+        observeClass = jasmine.createSpy()
+        up.awaken '.child', ($element) ->
+          observeClass($element.attr('class'))
+  
+        up.ready(affix('.container .child'))
+   
+        expect(observeClass).not.toHaveBeenCalledWith('container')
+        expect(observeClass).toHaveBeenCalledWith('child')           
+  
+      it 'lets allows initializers return a destructor function, which is called when the awakened fragments gets destroyed', ->
+  
+        destructor = jasmine.createSpy()
+        up.awaken '.child', ($element) ->
+          destructor
+  
+        up.ready(affix('.container .child'))
+        expect(destructor).not.toHaveBeenCalled()
+        
+        up.destroy('.container')
+        expect(destructor).toHaveBeenCalled()
+            
+    describe 'up.ready', ->
+
+      it 'should have tests'
+      
+     

data/spec_app/spec/javascripts/up/marker_spec.js.coffee

@@ -1,14 +1,16 @@
 describe 'up.marker', ->
   
-  describe '[up-marker]', ->
-    
-    it 'hides empty containers', ->
-      $element = affix('.element[up-marker]')
-      up.ready($element)
-      expect($element).not.toBeVisible()
-    
-    it 'does not hide empty containers', ->
-      $element = affix('.element[up-marker]').text('content')
-      up.ready($element)
-      expect($element).toBeVisible()
-    
+  describe 'unobtrusive behavior', ->
+  
+    describe '[up-marker]', ->
+      
+      it 'hides empty containers', ->
+        $element = affix('.element[up-marker]')
+        up.ready($element)
+        expect($element).not.toBeVisible()
+      
+      it 'does not hide empty containers', ->
+        $element = affix('.element[up-marker]').text('content')
+        up.ready($element)
+        expect($element).toBeVisible()
+      

data/spec_app/spec/javascripts/up/modal_spec.js.coffee

@@ -0,0 +1,30 @@
+describe 'up.modal', ->
+  
+  describe 'Javascript functions', ->
+    
+    describe 'up.modal.defaults', ->
+      
+      it 'should have tests'
+        
+    describe 'up.modal.open', ->
+
+      it 'should have tests'
+
+    describe 'up.modal.close', ->
+
+      it 'should have tests'
+
+    describe 'up.modal.source', ->
+
+      it 'should have tests'
+
+  describe 'unobtrusive behavior', ->
+    
+    describe 'a[up-modal]', ->
+      
+      it 'should have tests'
+      
+    describe '[up-close]', ->
+      
+      it 'should have tests'
+      

data/spec_app/spec/javascripts/up/motion_spec.js.coffee

@@ -0,0 +1,25 @@
+describe 'up.motion', ->
+  
+  describe 'Javascript functions', ->
+  
+    describe 'up.animate', ->
+
+      it 'should have tests'
+      
+    describe 'up.morph', ->
+
+      it 'should have tests'
+      
+    describe 'up.transition', ->
+
+      it 'should have tests'
+      
+    describe 'up.animation', ->
+
+      it 'should have tests'
+      
+    describe 'up.motion.none', ->
+
+      it 'should have tests'
+
+