[wp-trac] [WordPress Trac] #63275: Twenty Fourteen: Sticky menu problem

[WordPress Trac]

#63275: Twenty Fourteen: Sticky menu problem
-------------------------------------------------+-------------------------
 Reporter:  SergeyKovalets                       |       Owner:  (none)
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  Future
                                                 |  Release
Component:  Bundled Theme                        |     Version:
 Severity:  trivial                              |  Resolution:
 Keywords:  has-patch dev-feedback has-testing-  |     Focuses:  ui
  info                                           |
-------------------------------------------------+-------------------------
Changes (by SirLouen):

 * keywords:  has-patch needs-testing-info => has-patch dev-feedback has-
     testing-info
 * severity:  normal => trivial


Comment:

 == Test Report
 === Description
 🟠 This report validates that the indicated patch works as expected with
 some caveats

 Patch tested:
 https://core.trac.wordpress.org/attachment/ticket/63275/63275.diff

 === Environment
 - WordPress: 6.9-alpha-60093-src
 - PHP: 8.2.28
 - Server: nginx/1.27.5
 - Database: mysqli (Server: 8.4.5 / Client: mysqlnd 8.2.28)
 - Browser: Chrome 135.0.0.0
 - OS: Windows 10/11
 - Theme: Twenty Fourteen 4.2
 - MU Plugins: None activated
 - Plugins:
   * Test Reports 1.2.0

 === Testing Instructions
 1. Set up manually force the CSS `masthead` element ID to anything
 `height` over 48
 2. 🐞 The top menu header will not scroll.
 3. The reasons of why that height is over 48 are not many, but are. Don't
 try to reproduce this, moon, sun and Jupyter must be aligned the day you
 try it. Just trust.

 === Actual Results
 1. 🟠 The Patch solves the problem if the height is just under or equal
 49px. Still too precise for my pov.

 === Additional Notes
 @sabernhardt why not `mastehead` >= 50 or even 60? I think that, as-is,
 it's so prone to end with the scroll broken for whatever reason. And I
 think that setting it to a bigger size, will not do any harm. I believe
 that the sole idea here is to avoid sticking the menu if it happens to
 become double sized (two lines because of the menu items, for example). So
 I would rather choose a much wider range (60px for example would do the
 trick).

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/63275#comment:19>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform