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

Warn about .pull-right or .pull-left within a navbar #189

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

zacechola
Copy link
Collaborator

I added this as W011, due to the similarity to W010. LMK if you feel differently.

Closes #188

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling ca920c0 on zacechola:quick_floats into 6bbb189 on twbs:master.

@cvrebert
Copy link
Collaborator

Using W009 instead would fill the gap.

@zacechola
Copy link
Collaborator Author

W009 is in use in the code for the empty column lint. Needs to be documented on the wiki.

@cvrebert
Copy link
Collaborator

Indeed! Documenting...

@cvrebert cvrebert added this to the v0.9.0 milestone Dec 12, 2014
@hnrch02 hnrch02 modified the milestones: v0.10.0, v0.9.0 Dec 18, 2014
@@ -11095,6 +11104,12 @@ var LocationIndex = _location.LocationIndex;
reporter('Using `.pull-left` or `.pull-right` as part of the media object component is deprecated as of Bootstrap v3.3.0. Use `.media-left` or `.media-right` instead.', mediaPulls);
}
});
addLinter("W011", function lintMediaPulls($, reporter) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Function name needs changing

@cvrebert cvrebert modified the milestones: v0.10.0, v0.11.0 Jan 21, 2015
@cvrebert cvrebert modified the milestones: v0.11.0, v0.12.0 Feb 24, 2015
@cvrebert cvrebert modified the milestones: v0.12.0, v0.13.0 Mar 16, 2015
Zac Echola added 2 commits April 6, 2015 14:55
Quick floats can be used within components that are within a navbar, but
not on individual components.

See this [comment](twbs#189 (comment))

Add brand to list and adjust test
Conflicts:
	dist/browser/bootlint.js
	src/bootlint.js
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling ed4f576 on zacechola:quick_floats into 3eb0a3c on twbs:master.

@cvrebert cvrebert modified the milestones: v0.13.0, v0.14.0 Nov 16, 2015
@cvrebert cvrebert modified the milestones: v0.14.0, v0.13.0, v1.0.0 Nov 16, 2015
@XhmikosR
Copy link
Member

XhmikosR commented Aug 10, 2017

@zacechola: can you rebase and skip the dist file?

@XhmikosR
Copy link
Member

Alternatively, @zacechola, you can enable Allow edits from maintainers. on the right in this PR and I'll rebase myself.

@Herst Herst removed this from the v1.0.0 milestone Jul 29, 2019
@Herst
Copy link
Collaborator

Herst commented Jul 29, 2019

Removing from v1.0.0 milestone since it is not necessary for supporting Bootstrap v4+.

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.

Warn about .pull-right or .pull-left within a navbar
6 participants