Closed Bug 1868788 Opened 1 year ago Closed 8 months ago

[Inactive CSS] Show table-related CSS properties as inactive when they're not on a table-related element

Categories

(DevTools :: Inspector: Rules, enhancement, P3)

enhancement

Tracking

(firefox129 fixed)

RESOLVED FIXED
129 Branch
Tracking Status
firefox129 --- fixed

People

(Reporter: julienw, Assigned: sebo)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

I noticed border-spacing and border-collapse, but there may be others.

Severity: -- → S3
Priority: -- → P3
Assignee: nobody → sebastianzartner
Status: NEW → ASSIGNED

border-spacing and border-collapse seem to be the only ones left that had to be tested. The other table-related properties like table-layout and empty-cells are already tested.

Sebastian

Summary: Show table-related CSS properties as inactive when they're not on a table-related element → [Inactive CSS] Show table-related CSS properties as inactive when they're not on a table-related element

There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:sebo, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.

Flags: needinfo?(sebastianzartner)
Flags: needinfo?(nchevobbe)

I think everything was good. Sebo, is there anything preventing us from landing this?

Flags: needinfo?(nchevobbe)

No, I just forgot to land this. ☺️ I've pushed this to Lando now.

Sebastian

Flags: needinfo?(sebastianzartner)

Needed to do a rebase. Trying to land again.

Sebastian

Pushed by sebastianzartner@gmail.com: https://hg.mozilla.org/integration/autoland/rev/fd8670544009 [devtools] Handle table-related issues in inactive CSS. r=devtools-reviewers,fluent-reviewers,nchevobbe.
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 129 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: