Closed Bug 955738 Opened 10 years ago Closed 9 years ago

Port 'Bug 733632 - Remove TLS version UI (Options > Advanced > Encryption > Protocols)'

Categories

(Instantbird Graveyard :: Preferences, defect)

x86
Other
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Instantbird 42

People

(Reporter: florian, Assigned: clokep)

References

Details

(Whiteboard: [1.6-blocking])

Attachments

(1 file, 1 obsolete file)

*** Original post on bio 2286 at 2013-12-15 12:48:00 UTC ***

This UI was changed in Mozilla 23. We should port it, but it's not a pure string removal (there's one string changed/added) so we can't do it for 1.5. It's also not a completely broken UI, so it's OK to do it at the next cycle.
Blocks: 955469
Whiteboard: [1.6-wanted]
Whiteboard: [1.6-wanted] → [1.6-blocking]
Assignee: nobody → clokep
Status: NEW → ASSIGNED
Attached patch Patch v1 (obsolete) — Splinter Review
I did this by copying the relevant files from browser and then stripping out various components we don't use.

I tested this by clicking around and ensuring the prefs updated properly.
Attachment #8638492 - Flags: review?(florian)
Comment on attachment 8638492 [details] [diff] [review]
Patch v1

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

I'm not sure the "Data Choices" tab makes sense for Instantbird.

::: im/content/preferences/connection.js
@@ +105,5 @@
>      var proxyTypeCur = document.getElementById("network.proxy.type").value;
>  
> +    var prefs =
> +        Components.classes["@mozilla.org/preferences-service;1"].
> +        getService(Components.interfaces.nsIPrefBranch);

This change doesn't seem desirable.

::: im/locales/en-US/chrome/instantbird/preferences/advanced.dtd
@@ +7,5 @@
>  <!ENTITY generalTab.label                "General">
>  
>  <!ENTITY accessibility.label             "Accessibility">
>  
> +<!ENTITY useCursorNavigation.label       "Always use the cursor keys to navigate within pages">

Please revert this to the Instantbird wording talking about conversations.

@@ +12,4 @@
>  <!ENTITY useCursorNavigation.accesskey   "c">
>  <!ENTITY searchStartTyping.label         "Search for text when I start typing">
>  <!ENTITY searchStartTyping.accesskey     "x">
> +<!ENTITY blockAutoRefresh.label          "Warn me when websites try to redirect or reload the page">

Doesn't seem relevant to Instantbird.

@@ +17,2 @@
>  
> +<!ENTITY browsing.label                  "Browsing">

This change doesn't seem intentional.

@@ +36,5 @@
>  <!ENTITY configEdit.accesskey            "E">
>  
> +<!ENTITY dataChoicesTab.label            "Data Choices">
> +
> +<!ENTITY crashReporterDesc.label         "&brandShortName; submits crash reports to help &vendorShortName; make your browser more stable and secure">

This string isn't appropriate for Instantbird.
Attachment #8638492 - Flags: review?(florian) → feedback+
Attached patch Patch v2Splinter Review
I ended up removing the data choices tab, so that took care of a few comments.

I did not change to using Services.prefs in connection.js since we had modified this file to add imServices at the top. It seems better to keep it in sync with browser.

I made the request string changes and reverted some XUL around the conversation options group.
Attachment #8638492 - Attachment is obsolete: true
Attachment #8638510 - Flags: review?(florian)
Attachment #8638510 - Flags: review?(florian) → review+
https://hg.mozilla.org/comm-central/rev/0b348fa4d344
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.6
Updating milestone from 1.6 to 42.
Target Milestone: 1.6 → Instantbird 42
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: