- Issue created by @mherchel
- Assigned to mherchel
- @mherchel opened merge request.
- Status changed to Needs review
about 1 year ago 5:34pm 13 September 2023 - πΊπΈUnited States mherchel Gainesville, FL, US
This is ready for review. Note I'd like to get this in ASAP, as any further JS work will conflict with this.
- Status changed to RTBC
about 1 year ago 8:49pm 13 September 2023 - πͺπΈSpain ckrina Barcelona
I've tested it and it works the same as it did, so no bugs introduced. Plus the code looks good. I'd rather merge this and work on top of that for any further change. Thanks Mike!
- π·πΈSerbia finnsky
I would also manage black hover text with floating UI here.
Because now when we put submenu to fixed position it works fine with sidebar overflow: hidden
But that black text is not - πͺπΈSpain ckrina Barcelona
@finnsky I'd keep improvements to smaller follow-ups to avoid conflicts, and leave this to a refactor :)
-
mherchel β
committed dbba902c on 1.x
Issue #3386509 by mherchel, ckrina, claireristow: Refactor JavaScript
-
mherchel β
committed dbba902c on 1.x
- Status changed to Fixed
about 1 year ago 12:39pm 14 September 2023 Automatically closed - issue fixed for 2 weeks with no activity.