Add Clipboard to Firefox will save in Private Browser

RESOLVED FIXED in Firefox 64

Status

()

P5
normal
RESOLVED FIXED
9 months ago
a month ago

People

(Reporter: combettaworld, Assigned: savvysiddharth, Mentored)

Tracking

58 Branch
Firefox 64
Points:
---

Firefox Tracking Flags

(firefox64 fixed)

Details

(Whiteboard: [lang=html])

Attachments

(3 attachments)

(Reporter)

Description

9 months ago
Add Clipboard to "Firefox will save" in Private Browser

Updated

9 months ago
Status: UNCONFIRMED → NEW
Ever confirmed: true

Updated

9 months ago
Mentor: josh
Whiteboard: [lang=html]
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

Comment 3

9 months ago
Hey,I would like work in this issue.if you could provide some insight about this issue,I would love work on this Thanks!
Flags: needinfo?(josh)
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.
Flags: needinfo?(josh)

Comment 5

9 months ago
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.

Comment 7

9 months ago
(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?

Comment 9

9 months ago
(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 hidden (mozreview-request)
Comment hidden (mozreview-request)

Updated

9 months ago
Attachment #8963297 - Flags: review?(josh)
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 14

8 months ago
mozreview-review
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)

Comment 15

4 months ago
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.
(Assignee)

Comment 17

3 months ago
Created attachment 9009360 [details]
Added firefox will save clipboard in private browsing

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.
(Assignee)

Comment 18

3 months ago
I've submitted changes, can you assign this to me?

Updated

3 months ago
Assignee: nobody → savvysiddharth
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!

Updated

3 months ago
Flags: needinfo?(josh)

Updated

3 months ago
Flags: needinfo?(josh)

Comment 21

3 months ago
Pushed by jmatthews@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0d3a741ff37c
Added firefox will save clipboard in private browsing r=jdm

Comment 22

3 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/0d3a741ff37c
Status: NEW → RESOLVED
Last Resolved: 3 months ago
status-firefox64: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64

Comment 23

3 months ago
The "copied text" sounds not including images, HTML, etc. I would rather it is the "Clipboard".
(Assignee)

Comment 24

3 months ago
"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?
Flags: needinfo?(josh)
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.
Flags: needinfo?(josh)
(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.