Project

General

Profile

Bug #7400

Feature #7377: Improve Windows 8 camouflage

Don't start Vidalia in Windows 8 camouflage

Added by anonym over 5 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Low
Assignee:
-
Category:
Camouflage
Target version:
Start date:
06/11/2014
Due date:
% Done:

100%

Feature Branch:
kytv:bugfix/7400-dont_start_vidalia_if_camouflage_enabled
Type of work:
Code
Blueprint:
Starter:
No
Affected tool:

Description

The white box underneath is a nasty eye sore, so perhaps we should consider not running Vidalia in camouflage mode? After all, the onion icon is pretty suspicious on its own.

Associated revisions

Revision fc69cac3 (diff)
Added by kytv over 4 years ago

Don't start vidalia if Windows Camouflage is enabled

Will-fix: #7400

Revision f9372d5a
Added by anonym over 4 years ago

Merge remote-tracking branch 'kytv/bugfix/7400-dont_start_vidalia_if_camouflage_enabled' into stable

Fix-committed: #7400

History

#1 Updated by anonym over 5 years ago

  • Parent task set to #7377

#2 Updated by aboules over 5 years ago

I agree, maybe one could just change the icon so functionality stays the same for people new to Tails/Tor/Vidalia.
In the same idea it is IMHO not clever to name the Tor browser "Tor Browser" in the title bar.

#3 Updated by anonym over 5 years ago

aboules wrote:

I agree, maybe one could just change the icon so functionality stays the same for people new to Tails/Tor/Vidalia.

Change it to some other stock Windows icon you mean? If so, sure, that's another option.

In the same idea it is IMHO not clever to name the Tor browser "Tor Browser" in the title bar.

I thought I had already opened a ticket about this, but apparently I forgot it. Now filed as #7419.

#4 Updated by sajolida about 5 years ago

I think that I have the same white box underneath without the Camouflage as well.

#5 Updated by BitingBird over 4 years ago

We're planning on removing Vidalia, so I think this should be rejected (better to work on the replacement).

#6 Updated by sajolida over 4 years ago

We're now considering replacing it but keeping the traditional green onion as a permanent feedback that Tor is started and everything is going fine. So this discussion still holds. Marking it for the meeting tonight.

#7 Updated by intrigeri over 4 years ago

  • Affected tool set to Vidalia

#8 Updated by sajolida over 4 years ago

  • Priority changed from Elevated to Low
  • Type of work changed from Discuss to Code

During the 201503 meeting we agreed to make this a low priority wishlist ticket.

#9 Updated by BitingBird over 4 years ago

  • Subject changed from Don't start Vidalia in Windows 8 camouflage? to Don't start Vidalia in Windows 8 camouflage

#10 Updated by kytv over 4 years ago

  • Target version set to Tails_1.3.2
  • % Done changed from 0 to 50
  • QA Check set to Ready for QA
  • Feature Branch set to kytv:bugfix/7400-dont_start_vidalia_if_camouflage_enabled

#11 Updated by intrigeri over 4 years ago

  • Status changed from Confirmed to In Progress

#12 Updated by intrigeri over 4 years ago

  • Assignee set to alant

#13 Updated by alant over 4 years ago

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

The changelog looks OK.

However, this branch is based on devel, but targets a point release. This can't work. Either rebase it on stable or target the text major release (1.4).

#14 Updated by kytv over 4 years ago

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

Per intrigeri's comments on IRC, I rebased against stable (which at the moment is identical to the 1.3.1 tag) and force pushed.

#15 Updated by intrigeri over 4 years ago

Either rebase it on testing or target the text major release (1.4).

Alan, FYI testing is only used post-freeze for major releases.

#16 Updated by alant over 4 years ago

So the diff is now fine. However, I think that the lack of vidalia icon when using windows camouflage should be documented in https://tails.boum.org/doc/anonymous_internet/vidalia/index.en.html and/or https://tails.boum.org/doc/first_steps/startup_options/windows_camouflage/index.en.html before we can merge.

#17 Updated by alant over 4 years ago

  • Assignee changed from alant to kytv

#18 Updated by alant over 4 years ago

  • QA Check changed from Ready for QA to Dev Needed

#19 Updated by kytv over 4 years ago

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

Perhaps what I just pushed will be sufficient.

#20 Updated by BitingBird over 4 years ago

  • Assignee changed from alant to anonym
  • QA Check changed from Ready for QA to Pass

Reviewed the doc, please merge when 1.3.1 is out :)

#21 Updated by kytv over 4 years ago

#22 Updated by anonym over 4 years ago

  • Status changed from In Progress to Fix committed
  • % Done changed from 50 to 100

#23 Updated by anonym over 4 years ago

  • Assignee deleted (anonym)

#24 Updated by anonym over 4 years ago

  • Status changed from Fix committed to Resolved

#25 Updated by intrigeri over 3 years ago

  • Affected tool deleted (Vidalia)

Also available in: Atom PDF