890910 - Reader Mode: Initial reader toolbar position is offset and not fixed to bottom of content window

Status: RESOLVED DUPLICATE of bug 923969

(Firefox for Android Graveyard :: Reader View, defect)

Description

[Aaron Train [:aaronmt]]

Attachment: Reader Mode toolbar offset positioning (Screenshot)

Currently, entering Reader Mode on the aforementioned bug URL yields the initial reader toolbar positioning to be offset

Build: Nightly (07/08)
Device: HTC One (Android 4.2.2)

Steps to Reproduce

1. Visit http://9to5mac.com/2013/07/05/apple-details-flexible-battery-packs-suitable-for-rumored-iwatch-product/

2. Enter Reader Mode

Expected Results

1. Initial positioning of the toolbar is fixed to the bottom of the content window

Actual Results

1. Initial positioning of the toolbar is offset above the bottom of the content window and not anchored to the bottom

Comment 1

[:Margaret Leibovic]

Is this a regression? This looks more like a layout problem with fixed position footers, rather than a reader mode specific problem.

Comment 2

[Aaron Train [:aaronmt]]

This is working-for-me testing on the same device on 07/28 Nightly.

Comment 4

[Aaron Train [:aaronmt]]

Peter is seeing this in the duplicate bug report above.

Comment 5

[Aaron Train [:aaronmt]]

Steps: http://mozilla.org, enter reader-mode

Comment 6

[Teodora Vermesan (:TeoVermesan)]

(In reply to Aaron Train [:aaronmt] from comment #5)
> Steps: http://mozilla.org, enter reader-mode

The regression window is:

mozilla-central
good build: 22.07.2013 
bad build: 23.07.2013 
-pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=e3c19a339b36&tochange=5ceea82a79c7
The tinderbox inbound builds are missing since the issue is older than one month.

Comment 7

[Kartikaya Gupta (email:[e-mail])]

Guessing this was caused by the fixed position changes, since that toolbar is a fixed position item.

Comment 8

[Aaron Train [:aaronmt]]

Double checking here. This is fixed on Nightly (27.0) and Aurora (26.0) but is still affecting Beta (25.0). Do we know which bug has been fixed that either got denied beta approval or could be requested for beta approval? Otherwise, shall we wontfix 25?

Comment 9

[Kartikaya Gupta (email:[e-mail])]

I don't know what fixed it. Probably the same thing that fixed bug 903473.

Comment 10

[Kartikaya Gupta (email:[e-mail])]

Duping to bug 923969 as that has more information about what fixed this and whether or not it should be uplifted.