[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #14429 [Tor Browser]: Automated rounding of content window dimensions



#14429: Automated rounding of content window dimensions
-------------------------+-------------------------------------------------
     Reporter:           |      Owner:  tbb-team
  arthuredelstein        |     Status:  needs_revision
         Type:  defect   |  Milestone:
     Priority:  normal   |    Version:
    Component:  Tor      |   Keywords:  tbb-fingerprinting-resolution, tbb-
  Browser                |  torbutton, TorBrowserTeam201502
   Resolution:           |  Parent ID:
Actual Points:           |
       Points:           |
-------------------------+-------------------------------------------------
Changes (by gk):

 * cc: gacar (added)


Comment:

 Replying to [comment:6 arthuredelstein]:
 > Here's a new version of the patch that auto-trims the window after the
 user resizes it. That way we don't have ugly gray bars taking up a lot of
 screen space.
 >
 >
 https://github.com/arthuredelstein/torbutton/commit/e05b071304c42d0f55770ef3fddd38953a53d4ee

 Looks much better, thanks! Two comments:

 1) I think we should keep the 200x100 rounding as we already do on start-
 up for now. Vertical space is precious (assuming we have a user with a
 horizontal display as we are currently doing). However, having some more
 data driven clues would be really nice. See:
 https://trac.torproject.org/13025#comment:17 for a beginning. As a side
 note: there has already been a wrestling in the past with rounding windows
 to 50px x 50 px: https://www.torproject.org/docs/torbutton/en/design/
 (section Resize windows to multiples of 50px during Tor usage
 (recommended)) + see #3506...

 2) Your patch re-rounded the already rounded window on start-up which is
 quite confusing: the properly rounded window shows up and then your logic
 kicks in making the window smaller. I think the resizing logic works on
 start-up in the most cases. Thus, what about only resizing again with your
 logic if there is no multiple of 200x100 after start-up? This should be
 easily doable and does not scare the user so much ("What the heck is going
 on with my browser windows here???") provided she already has rounded
 dimensions.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/14429#comment:12>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs