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

Conversation

cvrebert
Copy link
Collaborator

Fixes #133.
Closes #136.
Blocked on cheeriojs/cheerio#582 or a monkeypatch equivalent.

@coveralls
Copy link

Coverage Status

Coverage decreased (-29.03%) when pulling 481a9e9 on alert-dismissible into 0346af8 on master.

@hnrch02
Copy link
Collaborator

hnrch02 commented Oct 23, 2014

Wow, a 29.03% decrease in test coverage.

@hnrch02
Copy link
Collaborator

hnrch02 commented Oct 23, 2014

The has() selector works fine for me though:

$ node
> var cheerio = require('cheerio')
undefined
> var $ = cheerio.load('<div class="foo"><div class="bar"></div></div><div class="foo"></div>')
undefined
> $('.foo').length
2
> $('.foo:has(.bar)').length
1

@cvrebert
Copy link
Collaborator Author

@hnrch02 Excellent find!

@cvrebert cvrebert added this to the v0.6.0 milestone Oct 23, 2014
@hnrch02
Copy link
Collaborator

hnrch02 commented Oct 23, 2014

🎉

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling b2ca3cd on alert-dismissible into 0346af8 on master.

cvrebert added a commit that referenced this pull request Oct 23, 2014
Add .alert-dismissible lint checks
@cvrebert cvrebert merged commit 3cef080 into master Oct 23, 2014
@cvrebert cvrebert deleted the alert-dismissible branch October 23, 2014 04:49
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.

Check for correct usage of Close button in Alerts
4 participants