Closed Bug 1272677 Opened 5 years ago Closed 5 years ago

Two parallel lines are displayed in "Aa" floating layout button in reader view

Categories

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

49 Branch
ARM
Android
defect

Tracking

(firefox48 unaffected, firefox49 verified, fennec49+)

VERIFIED FIXED
Firefox 49
Tracking Status
firefox48 --- unaffected
firefox49 --- verified
fennec 49+ ---

People

(Reporter: TeoVermesan, Assigned: danhuang)

References

Details

Attachments

(2 files)

Steps to reproduce:
1. Open an article and enter reader view
2. Tap the "Aa" button

Actual results:
- Two parallel lines are displayed between "Aa" and "Auto" in the "Aa" floating layout button
Is this a recent regression? I'm worried this could be caused by desktop reader view changes.
(In reply to :Margaret Leibovic from comment #1)
> Is this a recent regression? I'm worried this could be caused by desktop
> reader view changes.

Maybe... I don't think there have been that many recently... at least, not that I would assume would affect layout, in the last week or two, that also haven't been uplifted. :-\

A regression window would be very helpful.
(In reply to Teodora Vermesan (:TeoVermesan) from comment #3)
> Regression:
> 10-05 not affected
> 11-05 affected
> 
> pushlog: 
> http://hg.mozilla.org/mozilla-central/
> pushloghtml?fromchange=043082cb7bd8490c60815f67fbd1f33323ad7663&tochange=674a
> 552743785c28c75866969aad513bd8eaf6ae

Thanks! That's bug 1269521 then... though looking at the markup:

         </div>
         <hr></hr>
         <div id="content-width-buttons">
           <button id="content-width-minus" class="content-width-minus-button"/>
           <button id="content-width-plus" class="content-width-plus-button"/>
         </div>
         <hr></hr>
+        <div id="line-height-buttons">
+          <button id="line-height-minus" class="line-height-minus-button"/>
+          <button id="line-height-plus" class="line-height-plus-button"/>
+        </div>
+        <hr></hr>
         <div id="color-scheme-buttons"></div>

not sure why the added <hr>s are showing up on Android but the pre-existing ones don't. Margaret?
Blocks: 1269521
Flags: needinfo?(margaret.leibovic)
Priority: -- → P2
(In reply to :Gijs Kruitbosch from comment #4)
> (In reply to Teodora Vermesan (:TeoVermesan) from comment #3)
> > Regression:
> > 10-05 not affected
> > 11-05 affected
> > 
> > pushlog: 
> > http://hg.mozilla.org/mozilla-central/
> > pushloghtml?fromchange=043082cb7bd8490c60815f67fbd1f33323ad7663&tochange=674a
> > 552743785c28c75866969aad513bd8eaf6ae
> 
> Thanks! That's bug 1269521 then... though looking at the markup:
> 
>          </div>
>          <hr></hr>
>          <div id="content-width-buttons">
>            <button id="content-width-minus"
> class="content-width-minus-button"/>
>            <button id="content-width-plus"
> class="content-width-plus-button"/>
>          </div>
>          <hr></hr>
> +        <div id="line-height-buttons">
> +          <button id="line-height-minus" class="line-height-minus-button"/>
> +          <button id="line-height-plus" class="line-height-plus-button"/>
> +        </div>
> +        <hr></hr>
>          <div id="color-scheme-buttons"></div>
> 
> not sure why the added <hr>s are showing up on Android but the pre-existing
> ones don't. Margaret?

I see we do have a style in place to hide these <hr>s:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/themes/core/aboutReaderControls.css#130

Not sure why that's not being picked up...

Dan, can you own fixing this?
tracking-fennec: --- → 49+
Flags: needinfo?(margaret.leibovic) → needinfo?(dhuang)
Assignee: nobody → dhuang
Status: NEW → ASSIGNED
Flags: needinfo?(dhuang)
Comment on attachment 8757238 [details]
MozReview Request: Bug 1272677 - Hide content width setting and line-height setting buttons in android's reader mode control panel. r?Margaret

https://reviewboard.mozilla.org/r/55770/#review52910
Attachment #8757238 - Flags: review?(margaret.leibovic) → review+
Thanks for the review.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/dd9e8143abd8
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Verified as fixed using:
Device: One A2001 (Android 5.1.1)
Build: Firefox for Android 49.0a1 (2016-06-01)
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.