https://linen.dev logo
Join Discord
Powered by
# react
  • a

    ambitious-knife-25690

    01/08/2020, 4:04 PM
    but it would be a breaking change if you ever want to consider that :p
  • a

    ambitious-knife-25690

    01/08/2020, 4:04 PM
    I wouldn't mind helping port that annoying abstract rework 😂
  • a

    ambitious-knife-25690

    01/08/2020, 4:05 PM
    oh wait, you're aiming to keep 3.4.7 compat until 4.1
  • a

    ambitious-knife-25690

    01/08/2020, 4:05 PM
    that makes life a little sadder x)
  • u

    user

    01/08/2020, 4:06 PM
    Fixing what?
  • a

    ambitious-knife-25690

    01/08/2020, 4:06 PM
    changing the api from
    Fixed
    back to
    fixed
    for these things
  • u

    user

    01/08/2020, 4:07 PM
    I don't plan to do that
  • a

    ambitious-knife-25690

    01/08/2020, 4:07 PM
    ahh
  • a

    ambitious-knife-25690

    01/08/2020, 4:19 PM
    It's alive!!
  • a

    ambitious-knife-25690

    01/08/2020, 4:19 PM
    @User Thanks <3
  • u

    user

    01/08/2020, 4:21 PM
    \o/
  • b

    brainy-machine-50829

    01/08/2020, 4:22 PM
    Now I wanna go back to react material... 😄
  • a

    ambitious-knife-25690

    01/08/2020, 4:27 PM
    :D
  • a

    ambitious-knife-25690

    01/08/2020, 4:43 PM
    You didn't manually do all of these, did you?
  • a

    ambitious-knife-25690

    01/08/2020, 4:47 PM
    hmm, it doesn't appear like you're using material icons
  • u

    user

    01/08/2020, 4:47 PM
    @material-ui/icons
  • u

    user

    01/08/2020, 4:48 PM
    Icons are scripted, yeah x)
  • a

    ambitious-knife-25690

    01/08/2020, 4:51 PM
    Ah okay that makes sense
  • a

    ambitious-knife-25690

    01/08/2020, 4:51 PM
    I was confused but it was just auto completion being laggy
  • a

    ambitious-knife-25690

    01/08/2020, 4:52 PM
    I couldn't find
    MailIcon
    but just saw that you got rid of the
    Icon
    suffix
  • a

    ambitious-knife-25690

    01/10/2020, 10:51 PM
    History seems to be optional in but react-router makes requires it, is that by design?
  • a

    ambitious-knife-25690

    01/10/2020, 10:52 PM
    Either way, is the proper way to utilise
    wrap
    by adding
    @:wrap(react.router.ReactRouter.withRouter)
    to the Component and then adding
    history
    as prop?
  • a

    ambitious-knife-25690

    01/10/2020, 11:06 PM
    Copy code
    ×
    You should not use <Route> or withRouter() outside a <Router>
  • a

    ambitious-knife-25690

    01/10/2020, 11:06 PM
    Copy code
    @:wrap(ReactRouter.withRouter)
    @:wrap(Styles.withStyles(styles))
    @:expose('App')
    class App extends ReactComponentOf<Props, State> {
  • a

    ambitious-knife-25690

    01/10/2020, 11:07 PM
    and my code looks like the basic example
  • a

    ambitious-knife-25690

    01/10/2020, 11:16 PM
    It seems to not be related to Styles
  • a

    ambitious-knife-25690

    01/10/2020, 11:41 PM
    okay i've gotten past that annoying issue
  • a

    ambitious-knife-25690

    01/10/2020, 11:41 PM
    Copy code
    override function render() {
            // trace(this.props.history);
            return jsx('
            <Router history={this.props.history}>
            <ul>
            <li>
              <Link to="/">Home</Link>
            </li>
            <li>
              <Link to="/about">About</Link>
            </li>
            <li>
              <Link to="/dashboard">Dashboard</Link>
            </li>
         </ul>
         <hr />
    
         <Switch>
            <Route exact path="/">
              <Overview />
            </Route>
            <Route path="/about">
              <SystemLogs />
            </Route>
         </Switch>     
            </Router>
  • a

    ambitious-knife-25690

    01/10/2020, 11:42 PM
    The error is on the first
    Route
    in the
    <Switch>
  • a

    ambitious-knife-25690

    01/10/2020, 11:48 PM
    aaaaaand invariant failed error is back
1...272829...80Latest