Skip to content
This repository was archived by the owner on Feb 1, 2022. It is now read-only.

.img-responsive should only be used on <img>s#280

Merged
cvrebert merged 1 commit intotwbs:masterfrom
kkirsche:issue#277
May 18, 2015
Merged

.img-responsive should only be used on <img>s#280
cvrebert merged 1 commit intotwbs:masterfrom
kkirsche:issue#277

Conversation

@kkirsche
Copy link
Copy Markdown
Contributor

[Fixes #277] .img-responsive should only be used on <img>s

@kkirsche
Copy link
Copy Markdown
Contributor Author

Same as #279, for whatever reason I'm getting errors when I try to add tests

src/bootlint.js Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should be imgResponsiveNotOnImg instead of this. This is probably the cause of the test failures.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed. It was the lack of a jQuery object as the second argument of reporter which caused it.

@cvrebert cvrebert added this to the v0.13.0 milestone May 18, 2015
@cvrebert
Copy link
Copy Markdown
Collaborator

Could you add a testcase asserting that correct usage of .img-responsive on an <img> results in no warnings?

@kkirsche
Copy link
Copy Markdown
Contributor Author

Course. Give me 5–10 min

@kkirsche
Copy link
Copy Markdown
Contributor Author

@cvrebert added

EDIT: Now with the correct number of expectations...

[Fixes twbs#277] .img-responsive should only be used on <img>s
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling 7c539d3 on kkirsche:issue#277 into 74cfaa8 on twbs:master.

cvrebert added a commit that referenced this pull request May 18, 2015
.img-responsive should only be used on <img>s
@cvrebert cvrebert merged commit ac3833e into twbs:master May 18, 2015
@cvrebert
Copy link
Copy Markdown
Collaborator

Thanks!

@kkirsche kkirsche deleted the issue#277 branch May 18, 2015 23:56
@kkirsche
Copy link
Copy Markdown
Contributor Author

My pleasure. Happy to help out @cvrebert. Thanks for the review and help finding that test issue

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

.img-responsive should only be used on <img>s

3 participants