Closed
Bug 603688
Opened 14 years ago
Closed 14 years ago
copy about:support to pastebin
Categories
(Other Applications Graveyard :: Nightly Tester Tools, enhancement)
Other Applications Graveyard
Nightly Tester Tools
Tracking
(Not tracked)
RESOLVED
FIXED
1.0
People
(Reporter: aakashd, Assigned: harth)
References
Details
Attachments
(1 file)
10.88 KB,
patch
|
k0scist
:
review+
|
Details | Diff | Splinter Review |
I'd like to have a button that copies contents of about:support to pastebin...pwease?
Comment 1•14 years ago
|
||
Other information could be made paste-binnable as well
Reporter | ||
Updated•14 years ago
|
Product: Testing → Other Applications
Reporter | ||
Updated•14 years ago
|
Target Milestone: --- → 1.0
Comment 2•14 years ago
|
||
(In reply to comment #0) > I'd like to have a button that copies contents of about:support to > pastebin...pwease? Which optional parameters are needed? See parameters at http://pastebin.com/api.php So - Do NTT need the optional parameterN? (paste_name, paste_email, etc ...) - If needed then what about the default value for optional parameterN?
Reporter | ||
Comment 3•14 years ago
|
||
We don't need any optional parameters in there, just the one required parameter should be fine
Assignee | ||
Comment 4•14 years ago
|
||
This patch adds a 'Copy about:support to Pastebin' menuitem. It loads about:support into a hidden iframe and converts the HTML to text using the functions borrowed from about:support.
Assignee: nobody → fayearthur+bugs
Attachment #494248 -
Flags: review?(jhammel)
Comment 5•14 years ago
|
||
Comment on attachment 494248 [details] [diff] [review] add Copy about:support to pastebin menuitem to Firefox/Seamonkey menu Looks and works fine. You don't need the parent_id parameter in the request. So take that out. I'm not sure if the pastebinUrl should be a preference. It may only work with pastebin.com (it doesn't work with pastebin.mozilla.org, last i checked). If it is kept a preference it should probably be the whole url (including the path info). Or just taking it out is fine too. So nightlyPPrint is a little misleading: + // Trim extraneous whitespace before newlines, then squash extraneous + // blank lines. However, I consistently get two blank lines for every paste. Other than that, looks great!
Attachment #494248 -
Flags: review?(jhammel) → review+
Assignee | ||
Comment 6•14 years ago
|
||
thanks! https://github.com/mozautomation/nightlytt/commit/1d7bb7b5a397a2603ea83adf73854da778395727
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•