Opened 8 years ago

Last modified 21 months ago

#4967 new enhancement

Hint new users on how the toolbar button works

Reported by: pde Owned by: pde
Priority: High Milestone:
Component: HTTPS Everywhere/EFF-HTTPS Everywhere Version:
Severity: Normal Keywords:
Cc: nullishzero@…, micahlee Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Often, if people encounter a broken ruleset, they do not realise they can turn it off from the toolbar button.

Our chrome code does a great job of pointing out that little button at install time. Our Firefox code should, too!

Child Tickets

Attachments (1)

hintUsers.jpg (119.3 KB) - added by nullishzero 6 years ago.
Potential Solution Image

Download all attachments as: .zip

Change History (16)

comment:1 Changed 8 years ago by pde

Type: defectenhancement

comment:2 Changed 7 years ago by pde

I think this bug is rather important, and we've made no progress in addressing it. Here are some ways we could try:

  1. Have those yellow popups, like the ones that NoScript uses out of the box. They're very annoying, and the first thing you want to do is turn them off, but people do notice them.
  2. Animate the HTTPS Everywhere logo when it's active. That will make it stand out more.
  3. Open a new tab after first install, which points out the menu and shows how to use it.

comment:3 Changed 6 years ago by nullishzero

Cc: nullishzero@… added

I've started work on a potential solution and am still messing around with which method might be best. Of course it might take a while until I can showcase something.

CCing myself.

Changed 6 years ago by nullishzero

Attachment: hintUsers.jpg added

Potential Solution Image

comment:4 Changed 6 years ago by nullishzero

One potential solution that I'm leaning towards is after startup, having the browser open a new tab and print a notification at the top. The new tab would point to "https://www.eff.org/https-everywhere/faq." For how this would look, I've added an attachment. Any thoughts?

As for the potential solution of animating the logo, the following issue would probably work (which should get the icon noticed): https://trac.torproject.org/projects/tor/ticket/4886.

comment:5 Changed 6 years ago by nullishzero

Sorry, to clarify, the new tab/notification would happen only after first install.

comment:6 Changed 6 years ago by pde

Cc: micahlee added
Milestone: HTTPS-E 4.0dev7

Micah, nullthree/nullishzero's nice starting patch for this has landed in master, but I think we should do a UI review to see what if any extra polish we can add.

comment:7 Changed 6 years ago by nullishzero

I'd be more than happy to help with any improvements or questions about my code :)

I've also been working on the related issue #4486, but progress has been a bit slow because of limited free time during the last few weeks.

comment:8 in reply to:  7 Changed 6 years ago by nullishzero

Replying to nullishzero:

I'd be more than happy to help with any improvements or questions about my code :)

I've also been working on the related issue #4486, but progress has been a bit slow because of limited free time during the last few weeks.

Correction: Issue #4886, not 4486

comment:9 Changed 6 years ago by pde

Hi Nullishzero,

I was just looking at this UI feature again in preparation for a 4.0dev7 release. I think the existing patch is good enough for that release, an improvement we could make is to align the text over on the right side of the notification box, and end it with an arrow of some sort that points upward. It'll require some multi-platform testing to be certain, but I think in most or all cases that arrow will wind up pointing at the HTTPS Everywhere button that's just been added to the toolbar.

I'm not sure if that's possible with simple XUL CSSery, or whether we'll need to use this sort of method. I just tried this, but it didn't seem to work.

comment:10 in reply to:  9 Changed 6 years ago by nullishzero

Replying to pde:

Hi Nullishzero,

I was just looking at this UI feature again in preparation for a 4.0dev7 release. I think the existing patch is good enough for that release, an improvement we could make is to align the text over on the right side of the notification box, and end it with an arrow of some sort that points upward. It'll require some multi-platform testing to be certain, but I think in most or all cases that arrow will wind up pointing at the HTTPS Everywhere button that's just been added to the toolbar.

I'm not sure if that's possible with simple XUL CSSery, or whether we'll need to use this sort of method. I just tried this, but it didn't seem to work.

Hi pde,

I'll take a look into seeing if I can make the improvement suggested. I'm hoping it's possible with simple XUL CSSery, but I wouldn't mind writing a custom notification if needed.

Give me a few days, and I'll see if I can put something together by Monday (05/11/13). I'll test it on Windows XP/7 and GNU/Linux (I'm on Gentoo).

comment:11 Changed 6 years ago by nullishzero

Hmm... looks like aligning the text right isn't as trivial as I thought. There is a CSS trick you could do to align the text right, but it doesn't work well; the alignment is closer to the center than to the right. Also, based on my testing, having an arrow may not point correctly if there are other plugins (e.g. ghostery + firebug).

Another potential solution might be to have a button on the notification...

comment:12 in reply to:  11 Changed 6 years ago by pde

Replying to nullishzero:

Also, based on my testing, having an arrow may not point correctly if there are other plugins (e.g. ghostery + firebug).

This is annoying :(

I wonder if there's a way to use CSS introspection to figure out how far from the right hand side we are, and set a corresponding offset. Micah, do you have any ideas about that?

Another potential solution might be to have a button on the notification...

Yes but since we don't want to have the notification open all the time, that doesn't seem like a good idea. NoScript does something like this by default, and I find it Extremely Annoying because it takes up so much screen space.

comment:13 Changed 6 years ago by pde

Micah, should we close this bug, or should we try harder to some kind of XUL/CSS introspection to figure out where the toolbar button is, and point at it?

comment:14 in reply to:  13 Changed 6 years ago by micahlee

Replying to pde:

Micah, should we close this bug, or should we try harder to some kind of XUL/CSS introspection to figure out where the toolbar button is, and point at it?

pde, have you merged nullishzero's commits? If you have, I think just showing the number of rules is good and we can close this bug.

comment:15 Changed 21 months ago by teor

Severity: Normal

Set all open tickets without a severity to "Normal"

Note: See TracTickets for help on using tickets.