Closed Bug 1168055 Opened 9 years ago Closed 9 years ago

Active Pseudo class not working for textbox in firefox 38

Categories

(Core :: Layout: Form Controls, defect)

38 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox39 + fixed
firefox40 + fixed
firefox41 + fixed

People

(Reporter: emailron1992, Assigned: ehsan.akhgari)

References

Details

(4 keywords)

Attachments

(2 files)

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
Component: Untriaged → Layout: Form Controls
Keywords: testcase
Product: Firefox → Core
Thanks for creating the Text case. issue is being reproduced on the same
Flags: needinfo?(ehsan)
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...
Status: UNCONFIRMED → NEW
Ever confirmed: true
[Tracking Requested - why for this release]: Web compat regression.
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.
Assignee: nobody → ehsan
Flags: needinfo?(ehsan)
Attachment #8610986 - Flags: review?(bugs) → review+
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.
Attachment #8610986 - Flags: approval-mozilla-beta?
Attachment #8610986 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/d28aa0b0b47f
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
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.
Attachment #8610986 - Flags: approval-mozilla-beta?
Attachment #8610986 - Flags: approval-mozilla-beta+
Attachment #8610986 - Flags: approval-mozilla-aurora?
Attachment #8610986 - Flags: approval-mozilla-aurora+
Added tracking for 39+
You need to log in before you can comment on or make changes to this bug.