Skip to content

Feedback implementation#17

Open
rhenshaw56 wants to merge 4 commits into
stagingfrom
feedback-implementation
Open

Feedback implementation#17
rhenshaw56 wants to merge 4 commits into
stagingfrom
feedback-implementation

Conversation

@rhenshaw56

Copy link
Copy Markdown
Owner

No description provided.


describe('Navbar', () => {
let wrapper = shallow(<Navbar {...props} />);
console.log('nav-props', wrapper.props());

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unexpected console statement no-console

}
#full-edit {
margin: 10%;
margin-left: -50%;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Properties should be ordered margin, margin-bottom, margin-left, margin-top, width

div.card-action.row{
height: 9%
}
#full-edit {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoid using id selectors

}
div.card-action.row{
height: 9%
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rule declaration should be followed by an empty line

Comment thread server/server.js
import webpackConfig from '../webpack.config';

import routes from './routes/index';
import routes from './routes';

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing file extension for "./routes" import/extensions


describe('Navbar', () => {
let wrapper = shallow(<Navbar {...props} />);
console.log('nav-props', wrapper.props());

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unexpected console statement no-console

name="password"
required
/>
<label htmlFor="new">Comfirm Password</label>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Form label must have associated control jsx-a11y/label-has-for

name="lastName"
required
/>
<label htmlFor="new">Password</label>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Form label must have associated control jsx-a11y/label-has-for

name="firstName"
required
/>
<label htmlFor="new">Last Name</label>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Form label must have associated control jsx-a11y/label-has-for

onClick={this.updateFullProfile}
>Save
</Button>
]

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Expected indentation of 28 spaces but found 26 indent

className="btn-save"
onClick={this.updateFullProfile}
>Save
</Button>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Expected indentation of 30 spaces but found 36 indent
Expected closing tag to match indentation of opening react/jsx-closing-tag-location
Expected indentation of 28 space characters but found 36 react/jsx-indent

flat
className="btn-save"
onClick={this.updateFullProfile}
>Save

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Expected indentation of 30 spaces but found 28 indent

waves="light"
flat
className="btn-save"
onClick={this.updateFullProfile}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Expected indentation of 32 spaces but found 30 indent

id="sav"
waves="light"
flat
className="btn-save"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Expected indentation of 32 spaces but found 30 indent

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants