Closed
Bug 772456
Opened 12 years ago
Closed 12 years ago
Replace '-moz-transition' with 'transition' in toolkit/
Categories
(Toolkit :: General, defect)
Toolkit
General
Tracking
()
RESOLVED
FIXED
mozilla16
People
(Reporter: dao, Assigned: ananuti)
References
Details
(Whiteboard: [good first bug][mentor=dao][lang=css])
Attachments
(1 file)
|
(deleted),
patch
|
dao
:
review+
|
Details | Diff | Splinter Review |
|
Reporter | |
Updated•12 years ago
|
OS: Windows 7 → All
Hardware: x86_64 → All
|
Assignee | |
Updated•12 years ago
|
Assignee: nobody → ananuti
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 1•12 years ago
|
||
Attachment #640888 -
Flags: review?(dao)
|
|
Reporter | |
Comment 2•12 years ago
|
||
Comment on attachment 640888 [details] [diff] [review]
patch
> #detail-contrib-btn:not(:active):hover {
> border-color: #4271FF;
> background-color: #0459F7;
> box-shadow: 0 1px 0 rgba(0, 0, 0, 0.1),
> 0 0 3.5px hsl(190, 90%, 80%);
>- -moz-transition: background-color .4s ease-in,
>- border-color .3s ease-in,
>- box-shadow .3s ease-in
>+ transition: background-color .4s ease-in,
>+ border-color .3s ease-in,
>+ box-shadow .3s ease-in
> }
Glad you updated the indentation. Not doing so is a common mistake for seemingly mechanic search-and-replace tasks.
There's a semicolon missing at the end (before and after your change). I'll add it when checking this in.
Attachment #640888 -
Flags: review?(dao) → review+
|
|
Reporter | |
Comment 3•12 years ago
|
||
Target Milestone: --- → mozilla16
|
||
Comment 4•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•