Closed
Bug 994307
Opened 10 years ago
Closed 10 years ago
Remove the linear-gradients -moz- prefixes in from findbar.css
Categories
(Toolkit :: Themes, defect)
Toolkit
Themes
Tracking
()
RESOLVED
FIXED
mozilla31
People
(Reporter: ntim, Assigned: lviknesh)
Details
(Whiteboard: [good first bug][mentor=?][lang=css])
Attachments
(1 file)
1.63 KB,
patch
|
ntim
:
review+
dao
:
review+
|
Details | Diff | Splinter Review |
These files have -moz- prefixes : http://mxr.mozilla.org/mozilla-central/source/toolkit/themes/windows/global/findBar.css http://mxr.mozilla.org/mozilla-central/source/toolkit/themes/linux/global/findBar.css I'm not a toolkit/theme peer, so I can't mentor this bug. But I'd be welcome to help.
Assignee | ||
Comment 1•10 years ago
|
||
Sir , can i get this bug assigned . And i need to just remove moz prefix in all properties ?
Reporter | ||
Comment 2•10 years ago
|
||
(In reply to vikneshwar from comment #1) > Sir , can i get this bug assigned . And i need to just remove moz prefix in > all properties ? Not all properties, just the linear-gradients.
Assignee: nobody → lviknesh
Reporter | ||
Updated•10 years ago
|
Summary: Remove the -moz- prefixes from findbar.css → Remove the linear-gradients -moz- prefixes in from findbar.css
Reporter | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Reporter | ||
Comment 3•10 years ago
|
||
Here are some tutorials to set up mercurial, create patches and other things : http://codefirefox.com Now keep in mind that this is just the linear-gradients (sorry I haven't specified in the title).
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8404806 -
Flags: review?(ntim007)
Reporter | ||
Comment 5•10 years ago
|
||
Comment on attachment 8404806 [details] [diff] [review] Removed -moz- prefix from linear-gradient Looks good to me :) Now, since I'm not a peer, I can't allow this to land. Asking review from peer.
Attachment #8404806 -
Flags: review?(ntim007)
Attachment #8404806 -
Flags: review?(dao)
Attachment #8404806 -
Flags: review+
Updated•10 years ago
|
Component: XUL Widgets → Themes
Updated•10 years ago
|
Attachment #8404806 -
Flags: review?(dao) → review+
Reporter | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 6•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/b9ef681ba11c
Keywords: checkin-needed
Whiteboard: [good first bug][mentor=?][lang=css] → [good first bug][mentor=?][lang=css][fixed-in-fx-team]
Comment 7•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/b9ef681ba11c
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [good first bug][mentor=?][lang=css][fixed-in-fx-team] → [good first bug][mentor=?][lang=css]
Target Milestone: --- → mozilla31
You need to log in
before you can comment on or make changes to this bug.
Description
•