Opened 21 months ago

Closed 5 months ago

#17517 closed defect (fixed)

Consider using different color for "Add Exception"

Reported by: ilv Owned by: huyvq
Priority: Medium Milestone:
Component: Applications/Tor Messenger Version:
Severity: Minor Keywords: UX
Cc: arlolra, sukhbir Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

When trying to connect to an onion service I receive a certificate error, which is expected. However, I had a hard time trying to figure out what to do next, and the main reason was that I didn't notice the "Add Exception" option (See image). I'm not sure if this depends on my system's configuration, so I'm creating a ticket anyways.

No image "tor-messenger-add-exception.png" attached to Ticket #1

Child Tickets

Attachments (4)

tor-messenger-add-exception.png (30.1 KB) - added by ilv 21 months ago.
Screenshot
add-exception-red.png (11.2 KB) - added by huyvq 7 months ago.
add-exception-white.png (11.7 KB) - added by huyvq 7 months ago.
add-exception-hidden.png (13.6 KB) - added by huyvq 7 months ago.

Download all attachments as: .zip

Change History (20)

Changed 21 months ago by ilv

Screenshot

comment:1 Changed 21 months ago by ilv

  • Keywords UX added

comment:2 Changed 21 months ago by sukhbir

  • Owner set to sukhbir
  • Status changed from new to assigned

comment:3 Changed 21 months ago by arlolra

  • Priority changed from Low to Medium

comment:4 Changed 17 months ago by sukhbir

Suggestions for a different colour or other ideas to fix this are welcome. One way of fixing this can be to make the "add exception" a button instead of a link.

comment:5 Changed 17 months ago by arlolra

Just make it a darker blue than the highlight colour?

comment:6 Changed 7 months ago by arlolra

  • Owner sukhbir deleted

comment:7 Changed 7 months ago by arlolra

  • Status changed from assigned to new

comment:8 Changed 7 months ago by huyvq

  • Owner set to huyvq
  • Status changed from new to assigned

comment:9 Changed 7 months ago by arlolra

See #21017 about the highlight colour as well.

Changed 7 months ago by huyvq

Changed 7 months ago by huyvq

comment:10 Changed 7 months ago by huyvq

What do you think about these changes?

comment:11 Changed 7 months ago by huyvq

  • Cc arlolra sukhbir added

comment:12 Changed 7 months ago by arlolra

The white looks clearer than the red, but what happens when you remove the highlight? (ie. create a second account and click on that to remove the blue background)

comment:13 Changed 7 months ago by huyvq

The button is being hidden (It's not be affected by my patch). Please see the attachment.

Last edited 7 months ago by huyvq (previous) (diff)

Changed 7 months ago by huyvq

comment:14 Changed 7 months ago by arlolra

Thanks, ok, let's go with that.

comment:15 Changed 7 months ago by ilv

white looks good!

comment:16 Changed 5 months ago by arlolra

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.