Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Wall of Browser Bugs entries for Selectors Level 4 :nth-child(An+B of S) #20143

Merged
merged 1 commit into from
Jun 22, 2016

Conversation

cvrebert
Copy link
Collaborator

I propose adding these on the grounds that the of S feature would allow us to make our widgets more robust. We've had several issue reports (e.g. #15684, #13825, #16540) where folks tried to put stuff like <input type="hidden"> within .input-group or .btn-group, but this broke the :first-child/:last-child-based selectors we use to adjust rounded corners.
With this feature, we could restrict the children considered to only the relevant ones, thus ignoring other irrelevant elements. For example, we could use :nth-child(1 of .btn) in our .btn-group CSS to ignore non-.btn children and thus round the corners correctly in such cases.

Refs:

CC: @twbs/team for review

@hnrch02
Copy link
Collaborator

hnrch02 commented Jun 20, 2016

👍

With regards to the feature: Is it backwards compatible? I presume it isn't.

@cvrebert
Copy link
Collaborator Author

AIUI, it isn't. We'd probably only be able to use it in the far future when IE has been dropped completely. But I think we'd like this feature to be ready by the time that world arrives.

@cvrebert
Copy link
Collaborator Author

@patrickhlauke @XhmikosR LGTY?

@XhmikosR
Copy link
Member

LGTM although I do feel it's a little early but it doesn't hurt.

@cvrebert cvrebert merged commit d39e243 into v4-dev Jun 22, 2016
@cvrebert cvrebert deleted the nth-child-of branch June 22, 2016 08:20
cvrebert added a commit that referenced this pull request Jun 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants