Closed Bug 772456 Opened 12 years ago Closed 12 years ago

Replace '-moz-transition' with 'transition' in toolkit/

Categories

(Toolkit :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla16

People

(Reporter: dao, Assigned: ananuti)

References

Details

(Whiteboard: [good first bug][mentor=dao][lang=css])

Attachments

(1 file)

OS: Windows 7 → All
Hardware: x86_64 → All
Assignee: nobody → ananuti
Status: NEW → ASSIGNED
Attached patch patchSplinter Review
apply on top of attachment 640887 [details] [diff] [review] bug 772462
Attachment #640888 - Flags: review?(dao)
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+
https://hg.mozilla.org/mozilla-central/rev/1fc8bc11a0cd
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.

Attachment

General

Created:
Updated:
Size: