Open Bug 1305664 Opened 8 years ago Updated 2 years ago

Remove css class workaround that controlled the clear button display in search input

Categories

(DevTools :: Inspector, defect, P3)

Product:
DevTools
Component:
Inspector
Type:
defect

Tracking

(firefox52 affected)

Tracking Flags:
Tracking Status
firefox52 --- affected

People

(Reporter: steveck, Unassigned)

References

Details

(Whiteboard: [reserve-html])

This bug is a follow up for unifying the structure/style of the searchinput. While bug 1296187 applied a workaround the added additional class to make the styling works for both html/xul cases. We should remove the workaround after xul replacement finished.
Should this be part of the devtools.html project? If not, why is this a P1?
Flags: needinfo?(schung)
Sorry for the misleading flag(and the title). They're just carried from bug 1296187 and I didn't set the correct information...
Flags: needinfo?(schung)
Priority: P1 → --
Summary: Don't overlap inspector-searchinput-clear with text → Remove css class workaround that controlled the clear button display in search input
Blocks: 1296187
No longer depends on: 1296187
Blocks: devtools-html-phase2
No longer blocks: 1296187
Depends on: 1296187
Whiteboard: [devtools-html] [triage]
Priority: -- → P3
Whiteboard: [devtools-html] [triage] → [reserve-html]
Flags: qe-verify?
Priority: P3 → P2
Whiteboard: [reserve-html] → [devtools-html]
Priority: P2 → P1
Priority: P1 → P3
Whiteboard: [devtools-html] → [reserve-html]
Product: Firefox → DevTools
Severity: normal → S3

Changing qe-verify? to qe-verify+.

Flags: qe-verify? → qe-verify+
You need to log in before you can comment on or make changes to this bug.