Skip to content
This repository was archived by the owner on Jul 19, 2019. It is now read-only.
This repository was archived by the owner on Jul 19, 2019. It is now read-only.

render mode is unconfigurable #54

Open

Description

@fumihwh

I have read the issue 17 and test source.
#17
https://github.com/reactjs/react-art/blob/master/src/__tests__/ReactART-test.js#L55-L59

And I think there is no way to change the render mode.
In ReactART.js, there is requiring the art/modes/fast, that decide the mode used in ReactART.
@sebmarkbage You said that is already configurable and already do this in the unit test.
Actually L56-L59 in ReactART-test.js do nothing about ReactART Object.

I think you can just delete the L14 in ReactART.js, and let us choose the mode freely.

Activity

  1. webxl commented on Sep 9, 2015

    @webxl

    I agree with @fumihwh. The unit test has access to the dependency's API directly, whereas any application built on top of this doesn't. Mode needs to be exposed in ReactART, along with the available modes (e.g. ReactART.AvailableModes.SVG). And ideally you could change the mode with a component property.

  2. varghesep commented on Sep 17, 2015

    @varghesep

    @fumihwh + I'm also looking for a solution.

  3. shripadk commented on Oct 3, 2015

    @shripadk

    +1. Would be great if the mode is configureable.

  4. zemlanin commented on Oct 28, 2015

    @zemlanin

    +1

  5. andru commented on Jan 1, 2016

    @andru

    +1 Also looking to have this configurable

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions