1511170 - Port bug 1498448: Migrate Certificate Manager Dialog to use fluent for localization

Status: RESOLVED FIXED

(Thunderbird :: General, task)

Description

[Richard Marti (:Paenglab)]

We use strings from certManager.dtd in chat/content/account.xml and mailnews/extensions/newsblog/content/feed-subscriptions.xul. These need to be changed to use the fluent strings.

Comment 1

[Richard Marti (:Paenglab)]

Attachment: 1511170-cert-manager-fluent.patch

This works for me. Without patch, TB closes when I try to open the subscribe dialog and the Chat account dialog is almost empty.

With patch, the strings aren't shown for me in the dialogs. But when I inspect them with the inspector, I see them.

Comment 2

[Jorg K (CEST = GMT+2)]

Comment on attachment 9028773 [details] [diff] [review]
1511170-cert-manager-fluent.patch

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

(In reply to Richard Marti (:Paenglab) from comment #1)
> This works for me. Without patch, TB closes when I try to open the subscribe
> dialog and the Chat account dialog is almost empty.
That doesn't happen for me. For feed subscription, the label is on a collapsed button I've never seen, and in the chat AM I can't see that "Add Exception" button either.

::: chat/content/account.xml
@@ +2,5 @@
>  <!-- This Source Code Form is subject to the terms of the Mozilla Public
>     - License, v. 2.0. If a copy of the MPL was not distributed with this
>     - file, You can obtain one at http://mozilla.org/MPL/2.0/. -->
>  
> +<!DOCTYPE binding SYSTEM "chrome://chat/locale/accounts.dtd">

This doesn't look right. There is DOCTYPE bindings. Why not just remove the certManagerDTD line?

Comment 3

[Jorg K (CEST = GMT+2)]

Attachment: 1511170-cert-manager-fluent.patch - JK

I'll land this now. The strings are gone, so time to act.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/57a3525213c4
Port bug 1498448: Migrate Certificate Manager Dialog to use fluent for localization. r=jorgk

Comment 5

[Richard Marti (:Paenglab)]

(In reply to Jorg K (GMT+1) from comment #2)
> Comment on attachment 9028773 [details] [diff] [review]
> 1511170-cert-manager-fluent.patch
> 
> Review of attachment 9028773 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> (In reply to Richard Marti (:Paenglab) from comment #1)
> > This works for me. Without patch, TB closes when I try to open the subscribe
> > dialog and the Chat account dialog is almost empty.
> That doesn't happen for me. For feed subscription, the label is on a
> collapsed button I've never seen, and in the chat AM I can't see that "Add
> Exception" button either.

You haven't deleted the certManager.dtd in your obj-dir.