Skip to content
This repository has been archived by the owner on Aug 30, 2018. It is now read-only.

Nth of type #604

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Nth of type #604

wants to merge 1 commit into from

Conversation

klaser
Copy link

@klaser klaser commented Dec 15, 2016

Bold Apps Customer Specific Pricing app injects <script> tags between <div> tags on the product grid. The nth-child pseudo-class did not appear to select the appropriate <div> tag because of the injection of <script> tags. When I switched to using nth-of-type, the issue was resolved.

Given the possibility that other apps may also modify the grid, using nth-of-type reduces the chances of other users encountering this issue.

Some Shopify Apps modify the grid using script tags injected between divs. nth-child does include script tags whereas nth-of-type does not.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant