New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Textures example from Elm website doesn't work #230

Closed
leonardoborges opened this Issue Apr 29, 2015 · 3 comments

Comments

Projects
None yet
3 participants
@leonardoborges

leonardoborges commented Apr 29, 2015

The example is as follows:

import Graphics.Collage exposing (..)
import Graphics.Element exposing (..)


main : Element
main =
  collage 300 300
    [ textured "/stripes.jpg" (ngon 5 75)
    ]

Nothing is rendered and the Javascript console reports the error:

Uncaught ReferenceError: redo is not defined

Offending snippet taken from Collage.js:

    function setFillStyle(ctx, style)
    {
        var sty = style.ctor;
        ctx.fillStyle = sty === 'Solid'
            ? Color.toCss(style._0)
            : sty === 'Texture'
                ? texture(redo, ctx, style._0)
                : gradient(ctx, style._0);
    }

I'm using Elm 0.15.

@hblumberg

This comment has been minimized.

Show comment
Hide comment
@hblumberg

hblumberg Sep 21, 2015

Seems like this might be an issue again: http://elm-lang.org/examples/texture

hblumberg commented Sep 21, 2015

Seems like this might be an issue again: http://elm-lang.org/examples/texture

@jvoigtlaender

This comment has been minimized.

Show comment
Hide comment
@jvoigtlaender

jvoigtlaender Sep 21, 2015

Contributor

No, not an issue again, but an issue still, in the sense that it will only be fixed on the site http://elm-lang.org when the next version of https://github.com/elm-lang/core/ has been released and is used for compiling the website.

So the fix exists and has been merged, but does not yet take effect in the deployed website.

Contributor

jvoigtlaender commented Sep 21, 2015

No, not an issue again, but an issue still, in the sense that it will only be fixed on the site http://elm-lang.org when the next version of https://github.com/elm-lang/core/ has been released and is used for compiling the website.

So the fix exists and has been merged, but does not yet take effect in the deployed website.

@hblumberg

This comment has been minimized.

Show comment
Hide comment
@hblumberg

hblumberg Sep 21, 2015

Oh, I'm sorry, I misunderstood! Thanks for your quick response.

hblumberg commented Sep 21, 2015

Oh, I'm sorry, I misunderstood! Thanks for your quick response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment