MaxGfeller / MaxGfeller/apply-css

Multi selectors throughout separator ',' doesn't work

Open
#2 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Hacktoberfest
Dominant language
JavaScript
Stars
1
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Hello found one strange bug, with multi css selectors.
The case look like :

div, div.example, div.example1{}

data-scoped-css apply only for first selector.

In module code, you doesn't process array of selectors, just only first, you could observe this in
line code 15 : var s = o.stylesheet.rules[i].selectors[0]

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Inspect the module code at line 15, especially how stylesheet.rules[i].selectors is read. Reproduce the issue with div, div.example, div.example1{} and verify that the scoped output handles every selector rather than only the first; done means all listed selectors receive the expected scope.

Written by the indexing model from the issue text.

Assessment

Tech stack
css, javascript
Domain
frontend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.