Closed Bug 1372043 Opened 4 years ago Closed 4 years ago

Icon of minimize button disappeared (but button still works properly)


(Firefox :: Theme, defect, P1)

55 Branch
Windows 10



Firefox 56
56.3 - Jul 24
Tracking Status
firefox-esr52 --- unaffected
firefox54 --- unaffected
firefox55 --- unaffected
firefox56 --- fixed
firefox57 --- verified


(Reporter: blakewolf, Assigned: urbankrajnc92)



(Keywords: regression, Whiteboard: [photon-visual][p1])


(2 files, 3 obsolete files)

Attached image minimize.png
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:55.0) Gecko/20100101 Firefox/55.0
Build ID: 20170611030208

Steps to reproduce:

The minimize icon goes away when the browser is maximized.

You can either resize to browser to click the maximize button to shrink the window.

Actual results:

The icon goes away

Expected results:

The icon should always be visible
The minimize button disappears when Maximize -> normal in my case.

1. Start Browser with maximized mode
2. Click normal size mode button
3. Observe the minimized button

Regression window:

Regressed by: 806cf3746d11	UK992 — Bug 1185482 - Remove gap between titlebar buttons and navigation toolbar r=dao
Blocks: 1185482
Component: Untriaged → Theme
Ever confirmed: true
Flags: needinfo?(urbankrajnc92)
Flags: needinfo?(dao+bmo)
Keywords: regression
OS: Unspecified → Windows 10
I can't reproduce on 1.25 DPI. What is your DPI?
This could be solved with removing padding top and bottom, and only setting height.
Flags: needinfo?(urbankrajnc92)
Attached patch Possible solution (obsolete) — Splinter Review
Padding top and bottom is removed from titlebar buttons, except when windows is maximized and menu bar visible, because titlebar buttons are more flat.
Attached image Before and after patch (obsolete) —
Attachment #8876531 - Attachment is obsolete: true
Whiteboard: [photon-visual][triage]
Flags: qe-verify+
Priority: -- → P2
QA Contact: brindusa.tot
Whiteboard: [photon-visual][triage] → [photon-visual]
Flags: needinfo?(dao+bmo)
Whiteboard: [photon-visual] → [photon-visual][p1]
Assignee: nobody → urbankrajnc92
Attachment #8876532 - Attachment is obsolete: true
Attachment #8876529 - Attachment is obsolete: true
Priority: P2 → P1
Comment on attachment 8878218 [details]
Bug 1372043 - Set height on titlebar buttons only in miximized window

::: browser/base/content/browser-tabsintitlebar.js:179
(Diff revision 2)
> +      isPhotonWindows10 = (AppConstants.MOZ_PHOTON_THEME &&
> +                           AppConstants.isPlatformAndVersionAtLeast("win", "10.0"));
> +
> +      if (isPhotonWindows10) {
> +        // Remove height property from #titlebar-buttonbox to get right #titlebar-content height
> +        $("titlebar-buttonbox").style.removeProperty("height");

Thanks for looking into this. Unfortunately, this breaks the expected order described here:

      // Try to avoid reflows in this code by calculating dimensions first and
      // then later set the properties affecting layout together in a batch.

      // Begin setting CSS properties which will cause a reflow
Attachment #8878218 - Flags: review?(dao+bmo)
Duplicate of this bug: 1374358
This bug looks fixed as of the 2017-06-24 nightly build on my Windows 10 x64 machine.
I can still reproduce with 2017-06-26 nightly:

1. Maximize current window, and make sure the menu bar is hidden
2. Open a new (maximided) window with Ctrl+N
3. Unmaximize the new window, there is this problem.

But if the window is created non-maximized, then it works well.
Duplicate of this bug: 1376791
Summary: minimize line disappears, buts button still functions properly → Icon of minimize button disappeared (but button still works properly)
Duplicate of this bug: 1379609
Duplicate of this bug: 1379199
Hey UK92, are you still going to try to fix this or should we try to find a new owner?
Flags: needinfo?(urbankrajnc92)
Sorry i forgot about this bug.
Latest patch set height only when window is maximized.
Flags: needinfo?(urbankrajnc92)
Comment on attachment 8878218 [details]
Bug 1372043 - Set height on titlebar buttons only in miximized window

Thanks! I can't seem to reproduce the bug with or without this patch, but at least it shouldn't make things worse.
Attachment #8878218 - Flags: review?(dao+bmo) → review+
Pushed by
Set height on titlebar buttons only in miximized window r=dao
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Iteration: --- → 56.3 - Jul 24

status-ff56.0a1 : FIXED & VERIFIED.

Managed to reproduce the issue on Firefox Nightly under Windows 10 X 64.

The issue is no longer reproducible on Firefox latest Nightly [BuildID : 20170725030209 , 56.0a1(2017-07-25)(32 bit)]. 
Tests were performed under Windows 10 X 64.
Verified on Windows 10 x 64 and 32 bit on latest Nightly 57.0a1 Build ID 20170815100349 and i cannot reproduce the issue. I will mark this as verified fixed. Thank you
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.