1168055 - Active Pseudo class not working for textbox in firefox 38

Status: RESOLVED FIXED

(Core :: Layout: Form Controls, defect)

Description

[ronnie kapoor]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Firefox/38.0
Build ID: 20150513174244

Steps to reproduce:

i created an HTML page having textbox control (input box) and i applied three classes . below is the code

body {
  background: lightblue;
}
.txtLogin {
  width: 200px;
  padding: 1px !important;
  border: 1px solid #ccc;
  height: 20px;
  font-size: 12px;
  font-weight: normal;
  color: #000;
  font-family: Arial;
}
.txtLogin:hover {
  background: #ededed;
  border: 1px solid #bfbfbf;
  border-top: 1px solid #b5b5b5;
}
.txtLogin:active {
  background: #d9d9d9;
  border: 1px solid #bfbfbf !important;
}


code for textbox in .html

<input type="textarea" class="txtLogin">


Actual results:

on clicking on text box , active pseudo class must have been applied but it isnt


Expected results:

active class should have been applied

Comment 1

[Loic]

Attachment: 1168055.html (reporter's testcase)

Comment 2

[ronnie kapoor]

Thanks for creating the Text case. issue is being reproduced on the same

Comment 3

[Alice0775 White]

Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=369a8f14ccf8&tochange=debc66b7bfe7

Regressed by: Bug 924087

Comment 4

[Boris Zbarsky [:bzbarsky]]

We should either condition that check on the content not being anonymous, or skip up to the non-anonymous parent when anonymous content IsEditable() and repeat the check there...

Comment 5

[Boris Zbarsky [:bzbarsky]]

[Tracking Requested - why for this release]: Web compat regression.

Comment 6

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Adding a tracking flag on Firefox40 and Firefox41. As we don't have a patch ready yet, leaving FF39 tracking unchanged. To be decided later.

Comment 7

[(no longer active)]

Attachment: Fix the :active pseudo-class on text controls

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d28aa0b0b47f

Comment 9

[(no longer active)]

Comment on attachment 8610986 [details] [diff] [review]
Fix the :active pseudo-class on text controls

Approval Request Comment
[Feature/regressing bug #]: Bug 924087
[User impact if declined]: See comment 0.
[Describe test coverage new/current, TreeHerder]: Locally, also has an automated test.
[Risks and why]: This is very low risk. 
[String/UUID change made/needed]: None.

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/d28aa0b0b47f

Comment 11

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8610986 [details] [diff] [review]
Fix the :active pseudo-class on text controls

Has test, should be safe, taking it for aurora & beta.

Comment 12

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/5d4c9dd10d5a

Comment 13

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/4f5b9c0de6ce

Comment 14

[Kate Glazko]

Added tracking for 39+

Comment 15

[Kohei Yoshino [:kohei]]

Posted the site compatibility doc for reference: https://www.fxsitecompat.com/en-US/docs/2015/active-pseudo-class-is-not-working-on-textbox/