• sunbeam60@feddit.uk
    link
    fedilink
    arrow-up
    1
    ·
    edit-2
    4 days ago

    It drives me bonkers! The browser already has a way to display loading and it’s even respectful of back buttons.

    I get that in a select few cases, for real time content, it makes sense to handle the loading inside the page. But if all you’re doing is displaying an article, I don’t need you to load a framework page that then loads the content. Just load the content.

  • bleistift2@sopuli.xyz
    link
    fedilink
    English
    arrow-up
    1
    ·
    5 days ago

    If your element has an id, you can just reference it from the window scope. The const page = is useless. Also the body has its own reference under document: document.body replaces document.querySelector('body')

    • dan@upvote.au
      link
      fedilink
      arrow-up
      1
      ·
      5 days ago

      If your element has an id, you can just reference it from the window scope

      This is brittle, as defining a global variable with the same name (or the browser adding a API with the same name) will override it. This functionality was only kept for backwards compatibility with sites designed for Internet Explorer. The spec says to use getElementById instead.

    • funkless_eck@sh.itjust.works
      link
      fedilink
      arrow-up
      1
      ·
      4 days ago

      “make the page transparent and show a spinning icon, wait 750ms, then make the page display normally”

      it’s a fake loading screen