Giter Site home page Giter Site logo

neal-react's People

Contributors

bartekus avatar bysabi avatar dennybritz avatar kristinbaumann avatar meriadec avatar skvale avatar trevor-coleman avatar waldyrious avatar weblancaster avatar

Stargazers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

Watchers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

neal-react's Issues

bootstrap reset

Hi,

I have a bootstrap reset SCSS file in my project.
(I'm using Angle as my template).

This reset file, of course, change a lot the visibility of the neal page.
I'm looking for a way to exclude all the reset file for lean.
Any idea on a way to do it?

Thaks!

React 16 compatibility?

I tried using this library with an app that uses React 16 and I get errors that look to be due to use of deprecated features. Are there any plans to bring this library up-to-date with recent React changes?
thanks

Unknown prop `backgroundImage` on <div> tag

Use of {... this.props} is causing this error in the browser console:

Unknown prop `backgroundImage` on <div> tag

From https://facebook.github.io/react/warnings/unknown-prop.html

Are you using {...this.props} or cloneElement(element, this.props)? Your component is transferring its own props directly to a child element (eg. https://facebook.github.io/react/docs/transferring-props.html). When transferring props to a child component, you should ensure that you are not accidentally forwarding props that were intended to be interpreted by the parent component.

incompatible with React 16?

I tried using this library with an app that uses React 16 and I get errors that look to be due to use of deprecated features. Are there any plans to bring this library up-to-date with recent React changes?
thanks

Can't override renderBody method on SignupModal component

I created a custom component based on SignupModal:

import React from "react";
import {
  SignupModal
} from "neal-react";

export class FeedbackModal extends SignupModal {
  renderBody() {
    if (this.props.children) return this.props.children;
    return (
      <div>
        <SignupModal.Input name="name" label="Имя" placeholder="Имя"/>
        <SignupModal.Input type="email" name="email" required label="E-mail" placeholder="E-mail"/>
        <SignupModal.TextArea name="message" required label="Сообщение" placeholder="Сообщение"/>
      </div>
    );
  }
}

Then i'm using it like so:

<FeedbackModal title="Обратная связь" buttonText="Отправить" modalId="feedback-modal" onSubmit={onFeedback}/>
<p>
    <a className="btn btn-primary btn-ghost" data-toggle="modal" data-target="#feedback-modal">Оставить отзыв</a>
</p>

The problem is that the original renderBody is getting called.

I tried to override a render method as well to examine which class the renderBody's method prototype belongs to. It turns out it points to the original SignupModal class.

Why is this happening? I'm quite new to ES6, but haven't found anyone having similar issues.
Is it because neal-react ships already transpiled code?

Using ES6 features in older browsers

Currently available versions of Internet Explorer used in Windows 7, Window 8.1, Windows Phone 8.1 and Windows 10 do not support ES6 (e.g. Object.assign) natively, and there are no polyfills included with neal-react for ES6 features it uses.

Due to this issue, a neal-react-based landing page (i.e. nealjs.com) cannot be opened on a Windows phone at all, as there are no third-party browsers for this platform.

neal-react need media queries

Hi @dennybritz. We need mediaqueries for Navbar code. I like very much current simple and clean aproach but I cannot put any logic on childrens cause this component are add twice on Navbar render.
I create a mediaquery module for this: https://www.npmjs.com/package/react-mediaquery-props

And rewrote Navbar like this.

import React from "react";
import classNames from "classnames";
import { mediaQuery } from "./media-query.jsx";
import { mdUp } from "./bootstrap4-media-query.js";

export const Navbar = mediaQuery(class Navbar extends React.Component {

  static propTypes = {
    brand: React.PropTypes.node.isRequired,
  };

  render() {
    const _className = classNames("navbar neal-navbar", this.props.className);

    const mdUp = this.props.mq.mdUp
      ? { id: 'header-nav', div: '', ul: 'pull-right' }   // medium Up
      : { id: 'mobile-nav', div: 'collapse neal-mobile-nav', ul: '' };

    return (
      <header className="neal-navbar-wrapper">
        <nav className={_className}>
          <div className="container">
            <button className="navbar-toggler hidden-md-up" type="button" data-toggle="collapse"
              data-target="#mobile-nav">
              &#9776;
            </button>
            <a className="navbar-brand hidden-sm-down" href="/">{this.props.brand}</a>
            <div className={`navbar-toggleable-sm ${mdUp.div}`} id={mdUp.id}>
              <ul className={`nav navbar-nav ${mdUp.ul}`}>
                {this.props.children}
              </ul>
            </div>
          </div>
        </nav>
      </header>
    );
  }
}, mdUp);

Tell me is OK for you and I will make a PR.

Custom Fields for Signup Modal

Signup modal should support custom fields besides the default name and email. How to keep the markup simple but make it flexible?

Site Generator

Add a generator for scaffolding a new project similar to neal-sample. Should include:

  • README
  • Basic Page
  • Build scripts
  • Linting best practices
  • Deployment script
  • Dockerfile / Development environment

problems with import bootstrap in base.scss

The current import is

@import '../node_modules/bootstrap/scss/bootstrap';

But with npm 3 if you use bootstrap in your app it will most of the time will be in the main node_modules folder instead of inside neal-react.
Then the import should be (one folder up)

@import '../../node_modules/bootstrap/scss/bootstrap';

typo on sample landing page

Under the "batteries included"...
"repetetive" should be "repetitive"...sorry could not figure out a better place to report this

[License issue] Please help update license for community growth

We appreciate your great work ! It'd be great if this PJ was Apache license. Please help change current license into Apache license. There will be many contributors waiting for your consideration !. Once again, because it's open-source, because community growth and because the world of happy coding !
Many thanks !

refactor to avoid jQuery

As far as I can see, jQuery is loaded only to use getScript(). Would it be an idea to refactor to vanilla JS instead? I'm happy to give it a try, if you interested in a PR. I don't like loading jQuery just for that single method.

Add email capture component

I'd like a component similar to the signup component, but for just requesting an email address for subscribing to a MailChimp or other list.

I guess this is somewhat related to #8, but in the absence of a solution there, capturing emails is one feature I was surprised to see missing.

Thanks for this, looks like a very neat project!

Currency symbol from Font Awesome?

$ symbol has a wide support of web fonts, but some currency, for example Russian ruble \u20BD has not.

It would be nice to be able to select the currency symbol in pricing-table component.

Not very familiar with React, tried to do, but unsuccessfully:

<div className="neal-pricing-plan-price-amount"><i className="fa fa-rub"></i> {this.props.price}</div

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.