Closed Bug 1442845 Opened 2 years ago Closed 2 years ago
Add Clipboard to Firefox will save in Private Browser
59 bytes, text/x-review-board-request
59 bytes, text/x-review-board-request
46 bytes, text/x-phabricator-request
|Details | Review|
Add Clipboard to "Firefox will save" in Private Browser
Marking as P3 for general backlog. We will need a bit of UX guidance on terms to use: e.g., "Clipboard data" vs. "Copied Text"
Priority: -- → P5
Hey,I would like work in this issue.if you could provide some insight about this issue,I would love work on this Thanks!
The goal is to add the note about "copied text" to the dtd file linked in comment 1, and add the name of the new string to the xhtml file from comment 1 along with the other strings that list things that Firefox does not clear. The result should be visible when you open a private window in a modified build.
Please bear with me,Its my first contribution Thing that I understood from your explanation is like creating a variable name(ie., aboutPrivateBrowsing.note.copiedText) and save "Firefox will save in" and then add this variable to xhtml file that you mentioned in comment 1
That's the idea, yes.
(In reply to Josh Matthews [:jdm] from comment #6) > That's the idea, yes. Like you said,I made variable and added it to xhtml file in div that contains list elements of bookmarks and downloads What should i do next..
Have you tried running the build and opening a private window? Do the changes appear? If not, does `./mach build` make a difference?
(In reply to Josh Matthews [:jdm] from comment #8) > Have you tried running the build and opening a private window? Do the > changes appear? If not, does `./mach build` make a difference? Yes I have run build and then opened the private window. It worked and the changes appeared.
https://developer.mozilla.org/en-US/docs/Mercurial/Using_Mercurial#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F has documentation about how to submit your changes for review. If it's still unclear, you can get assistance from the #introduction channel on IRC: https://chat.mibbit.com/?url=irc%3A%2F%2Firc.mozilla.org%2F%23introduction
Comment on attachment 8963297 [details] Bug 1442845 - Add Clipboard to Firefox will save in Private Browser I'm on leave but I don't want to leave this hanging for a month. Ehsan, could you deal with this or redirect it to a frontend reviewer?
Attachment #8963297 - Flags: review?(josh) → review?(ehsan)
Comment on attachment 8963297 [details] Bug 1442845 - Add Clipboard to Firefox will save in Private Browser https://reviewboard.mozilla.org/r/232192/#review239452 Hi Nikhil, Thanks a lot for your patch! Please see my comment below about the code changes. Also, do you mind squashing your changes so that all changes show up in a single changeset (right now you have two changesets instead of one). You can do that using the histedit command as explained here: http://mozilla-version-control-tools.readthedocs.io/en/latest/hgmozilla/firefoxworkflow.html#changing-code-after-reviews Thanks! ::: browser/locales/en-US/chrome/browser/aboutPrivateBrowsing.dtd:28 (Diff revision 1) > <!ENTITY aboutPrivateBrowsing.info.saved.before "Firefox "> > <!ENTITY aboutPrivateBrowsing.info.saved.emphasize "will save"> > <!ENTITY aboutPrivateBrowsing.info.saved.after2 " your:"> > <!ENTITY aboutPrivateBrowsing.info.downloads "downloads"> > <!ENTITY aboutPrivateBrowsing.info.bookmarks "bookmarks"> > +<!ENTITY aboutPrivateBrowsing.info.copiedText "clipboards"> As Josh mentioned in comment 4, the desired text here is "copied text" not "clipboards".
Attachment #8963297 - Flags: review?(ehsan)
Hi, can I work on this?
(In reply to irina6793 from comment #15) > Hi, can I work on this? Yes! Go ahead an submit a patch that addresses comment 4 and comment 14.
Before this patch, default home page of private sessions didn't inform about saving clipboard text. Now it informs that "copied text" will be saved in private session.
I've submitted changes, can you assign this to me?
Comment on attachment 9009360 [details] Added firefox will save clipboard in private browsing Josh Matthews [:jdm] has approved the revision.
Attachment #9009360 - Flags: review+
As soon as my commit access is re-enabled I'll merge this. Thanks!
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/0d3a741ff37c Added firefox will save clipboard in private browsing r=jdm
The "copied text" sounds not including images, HTML, etc. I would rather it is the "Clipboard".
"Clipboard" seemed more appropriate to me as well, but I made changes according to things mentioned in comment 14, Can I make changes to the patch after it is closed?
I have reproduced this bug with Nightly 60.0a1 (2018-03-02)on Windows 7, 64 Bit! This bug's fix is verified with latest Nightly! Build ID 20180921100113 User Agent Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:64.0) Gecko/20100101 Firefox/64.0
QA Whiteboard: [bugday-20180919]
I think the current text is fine. I'm not convinced that "clipboard" is a term in broad usage outside of developers.
(In reply to Josh Matthews [:jdm] from comment #26) > I'm not convinced that "clipboard" is a term in broad usage outside of developers. yeah, clipboard might be confusing for non-technical users 🤔
You need to log in before you can comment on or make changes to this bug.