Skip to content

TODO_TURBO_WARNING.md

Latest commit

 

History

History
29 lines (23 loc) · 1.57 KB

TODO_TURBO_WARNING.md

File metadata and controls

29 lines (23 loc) · 1.57 KB

TODO: Fix Turbo Warning in Future PR

Issue

There's a console warning about Turbo being loaded from within the <body> element instead of the <head>.

Root Cause

Conflicting requirements between three systems:

  1. Turbo - Wants to be loaded in <head> to avoid re-evaluation on page changes
  2. Shakapacker - Requires all append_javascript_pack_tag calls to happen before the final javascript_pack_tag
  3. React on Rails - The react_component helper internally calls append_javascript_pack_tag when rendering components in the body

Attempted Solutions That Failed

  1. Moving javascript_pack_tag to head - Breaks because react_component calls come after it
  2. Using data-turbo-suppress-warning - Doesn't properly suppress the warning

Potential Future Solutions

  1. Extract Turbo into a separate pack from stimulus-bundle and load it in the head
  2. Use prepend_javascript_pack_tag instead of append for component registration
  3. Configure React on Rails v16 to use a different component loading strategy
  4. Investigate if the auto-registration feature has a different recommended pack loading pattern

Current State

The application works correctly with the pack tags at the end of the body. The Turbo warning is cosmetic and doesn't affect functionality.

References