twbs / bootlint Goto Github PK

View Code? Open in Web Editor NEW

2.4K 2.4K 313.0 3.67 MB

HTML linter for Bootstrap projects

License: MIT License

JavaScript 99.84% Shell 0.16%

bookmarklet bootlint bootstrap html linter

bootlint's People

Contributors

Stargazers

Watchers

Forkers

zacechola mischah seco richguan edwinyzh ughnettdev ninjabydesign tanejaboy archiman2014 x-coder yfix interactivellama nunodotferreira intfrr iamphill aydev briandamage portosmoita craiglondon umeshptiwari mubassirhayat jeanperriault kodofish-forks ishawge roysirengo coneain asitti babukrishna animasling nitinbly rockyspade aamirchaudhary briannw ram-pratap angeliaz nooklee jjain2901 securecloud-biz yuanjack danieldalonzo shaoyizhi bidle closeststorm johanoloflindberg biotep zerluto colinwebb dnyli havalamit martindale alexisarce mmegaard kkirsche liukailcl daliryc superwzx tennessine yudhitech newlysoft kailashsingh marquisknox atiqsu antrax69 ledonihr chris02stang jkirsh cronk sergiososasr khamull cloudrifles rusjames darineraka patmart nvminh91 cmarx senterre pablobrady cloverandroid jean wichtounet nenaroig noopnik vrkansagara weaver-viii senthilmsv ravenb ravenda900 qiangck chenruixuan genaiwei digitien alexree zhuying016 diamondwilliams techkrish cnbin minniebest zhangbg cay914 foosbar

bootlint's Issues

Setup hosted, webhook-integrated instance for @twbs

Fix rows-within-containers rule: container merely has to be ancestor

The violations are:

The Customizer page
The gallery-ish "Downloads" and "Examples" sections of the Getting Started page
The "Column sizing" example in the "Forms" section of the CSS page

Warn about multiple buttons/dropdowns in an input-group-btn

Refs twbs/bootstrap#14293
Checking for exactly 1 <button> child seems sufficient.

use local copy of QUnit for faster & more reliable tests

Check that tooltips within btn-group or input-group have container=body set

Same for popovers too

Check that the modal markup isn't within another component

setup PhantomJS/QUnit tests

Grunt plugin

#!/usr/bin/env nodejs

$ ./src/cli.js  
env: nodejs: No such file or directory

Haven't seen #!/usr/bin/env nodejs before, all I know of is #!/usr/bin/env node.

Check that input groups are not mixed with other widgets

Specifically, form groups and grid classes should not be combined with them; nesting should be used instead

For the button plugin, check that preselected options also have the .active class

Check that input groups don't contain multiple form controls

Check that the original and the minified JS aren't both included

Divide problem reports into errors and mere warnings

Check that remote modals are not used

pluggable Reporters

Pull requests welcome.

Warn about sm/lg .input-groups that don't use .input-group-sm/lg

Refs twbs/bootstrap#14252

Wrong:

    <div class="input-group">
        <span class="input-group-btn">
            <button id="button" class="btn btn-default btn-lg" type="button">Go!</button>
        </span>
        <input id="input" type="text" class="form-control input-lg" />
    </div>

Right:

    <div class="input-group input-group-lg">
        <span class="input-group-btn">
            <button id="button" class="btn btn-default" type="button">Go!</button>
        </span>
        <input id="input" type="text" class="form-control" />
    </div>

Check that all rows are within a .container(-fluid)

use TypeScript

Because _FUCK JavaScript_, but this still needs to be able to run in-browser.

add david-dm badges

Try to enable block-scoped-var check in ESLint

add proper docs & a proper README

check that charset=utf-8 is set

Check that disabled elements don't have tooltips directly on them

Same for popovers too

Sanity check: Ensure that the Bootstrap docs pass bootlint

BS docs-specific exemptions to the row-must-have-container rule

Need to exempt .row.show-grid and .row.responsive-utilities-test

Check structure of form groups with feedback

Refs twbs/bootstrap#14230

Check that all columns are children of rows or form-groups

Check that input groups don't use a <select> or <textarea>

publish an npm module & add npm badge

check that HTML5 doctype is set

Check that viewport meta tag is included

.form-horizontal/.form-inline shouldn't be used directly on .form-group

Nesting should be used instead.
Need to verify this with @mdo.

Warn about split-button dropdowns where the button comes after the dropdown

Refs twbs/bootstrap#13150

Check that .checkbox and .radio markup is correctly structured

Valid structure is:

div.checkbox>label>input[type="checkbox"]
div.radio>label>input[type="radio"]
label.checkbox-inline>input[type="checkbox"]
label.radio-inline>input[type="radio"]

Warn about nested containers

I still see this more often than I'd like to see it.

<div class=".container">
  <!-- Some other things -->
  <div class=".container">
    <!-- Things in nested container>
  </div>
</div>

Sometimes this might be desired, but more often than not it screws up the grid.

See: twbs/bootstrap#11436

rows-within-containers check needs to allow rows to be children of columns

check that X-UA-Compatible meta tag is included

Integrate ESLint into Gruntfile

Flag outdated Bootstrap v2 .span* classes

CDN for browserify'd version

I guess http://www.bootstrapcdn.com/ , if they're willing.

Check that .col-* and .row are not used on the same element

Cheerio/bootlint false positive when linting renamed jQuery

Selecting for the jQuery script can return a false positive warning if a user renames the jQuery file, such as when jQuery is concatenated with other JS.

Probably not going to be a common issue in the wild, but should be noted in the caveats maybe?