loggerhead-breezy

Line numbers don't align on iPhone

Bug #940923 reported by Brian de Alwis on 2012-02-25

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	loggerhead	Triaged	Low	Unassigned
	loggerhead-breezy	Triaged	Low	Unassigned

Bug Description

[This is against Loggerhead as used on Launchpad.]

I had a bug report against bzr-tiplog with a stack trace, and wanted to look at the corresponding code using my iPhone. I brought up the code:

http://bazaar.launchpad.net/~slyguy/bzr-tiplog/trunk/view/head:/commands.py

But as shown on the attached screenshot, the line numbers do not match up at all!

Tags:

Revision history for this message

Brian de Alwis (slyguy) wrote on 2012-02-25:

Screen capture showing misaligned line numbers Edit (66.1 KiB, image/png)

Revision history for this message

Laura Czajkowski (czajkowski) wrote on 2012-02-27:

At present we don't support phone browsers but this is something that may be looked at in the future, but if you wanted to get involved in development, we do welcome patches and involvement.

Changed in loggerhead:
status:	New → Triaged
importance:	Undecided → Low
tags:	added: browser

Revision history for this message

Brian de Alwis (slyguy) wrote on 2012-06-06:

Patch to view.css Edit (451 bytes, text/plain)

I reported this on the Apple bugreporter and received the following reply:

-----------------------
This is a follow-up to Bug ID# 10937310.

Engineering has determined that this issue behaves as intended based on the following information:

This is text autosizing. If this behavior isn't appropriate for some web content, you can control this using the -webkit-text-size-adjust property.

See this article for more info: https://developer.apple.com/library/ios/#DOCUMENTATION/AppleApplications/Reference/SafariWebContent/AdjustingtheTextSize/AdjustingtheTextSize.html
-----------------------

The attached patch modifies view.css and fixes the problem in my testing. Note that testing requires a file with > 200 lines — the mismatch only starts to show around line 100 or so.

Jelmer Vernooij (jelmer) on 2018-10-20