[tor-bugs] #12684 [Firefox Patch Issues]: Make "Not Now" the default button for TorBrowser's canvas permission dialogue

[Tor Bug Tracker & Wiki]

#12684: Make "Not Now" the default button for TorBrowser's canvas permission
dialogue
-------------------------------------+-------------------------------------
     Reporter:  isis                 |      Owner:  mikeperry
         Type:  defect               |     Status:  needs_review
     Priority:  critical             |  Milestone:
    Component:  Firefox Patch        |    Version:
  Issues                             |   Keywords:  tbb-usability, tbb-
   Resolution:                       |  linkability, MikePerry201407R,
Actual Points:                       |  TorBrowserTeam201407
       Points:                       |  Parent ID:
-------------------------------------+-------------------------------------

Comment (by isis):

 Replying to [comment:14 gk]:
 > Looks good to me. Thanks isis! The only thing I am a bit unhappy with is
 that you now have the "Not now" option twice available if the drop down
 menu is open. That might be confusing to some users ("Why is there an
 additional 'Not now' and what is it doing compared to the already selected
 one?"). But I am happy if that is handled in a different bug if that is
 bothering users at all...

 Thanks! But... ''le sigh.''

 I couldn't figure out the `PopupNotification.jsm` API very well... I was
 never able to determine where the default Mozilla `[x] Not now` array item
 was coming from, nor how to disable/change it.

 I'd prefer to just do it right, and not substitute one confusing UI for a
 slightly-less-confusing UI. Do you know where the default
 `secondaryActions` array item comes from, gk?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/12684#comment:15>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online