https://htmx.org logo
Join Discord
Powered by
# htmx-general
  • m

    mysterious-toddler-20573

    07/04/2020, 10:29 PM
    the history API is so crazy, I can understand it for about 30 seconds a week, so I have to code fast, and then it's gone
  • b

    big-airline-13935

    07/04/2020, 10:32 PM
    Testing...
  • m

    mysterious-toddler-20573

    07/04/2020, 10:32 PM
    hard one to test... :/
  • m

    mysterious-toddler-20573

    07/04/2020, 10:33 PM
    if it doesn't work I'll try to sit down w/ it tomorrow. Today is the day americans endanger their lives and the lives of everone around them with explosives because we didn't want to pay taxes
  • b

    big-airline-13935

    07/04/2020, 10:39 PM
    Did you perhaps miss the state in this line?
    Copy code
    if(htmx.config.historyEnabled) history.replaceState({}, getDocument().title, window.location.href);
  • b

    big-airline-13935

    07/04/2020, 10:40 PM
    https://github.com/bigskysoftware/htmx/blob/947a4470de7cca21479ed6f9a761d4bf56e451d9/www/js/htmx.js#L1069
  • m

    mysterious-toddler-20573

    07/04/2020, 10:43 PM
    yep
  • m

    mysterious-toddler-20573

    07/04/2020, 10:45 PM
    uh
  • m

    mysterious-toddler-20573

    07/04/2020, 10:45 PM
    also, my fix was in the wrong file
  • m

    mysterious-toddler-20573

    07/04/2020, 10:45 PM
    :/
  • m

    mysterious-toddler-20573

    07/04/2020, 10:45 PM
    One sec
  • b

    big-airline-13935

    07/04/2020, 10:45 PM
    With that (correctly) in place it is working in my manual tests.
  • m

    mysterious-toddler-20573

    07/04/2020, 10:46 PM
    OK, pushed up w/ that fix in /src properly
  • m

    mysterious-toddler-20573

    07/04/2020, 10:47 PM
    thank you for pointing that out, sorry for the trouble
  • b

    big-airline-13935

    07/04/2020, 10:48 PM
    Perfect, thanks for jumping on this so quickly @User! Back to comforting the cats and dogs at home 🎆
  • b

    big-airline-13935

    07/04/2020, 10:58 PM
    @User Could it be that this addition has broken the back button from using local storage?
  • b

    big-airline-13935

    07/04/2020, 11:05 PM
    Never mind, was testing old code, dammit.
  • u

    user

    07/05/2020, 11:32 AM
    > @User Probably wise to require a specific version of htmx at this early stage, e.g. https://unpkg.com/htmx.org@0.0.7 @User Yup, I'm all ok with it. It just was a moment when I was reading the documentation, and after ordinary page refresh suddenly something changed 😛 And I even couldn't figure out what exactly 😛
  • m

    mysterious-toddler-20573

    07/05/2020, 3:52 PM
    I think by the end of the month I'll have a 0.1.0 released and at that point the breaking changes will slow down (I hope)
  • m

    mysterious-toddler-20573

    07/05/2020, 3:52 PM
    still wondering about the colon-separated event namespace
  • m

    mysterious-toddler-20573

    07/06/2020, 4:24 PM
    thx @User for the fixes
  • m

    mysterious-toddler-20573

    07/06/2020, 4:24 PM
    I want to change
    view
    to
    show
  • m

    mysterious-toddler-20573

    07/06/2020, 4:24 PM
    I will cut a new release today w/ the fixes
  • b

    big-airline-13935

    07/06/2020, 4:25 PM
    @User Great! I'm also wondering about
    target.scrollTop = 0;
  • b

    big-airline-13935

    07/06/2020, 4:26 PM
    Doesn't seem to be working in my initial tests.
  • m

    mysterious-toddler-20573

    07/06/2020, 4:26 PM
    a manual test is probably in order
  • m

    mysterious-toddler-20573

    07/06/2020, 4:28 PM
    https://developer.mozilla.org/en-US/docs/Web/API/Element/scrollTop
  • m

    mysterious-toddler-20573

    07/06/2020, 4:28 PM
    0 would seem to be the correct value to have it scroll to the top
  • m

    mysterious-toddler-20573

    07/06/2020, 4:28 PM
    can try 1 as well
  • m

    mysterious-toddler-20573

    07/06/2020, 4:29 PM
    one confusing thing is that this is on the target element only, so the target needs to be scrollable, is that the issue?
1...789...1146Latest