Gabriel95 / scalafx

Automatically exported from code.google.com/p/scalafx
Other
0 stars 0 forks source link

Node transforms= does not accept a list of ScalaFX wrapper Transform's #20

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
For some reason, the 'transforms' setter is defined as this:

<<
  def transforms_=(c: Iterable[jfxst.Transform]) {
    transforms.setAll(c)
  }
<<

It's taking a list of Java-level transforms, not ScalaFX level. Why?

There does not seem to be an implicit conversion available to make this work 
with Scala Transform's, either.

>What version of the product are you using? On what operating system?

Current hg (31-12-2012). OS X

Original issue reported on code.google.com by akau...@gmail.com on 31 Dec 2012 at 11:39

GoogleCodeExporter commented 8 years ago
Could you provide a sample code demonstrating this issue?

I would try to fix this by adding a mapping such as
    transforms.setAll(c.map(_.delegate))

but I have currently no code base against which to test.

Would you be so nice as to contribute a short test for this issue?

Original comment by Alain.Fa...@gmail.com on 17 Jan 2013 at 2:50

GoogleCodeExporter commented 8 years ago
There were, in the whole library, only three collection setters which were not 
following the pattern of setting the ScalaFX wrappers.
All three utterances have been fixed.

Original comment by Alain.Fa...@gmail.com on 29 Jan 2013 at 10:26