Feature #9882

An Adblock tab opens on the first launch of Icedove

Added by kytv 2015-08-02 10:48:58 . Updated 2015-10-07 07:09:13 .

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Target version:
Start date:
2015-08-02
Due date:
% Done:

100%

Feature Branch:
kytv:feature/5663-return-to-icedove
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Email Client
Deliverable for:
268

Description

An adblock tab is opened when the Debian package is installed and Icedove is started for the first time.

How should this be resolved?

1. Do we just disable the tab’s opening?; or
2. do we disable adblock completely in Icedove?

Installing enigmail—which we’ll want to do as part of the migration—will switch Icedove’s default from “HTML” to “text only”.


Subtasks


History

#1 Updated by intrigeri 2015-08-03 04:30:16

  • blocks #8668 added

#2 Updated by intrigeri 2015-08-03 04:32:26

  • Assignee set to kytv
  • Type of work changed from Discuss to Code

> 2. do we disable adblock completely in Icedove?

I think we should do that. I see little point in having an ad blocker for a MUA that is configured to display the raw text version of email by default. But if this assumption of mine is incorrect, then go for the 1st option instead (as we do in Tor Browser already).

#3 Updated by sajolida 2015-08-04 06:41:38

I’ve got the same feeling. We don’t need AdBlock in Icedove, even keeping in mind that we mind then advertise it as a feed reader.

#4 Updated by kytv 2015-08-04 07:50:00

The three of us agree then. :)

#5 Updated by kytv 2015-09-28 03:54:03

  • Status changed from Confirmed to In Progress
  • Target version changed from 246 to Tails_1.7
  • % Done changed from 0 to 40

#6 Updated by kytv 2015-10-03 02:48:00

  • Assignee changed from kytv to anonym
  • QA Check set to Ready for QA
  • Feature Branch set to kytv:feature/5663-return-to-icedove

#7 Updated by anonym 2015-10-05 01:46:27

  • Assignee changed from anonym to kytv
  • QA Check changed from Ready for QA to Dev Needed

#8 Updated by kytv 2015-10-05 02:32:31

  • QA Check changed from Dev Needed to Ready for QA

#9 Updated by kytv 2015-10-05 02:35:43

  • Assignee changed from kytv to anonym

#10 Updated by kytv 2015-10-05 07:11:31

Applied in changeset commit:432ba8efefce14057bac9a30afb113f297bc9472.

#11 Updated by anonym 2015-10-07 07:09:13

  • Status changed from In Progress to Resolved
  • Assignee deleted (anonym)
  • % Done changed from 40 to 100
  • QA Check changed from Ready for QA to Pass

Looks good to me.