Make .transition mixin take only 1 value and fix usage
authorTimo Tijhof <krinklemail@gmail.com>
Mon, 7 Oct 2013 23:01:45 +0000 (01:01 +0200)
committerTimo Tijhof <krinklemail@gmail.com>
Mon, 7 Oct 2013 23:59:57 +0000 (01:59 +0200)
commit7e953bbeb6d2e443212a84f3413964ddd38d3db2
tree77539dea29a6802dc28b99d724d06cdb369b00be
parent7c4b29efd8f17a1591c85850d729dc68aa259591
Make .transition mixin take only 1 value and fix usage

Follows-up 99c92a87fc which turned
  transition: left 250ms;
  transition: margin-left 250ms, padding 250ms
into:
  transition: 'left 250ms';
  transition: margin-left 250ms padding 250ms
  > Invalid CSS property value: 'left 250ms'
  > Invalid CSS property value: margin-left 250ms padding 250ms

Removed quotes from 'left 250ms' which was producing invalid CSS.

The @arguments operator in Less joins a variadic list of
arguments by space. In the case of .transition this is a problem
because its values are supposed to be separated by a comma
(like background-image and font-family).

Changing the .transition definition to take only 1 argument
instead of multiple (...). There is no point in supporting
multiple arguments. All it does is it take away our ability to
spot these errors at parse time (Less would be able to throw an
error for passing a 2 arguments to a mixin that only takes 1).

Fixed the invocation with 2 arguments to instead pass 1 argument
that has a comma in it. Both "~'a,b'" and "a, b;" would work here
(doesn't matter) choosing ; here because for consistency with calls
that do use mutiple argments:
 foo(~'foo, bar', ~'baz, quux')
 foo(foo, bar; baz, quux)

Change-Id: I80786c726a412d6b9e04c384112849e9587990b4
resources/mediawiki.less/mediawiki.mixins.less
skins/vector/screen.less