Closed Bug 948897 Opened 6 years ago Closed 6 years ago
Move inline scripts and styles into separate file for mobile/android/chrome/content/about
Feedback .xhtml (URL=about:feedback)
Hi, I'd like to pick this one up as my first ticket. I guess I'm facing a similar issue to others on the various sub-parts though in that I can't seem to find anyway to assign it to myself to prevent duplicate efforts.
patch for separating out the inline js previously found in aboutFeedback.
Comment on attachment 8355010 [details] [diff] [review] bug948897.patch Thanks! If you're ready for this patch to land, you can follow the instructions here and add "checkin-needed" to the "Keywords" field of this bug: https://developer.mozilla.org/en-US/docs/Creating_a_patch_that_can_be_checked_in
Attachment #8355010 - Flags: review?(mbrubeck) → review+
Hi, I think I maybe lack the required edit bug privileges to add the 'checkin-needed' flag for this. Not sure where to go from here, do I need to ask someone else to do something further with this?
Thank you for working on this patch, Neil. Neil, please change the patch so that it can be checked in (see Matt's link), I think you mostly need the r=mbrubeck part, if I'm not mistaken. After that you should be able to add "checkin-needed" in the Keyword field. If not, please ping again and I will do so for your.
As a follow-up, we should also file a bug to remove the inline onload/onunload/onclick/onsubmit calls in aboutFeedback.xhtml, and use JS event listeners instead.
Whiteboard: [email@example.com][lang=html][good first bug][lang=js] → [firstname.lastname@example.org][lang=html][good first bug][lang=js][fixed-in-fx-team]
(In reply to :Margaret Leibovic from comment #7) > As a follow-up, we should also file a bug to remove the inline > onload/onunload/onclick/onsubmit calls in aboutFeedback.xhtml, and use JS > event listeners instead. Oh, thanks for pointing that out! :) I have filed bug 957052 to track this.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Fixed = on m-c. Please don't resolve bugs if they haven't landed there yet.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Closed: 6 years ago → 6 years ago
Resolution: --- → FIXED
Whiteboard: [email@example.com][lang=html][good first bug][lang=js][fixed-in-fx-team] → [firstname.lastname@example.org][lang=html][good first bug][lang=js]
Target Milestone: --- → Firefox 29
You need to log in before you can comment on or make changes to this bug.