Closed Bug 1043835 Opened 10 years ago Closed 9 years ago

Add Rule on an element with multiple classes should include all the classes

Categories

(DevTools :: Inspector, defect)

Product:
DevTools
Component:
Inspector
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox41 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 41
Tracking Flags:
Tracking Status
firefox41 --- fixed

People

(Reporter: harth, Assigned: gl)

References

Details

Attachments

(1 file)

1043835.patch
2.41 KB, patch
pbro
: review+
Details | Diff | Splinter Review 
STR:

1) Edit an element to have class="foo bar"
2) right-click "Add Rule" in Inspector rules view

Expected:
See rule with selector ".foo" or ".bar" or maybe ".foo, .bar"

Actual:
See rule with selector ".foo bar", which won't match the element since that matches an element with tag name "bar" that is a descendent of an element with class "foo".
Assignee: nobody → gabriel.luong
Assignee: gabriel.luong → nobody
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Assignee: nobody → gabriel.luong
Summary: Add Rule on an element with multiple classes has wrong selector → Add Rule on an element with multiple classes should include all the classes
Attached patch 1043835.patch — — Splinter Review 
ttps://treeherder.mozilla.org/#/jobs?repo=try&revision=a235e33368de
Attachment #8608349 - Flags: review?(pbrosset)
Status: REOPENED → ASSIGNED
Attachment #8608349 - Flags: review?(pbrosset) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/389db04d382f
Status: ASSIGNED → RESOLVED
Closed: 9 years ago → 9 years ago
status-firefox41: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 41
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: