Opened 8 months ago

Closed 8 months ago

#28543 closed defect (fixed)

Tor Browser's about:tor has horizontal scroll bar between widths 900px and 1000px

Reported by: pospeselr Owned by: tbb-team
Priority: Medium Milestone:
Component: Applications/Tor Browser Version:
Severity: Normal Keywords: TorBrowserTeam201811R
Cc: mcs, brade Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Content does not horizontally center correctly between widths 900px and 1000px

Child Tickets

Attachments (2)

Screenshot from 2018-11-20 15-58-13.png (112.7 KB) - added by pospeselr 8 months ago.
questions_on_onions.png (15.1 KB) - added by gk 8 months ago.

Download all attachments as: .zip

Change History (7)

Changed 8 months ago by pospeselr

comment:1 Changed 8 months ago by mcs

Keywords: TorBrowserTeam201811R added
Status: assignedneeds_review

comment:2 Changed 8 months ago by gk

Looks reasonable in principle. However, that results in our question link etc. laid over the onions

while before that the text was between the search field and the onions (which looks cleaner). Do we want that change?

Last edited 8 months ago by gk (previous) (diff)

Changed 8 months ago by gk

Attachment: questions_on_onions.png added

comment:3 Changed 8 months ago by gk

Cc: mcs brade added

comment:4 in reply to:  2 Changed 8 months ago by mcs

Status: needs_reviewneeds_information

Replying to gk:

Looks reasonable in principle. However, that results in our question link etc. laid over the onions

while before that the text was between the search field and the onions (which looks cleaner). Do we want that change?

Hmm. Is this behavior new? Kathy and I see this even without the change proposed in this ticket. Since the onions are anchored to the bottom of the window, when the window is very short the onions will appear behind the content (which seems OK to Kathy and me).

comment:5 Changed 8 months ago by gk

Resolution: fixed
Status: needs_informationclosed

I've never seen this behavior before to be honest. If that's not related to this ticket (even though without your patch I did not see it on my Windows 7 testing system), even better. :) So, I merged your branch to master (commit 9ae78d32be280bcf58819676f700994544049ae8), thanks.

Note: See TracTickets for help on using tickets.