Turbo scrolls to first empty id

[jmfederico]

I have noticed the following behavior:

If a site has html tags with empty id attributes like this: id="", then turbo will scroll to that element after a new page navigation occurs.

Leaving aside the fact that there should be no empty id's, maybe Turbo should only scroll to elements that actually have an id?

[sebyx07]

Temporary fix:

document.addEventListener("turbo:before-render", () =>{
  document.querySelectorAll("[id]").forEach((el) => {
    if(el.id === "") el.removeAttribute("id");
  })
});

[dhh]

This has been fixed.