chore(docs): update code docs

Author: jellyrock[bot]

docs/components_ItemDetails.bs.html

@@ -12,7 +12,7 @@
 import "pkg:/source/utils/streamSelection.bs"
 
 ' Gap between bottom of itemInfoRows and top of extras pane when extras are open
-const ITEM_DETAILS_EXTRAS_PADDING = 48
+const ITEM_DETAILS_EXTRAS_PADDING = 24
 ' Minimum display height for the logo image — images too small are scaled up (aspect ratio preserved).
 const LOGO_MIN_DISPLAY_HEIGHT = 212 ' Note: LOGO_MAX_DISPLAY_WIDTH takes precedence for very wide/flat logos; this minimum may not be reached.
 ' Maximum display width for the logo image — prevents very wide/flat logos from overlapping buttons
@@ -59,7 +59,6 @@
   m.itemDetailsSlider = m.top.findNode("itemDetailsSlider")
   m.itemDetailsSliderInterp = m.top.findNode("itemDetailsSliderInterp")
   m.extrasActive = false
-  m.animationTargetCalculated = false
   m.extrasLoaded = false
 
   ' Inner group: title + infoGroup + directorGenreGroup (never changes during activate/deactivate)
@@ -1285,7 +1284,6 @@
 end function
 
 sub itemContentChanged()
-  m.animationTargetCalculated = false
   m.seasonSeriesData = invalid
   item = m.top.itemContent
 
@@ -1842,19 +1840,15 @@
 sub onItemDetailsRendered()
   if m.itemDetails.renderTracking = "none" then return
   updateTextGradient()
-  ' Only calculate animation target once per content load — itemInfoRows height is stable after metadata is set.
-  if not m.animationTargetCalculated
-    m.animationTargetCalculated = true
-    updateItemDetailsAnimationTarget()
-  end if
 end sub
 
 ' updateItemDetailsAnimationTarget: Slide itemDetails so the bottom of itemInfoRows sits just
 ' above the extras pane. boundingRect() on a child returns LOCAL coords relative to the parent's
 ' translation point, so we use translation[1] (not boundingRect().y) as the screen origin.
+' Called just-in-time from the DOWN key handler to guarantee the layout is fully settled
+' (text wrapping complete) before measuring. Eager calculation via renderTracking races
+' with async text layout, producing stale targets — especially for long descriptions.
 sub updateItemDetailsAnimationTarget()
-  if m.extrasActive then return
-
   currentTransY = m.itemDetails.translation[1]
   itemInfoRowsLocalRect = m.itemInfoRows.boundingRect()
   screenBottomOfInfoRows = currentTransY + itemInfoRowsLocalRect.y + itemInfoRowsLocalRect.height
@@ -2458,9 +2452,6 @@
 
   populateInfoGroupSeason(m.top.itemContent, m.global.user.settings)
 
-  ' Row heights changed — reset so onItemDetailsRendered() recalculates the extras animation target
-  m.animationTargetCalculated = false
-
   ' Update logo using series data if the season didn't carry parentLogoItemId
   if isValid(m.itemLogo) and not m.itemLogo.visible
     setItemLogo(m.seasonSeriesData)
@@ -2515,8 +2506,6 @@
   m.itemDescription.text = content[0]
   m.itemDescription.visible = true
   setDescriptionInLayout(true)
-  ' Row heights changed — reset so onItemDetailsRendered() recalculates the extras animation target
-  m.animationTargetCalculated = false
 end sub
 
 ' onFirstEpisodeLoaded: Start playback of first episode for Series Play button
@@ -2581,6 +2570,8 @@
     m.top.lastFocus = m.extrasGrid
     m.extrasGrid.setFocus(true)
 
+    ' Calculate animation target with the fully-settled layout before activating extras
+    updateItemDetailsAnimationTarget()
     activateExtras()
 
     ' Hide description and tracks before sliding up so they don't show during the transition