Closed Bug 1480038 Opened Last year Closed Last year

Update the "Continue" button in the new cert error pages to "Continue…"

Categories

(Firefox :: Security, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
Firefox 63
Tracking Status
firefox63 --- fixed

People

(Reporter: trisha, Assigned: trisha)

References

Details

Attachments

(1 file, 3 obsolete files)

No description provided.
Status: NEW → ASSIGNED
Attached patch bug1480038.patch (obsolete) — Splinter Review
Attachment #8996655 - Flags: review?(francesco.lodolo)
Comment on attachment 8996655 [details] [diff] [review]
bug1480038.patch

Review of attachment 8996655 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/locales/en-US/chrome/overrides/netError.dtd
@@ +10,4 @@
>  <!ENTITY returnToPreviousPage.label "Go Back">
>  <!ENTITY returnToPreviousPage1.label "Go Back (Recommended)">
>  <!ENTITY advanced.label "Advanced">
> +<!ENTITY continue.label "Continue…">

Sorry, this needs a new ID, since I assume we care for all localizations to update their translation accordingly.
https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_content_best_practices#Changing_existing_strings
Attachment #8996655 - Flags: review?(francesco.lodolo) → review-
Attached patch bug1480038.patch (obsolete) — Splinter Review
Thanks, that's what I suspected. Made the changes now.
Attachment #8996655 - Attachment is obsolete: true
Attachment #8996662 - Flags: review?(francesco.lodolo)
Comment on attachment 8996662 [details] [diff] [review]
bug1480038.patch

Review of attachment 8996662 [details] [diff] [review]:
-----------------------------------------------------------------

Is there a reason not to fix this one as well, and remove the old string completely?
https://searchfox.org/mozilla-central/rev/196560b95f191b48ff7cba7c2ba9237bba6b5b6a/browser/base/content/aboutNetError-new.xhtml#179
Attachment #8996662 - Flags: review?(francesco.lodolo) → review+
(In reply to Francesco Lodolo [:flod] from comment #4)
> Is there a reason not to fix this one as well, and remove the old string
> completely?
> https://searchfox.org/mozilla-central/rev/
> 196560b95f191b48ff7cba7c2ba9237bba6b5b6a/browser/base/content/aboutNetError-
> new.xhtml#179

Sorry, ignore this, brain clearly doesn't work well in warm weather, given that's the file you're already changing in this patch.

Having said that: if the old string (continue.label) is not used anymore, it should be removed from netError.dtd
https://searchfox.org/mozilla-central/search?q=continue.label&path=
Attached patch bug1480038.patch (obsolete) — Splinter Review
Attachment #8996662 - Attachment is obsolete: true
Keywords: checkin-needed
Hello,

I am getting this error when trying to apply your patch in order for it to be landed.

applying bug1480038.patch
patch failed, unable to continue (try -v)
patch failed, rejects left in working directory
errors during apply, please fix and qrefresh bug1480038.patch

Can you please take a look?
Flags: needinfo?(guptatrisha97)
I get this error when trying to import the patch

> abort: bad hunk #1: incomplete hunk

My gut instinct suggests that you edited the .patch file directly, instead of generating a new one from the tree?
Carrying r+ over from the previous patch. I have re-based it on Central and re-submitted it through MozReview to avoid confusion. Hope this works!
Flags: needinfo?(guptatrisha97)
Comment on attachment 8997342 [details]
Bug 1480038 - Update the Continue button in new cert-error pages

https://reviewboard.mozilla.org/r/261150/#review268192
Attachment #8997342 - Flags: review?(francesco.lodolo) → review+
Pushed by francesco.lodolo@mozillaitalia.org:
https://hg.mozilla.org/integration/autoland/rev/9243cab39914
Update the Continue button in new cert-error pages r=flod
https://hg.mozilla.org/mozilla-central/rev/9243cab39914
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a692112bf9b8
Update the 'Continue...' label in new cert-error pages. r=johannh
You need to log in before you can comment on or make changes to this bug.