https://htmx.org logo
Join Discord
Powered by
# ye-olde-intercoolerjspth
  • m
    we could cut a new release (!!!) to make this configurable
  • a
    Ha!
  • a
    It might be a complete misnomer, but we're having users suddenly getting "localstorage full" errors in Chrome the last 2 months or so... and when we look at it, it looks like that biggest chunk is about 30 (??) ic-hist-elt ... entries
  • a
    We're also not far away from replacing that intercooler app with the migrated htmx one, so whatever we can do that's quick and easy as an interim fix, will be great 😅
  • a
    @User ☝️
  • m
    yeesh
  • m
  • m
    have you seen this?
  • m
    is this on mobile?
  • m
    Would be happy to help troubleshoot
  • a
    Oh no I didn't know about that, we just grabbed it manually. It happens so sporadically that we've only managed to grab it once. And no it's on desktop (every time that know of, Mac laptops now that I think of it)
  • a
    Like I said.. we've been running intercooler in that app for years, and it only started happening now recently, so I reckon something else is at play... a quick fix might be to limit the history entries to 10 only. But we'll dig a bit more too
  • m
    OK, how do you deploy intercooler? Do you have a local copy?
  • a
    Yes, "special build" even 🙈 ... not that I can for the life of me remember what we fixed/changed in that special build anymore "intercooler-1.2.3.special-build.js"
  • a
    I think it was something you changed way bug when, but it wasn't released yet
  • a
    I'll do a diff quick to see what we changed in that
  • a
    Oh - this is what we did: "/* COMMSPACE - Early fix release, containing support for calling scoped methods in ic-action */"
  • a
    This was the change it seems - to make us be able to call scoped Js functions in ic-action, e.g. 
    ic-action="MyScope.doSomething"
  • a
    We can just change the 
    20
    to a 
    10
    I guess in our "special build" ... that would be a very quick fix 🙂, ie.
    Copy code
    function getSlotLimit() {
    return 20; // We could change this to 10
  }
  • m
    yep
  • m
    that's the line
  • a
    🙏
  • m
    just popping in to say that intercooler.js is still supported, still awesome, and I'm willing to help anyone who has issues with it
  • j
  • c
    when you spot someone's library used in the wild https://balto.baltorepo.com/legal/
  • m
    Ha!
  • g
    I was generating some stats about my codebase as work and counted 1,539 
    ic-*
    attributes in our HTML templates. Can anyone beat that? 🙂
  • m
    the original app built w/ intercooler: 659 instances
  • m
    adam wins
  • s
    Just a quick heads-up regarding the PR I made in https://github.com/bigskysoftware/intercooler-js/pull/340, since it's been almost a month with no feedback (no big deal, its a small fix — just making sure it gets seen)