unpoly-rails 0.55.1 → 0.56.0

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

@@ -16,7 +16,7 @@ describe 'up.popup', ->
                 Math.abs(popupDims.right - linkDims.right) < 1.0 && Math.abs(popupDims.top - linkDims.bottom) < 1.0
 
       beforeEach ->
-        @restoreBodyHeight = u.temporaryCss('body', 'min-height': '3000px')
+        @restoreBodyHeight = u.writeTemporaryStyle('body', minHeight: '3000px')
 
       afterEach ->
         @restoreBodyHeight()

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

@@ -13,7 +13,7 @@ describe 'up.syntax', ->
         up.hello(affix('.container .child'))
    
         expect(observeClass).not.toHaveBeenCalledWith('container')
-        expect(observeClass).toHaveBeenCalledWith('child')           
+        expect(observeClass).toHaveBeenCalledWith('child')
 
       describe 'destructors', ->
 

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

@@ -40,7 +40,7 @@ describe 'up.tooltip', ->
           )
 
         beforeEach ->
-          @restoreBodyHeight = u.temporaryCss('body', 'min-height': '3000px')
+          @restoreBodyHeight = u.writeTemporaryStyle('body', minHeight: '3000px')
 
         afterEach ->
           @restoreBodyHeight()

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

@@ -135,6 +135,82 @@ describe 'up.util', ->
             expect(callback).toHaveBeenCalledWith('return value')
             done()
 
+    describe 'up.util.kebabCase', ->
+
+      it 'converts a string of multiple words from camel-case to kebap-case', ->
+        result = up.util.kebabCase('fooBarBaz')
+        expect(result).toEqual('foo-bar-baz')
+
+      it 'does not change a single word', ->
+        result = up.util.kebabCase('foo')
+        expect(result).toEqual('foo')
+
+      it 'downcases the first word when it starts with a capital letter', ->
+        result = up.util.kebabCase('FooBar')
+        expect(result).toEqual('foo-bar')
+
+      it 'does not change a string that is already in kebab-case', ->
+        result = up.util.kebabCase('foo-bar-baz')
+        expect(result).toEqual('foo-bar-baz')
+
+    describe 'up.util.camelCase', ->
+
+      it 'converts a string of multiple words from kebap-case to camel-case', ->
+        result = up.util.camelCase('foo-bar-baz')
+        expect(result).toEqual('fooBarBaz')
+
+      it 'does not change a single word', ->
+        result = up.util.camelCase('foo')
+        expect(result).toEqual('foo')
+
+      it 'downcases the first word when it starts with a capital letter', ->
+        result = up.util.camelCase('Foo-Bar')
+        expect(result).toEqual('fooBar')
+
+      it 'does not change a string that is already in camel-case', ->
+        result = up.util.camelCase('fooBarBaz')
+        expect(result).toEqual('fooBarBaz')
+
+    describe 'up.util.kebabCaseKeys', ->
+
+      it "converts the given object's keys from camel-case to kebab-case", ->
+        input =
+          fooBar: 'one'
+          barBaz: 'two'
+        result = up.util.kebabCaseKeys(input)
+        expect(result).toEqual
+          'foo-bar': 'one'
+          'bar-baz': 'two'
+
+      it "does not change an object whose keys are already kebab-case", ->
+        input =
+          'foo-bar': 'one'
+          'bar-baz': 'two'
+        result = up.util.kebabCaseKeys(input)
+        expect(result).toEqual
+          'foo-bar': 'one'
+          'bar-baz': 'two'
+
+    describe 'up.util.camelCaseKeys', ->
+
+      it "converts the given object's keys from kebab-case to camel-case", ->
+        input =
+          'foo-bar': 'one'
+          'bar-baz': 'two'
+        result = up.util.camelCaseKeys(input)
+        expect(result).toEqual
+          fooBar: 'one'
+          barBaz: 'two'
+
+      it "does not change an object whose keys are already camel-case", ->
+        input =
+          fooBar: 'one'
+          barBaz: 'two'
+        result = up.util.camelCaseKeys(input)
+        expect(result).toEqual
+          fooBar: 'one'
+          barBaz: 'two'
+
     describe 'up.util.DivertibleChain', ->
 
       it "instantiates a task queue whose (2..n)th tasks can be changed by calling '.asap'", (done) ->
@@ -351,6 +427,98 @@ describe 'up.util', ->
           baz: 'baz-value'
           bam: 'bam-value'
 
+      it 'does not add empty keys to the returned object if the given object does not have that key', ->
+        original =
+          foo: 'foo-value'
+        whitelisted = up.util.only(original, 'foo', 'bar')
+        expect(whitelisted).toHaveOwnProperty('foo')
+        expect(whitelisted).not.toHaveOwnProperty('bar')
+
+    describe 'up.util.readInlineStyle', ->
+      
+      describe 'with a string as second argument', ->
+
+        it 'returns a CSS value string from an inline [style] attribute', ->
+          $div = affix('div').attr('style', 'background-color: #ff0000')
+          style = up.util.readInlineStyle($div, 'backgroundColor')
+          # Browsers convert colors to rgb() values, even IE11
+          expect(style).toEqual('rgb(255, 0, 0)')
+
+        it 'returns a blank value if the element does not have the given property in the [style] attribute', ->
+          $div = affix('div').attr('style', 'background-color: red')
+          style = up.util.readInlineStyle($div, 'color')
+          expect(style).toBeBlank()
+
+        it 'returns a blank value the given property is a computed property, but not in the [style] attribute', ->
+          $div = affix('div[class="red-background"]')
+          inlineStyle = up.util.readInlineStyle($div, 'backgroundColor')
+          computedStyle = up.util.readComputedStyle($div, 'backgroundColor')
+          expect(computedStyle).toEqual('rgb(255, 0, 0)')
+          expect(inlineStyle).toBeBlank()
+
+      describe 'with an array as second argument', ->
+
+        it 'returns an object with the given inline [style] properties', ->
+          $div = affix('div').attr('style', 'background-color: #ff0000; color: #0000ff')
+          style = up.util.readInlineStyle($div, ['backgroundColor', 'color'])
+          expect(style).toEqual
+            backgroundColor: 'rgb(255, 0, 0)'
+            color: 'rgb(0, 0, 255)'
+
+        it 'returns blank keys if the element does not have the given property in the [style] attribute', ->
+          $div = affix('div').attr('style', 'background-color: #ff0000')
+          style = up.util.readInlineStyle($div, ['backgroundColor', 'color'])
+          expect(style).toHaveOwnProperty('color')
+          expect(style.color).toBeBlank()
+
+        it 'returns a blank value the given property is a computed property, but not in the [style] attribute', ->
+          $div = affix('div[class="red-background"]')
+          inlineStyleHash = up.util.readInlineStyle($div, ['backgroundColor'])
+          computedBackground = up.util.readComputedStyle($div, 'backgroundColor')
+          expect(computedBackground).toEqual('rgb(255, 0, 0)')
+          expect(inlineStyleHash).toHaveOwnProperty('backgroundColor')
+          expect(inlineStyleHash.backgroundColor).toBeBlank()
+
+    describe 'up.util.writeInlineStyle', ->
+
+      it "sets the given style properties as the given element's [style] attribute", ->
+        $div = affix('div')
+        up.util.writeInlineStyle($div, { color: 'red', backgroundColor: 'blue' })
+        style = $div.attr('style')
+        expect(style).toContain('color: red')
+        expect(style).toContain('background-color: blue')
+
+      it "merges the given style properties into the given element's existing [style] value", ->
+        $div = affix('div[style="color: red"]')
+        up.util.writeInlineStyle($div, { backgroundColor: 'blue' })
+        style = $div.attr('style')
+        expect(style).toContain('color: red')
+        expect(style).toContain('background-color: blue')
+
+      it "converts the values of known length properties to px values automatically", ->
+        $div = affix('div')
+        up.util.writeInlineStyle($div, { paddingTop: 100 })
+        style = $div.attr('style')
+        expect(style).toContain('padding-top: 100px')
+
+    describe 'up.util.writeTemporaryStyle', ->
+
+      it "sets the given inline styles and returns a function that will restore the previous inline styles", ->
+        $div = affix('div[style="color: red"]')
+        restore = up.util.writeTemporaryStyle($div, { color: 'blue' })
+        expect($div.attr('style')).toContain('color: blue')
+        expect($div.attr('style')).not.toContain('color: red')
+        restore()
+        expect($div.attr('style')).not.toContain('color: blue')
+        expect($div.attr('style')).toContain('color: red')
+
+      it "does not restore inherited styles", ->
+        $div = affix('div[class="red-background"]')
+        restore = up.util.writeTemporaryStyle($div, { backgroundColor: 'blue' })
+        expect($div.attr('style')).toContain('background-color: blue')
+        restore()
+        expect($div.attr('style')).not.toContain('background-color')
+
     describe 'up.util.except', ->
 
       it 'returns a copy of the given object but omits the given blacklisted properties', ->
@@ -418,6 +586,29 @@ describe 'up.util', ->
         expect(up.util.selectorForElement($element)).toBe('[aria-label="foo\\"bar"]')
 
 
+    describe 'up.util.addTemporaryClass', ->
+
+      it 'adds the given class to the given element', ->
+        $element = affix('.foo.bar')
+        element = $element.get(0)
+
+        expect(element.className).toEqual('foo bar')
+
+        up.util.addTemporaryClass(element, 'baz')
+
+        expect(element.className).toEqual('foo bar baz')
+
+      it 'returns a function that restores the original class', ->
+        $element = affix('.foo.bar')
+        element = $element.get(0)
+
+        restoreClass = up.util.addTemporaryClass(element, 'baz')
+        expect(element.className).toEqual('foo bar baz')
+
+        restoreClass()
+        expect(element.className).toEqual('foo bar')
+
+
     describe 'up.util.castedAttr', ->
 
       it 'returns true if the attribute value is the string "true"', ->
@@ -578,6 +769,9 @@ describe 'up.util', ->
       it 'returns true for an empty object', ->
         expect(up.util.isBlank({})).toBe(true)
 
+      it 'returns false for a function', ->
+        expect(up.util.isBlank((->))).toBe(false)
+
       it 'returns true for an object with at least one key', ->
         expect(up.util.isBlank({key: 'value'})).toBe(false)
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: unpoly-rails
 version: !ruby/object:Gem::Version
-  version: 0.55.1
+  version: 0.56.0
 platform: ruby
 authors:
 - Henning Koch
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-04-13 00:00:00.000000000 Z
+date: 2018-05-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -89,14 +89,15 @@ files:
 - dist/unpoly.min.css
 - dist/unpoly.min.js
 - lib/assets/javascripts/unpoly-bootstrap3.coffee
+- lib/assets/javascripts/unpoly-bootstrap3/feedback-ext.coffee
 - lib/assets/javascripts/unpoly-bootstrap3/form-ext.coffee
 - lib/assets/javascripts/unpoly-bootstrap3/layout-ext.coffee
 - lib/assets/javascripts/unpoly-bootstrap3/modal-ext.coffee
-- lib/assets/javascripts/unpoly-bootstrap3/navigation-ext.coffee
 - lib/assets/javascripts/unpoly.coffee
 - lib/assets/javascripts/unpoly/browser.coffee.erb
 - lib/assets/javascripts/unpoly/bus.coffee.erb
 - lib/assets/javascripts/unpoly/classes/cache.coffee
+- lib/assets/javascripts/unpoly/classes/css_transition.coffee
 - lib/assets/javascripts/unpoly/classes/extract_cascade.coffee
 - lib/assets/javascripts/unpoly/classes/extract_plan.coffee
 - lib/assets/javascripts/unpoly/classes/extract_step.coffee
@@ -106,6 +107,7 @@ files:
 - lib/assets/javascripts/unpoly/classes/record.coffee
 - lib/assets/javascripts/unpoly/classes/request.coffee
 - lib/assets/javascripts/unpoly/classes/response.coffee
+- lib/assets/javascripts/unpoly/classes/url_set.coffee
 - lib/assets/javascripts/unpoly/dom.coffee.erb
 - lib/assets/javascripts/unpoly/feedback.coffee
 - lib/assets/javascripts/unpoly/form.coffee.erb
@@ -114,7 +116,7 @@ files:
 - lib/assets/javascripts/unpoly/link.coffee.erb
 - lib/assets/javascripts/unpoly/log.coffee
 - lib/assets/javascripts/unpoly/modal.coffee.erb
-- lib/assets/javascripts/unpoly/motion.coffee
+- lib/assets/javascripts/unpoly/motion.coffee.erb
 - lib/assets/javascripts/unpoly/namespace.coffee.erb
 - lib/assets/javascripts/unpoly/popup.coffee.erb
 - lib/assets/javascripts/unpoly/protocol.coffee
@@ -196,6 +198,7 @@ files:
 - spec_app/app/views/method_test/page1.erb
 - spec_app/app/views/method_test/page2.erb
 - spec_app/app/views/motion_test/animations.erb
+- spec_app/app/views/motion_test/transitions.erb
 - spec_app/app/views/pages/start.erb
 - spec_app/app/views/replace_test/_nav.erb
 - spec_app/app/views/replace_test/page1.erb
@@ -260,7 +263,9 @@ files:
 - spec_app/spec/javascripts/helpers/restore_body_scroll.js.coffee
 - spec_app/spec/javascripts/helpers/show_lib_versions.coffee
 - spec_app/spec/javascripts/helpers/to_be_around.js.coffee
+- spec_app/spec/javascripts/helpers/to_be_attached.coffee
 - spec_app/spec/javascripts/helpers/to_be_blank.js.coffee
+- spec_app/spec/javascripts/helpers/to_be_detached.coffee
 - spec_app/spec/javascripts/helpers/to_be_error.coffee
 - spec_app/spec/javascripts/helpers/to_be_given.js.coffee
 - spec_app/spec/javascripts/helpers/to_be_jquery.js.coffee
@@ -270,6 +275,8 @@ files:
 - spec_app/spec/javascripts/helpers/to_contain.js.coffee
 - spec_app/spec/javascripts/helpers/to_end_with.js.coffee
 - spec_app/spec/javascripts/helpers/to_equal_jquery.js.coffee
+- spec_app/spec/javascripts/helpers/to_have_opacity.coffee
+- spec_app/spec/javascripts/helpers/to_have_own_property.js.coffee
 - spec_app/spec/javascripts/helpers/to_have_request_method.js.coffee
 - spec_app/spec/javascripts/helpers/to_have_unhandled_rejections.coffee
 - spec_app/spec/javascripts/helpers/to_match_text.js.coffee