Initial fontSize value is not updated and onStart/onFinish do not fire

[Nantris]

I'm not sure what the issue is, because we use useFitText in one other place and it works, but in this other circumstance it simply defaults to the specified maxFontSize, even if that means it wraps lines. It's the same whether we use a width specified in pixels or percentages, whether we set a height on the element, whether we set overflow: hidden - whether we change the text or resize the window. Nothing happens under any circumstances. Any thoughts?

[saltycrane]

hi @slapbox, I'm not sure if I can solve it, but if you can create a reproduction example by forking this Codesandbox https://codesandbox.io/s/use-fit-text-basic-example-qpo04?file=/pages/index.js it would be helpful.

[Nantris]

Hey @saltycrane thanks for your reply and for the great library!

I've been trying like hell to make a repro case, but no such luck. Do you have any "shot in the dark" ideas about things we might look into?

I've even tried poking around in the index.js of the library, although it's the minified version and I wasn't able to find any clues. I'm pretty much at a total loss for where to look for clues about either the cause or a possible solution.

All I was able to notice from setting breakpoints in the index.js is that it all runs when I'd expect - it just doesn't run the callbacks or update the font size.

[Nantris]

I just got it to work once, but I have no idea how. I was playing around with resize: both; overflow: hidden to allow manual resizing (which didn't trigger useFitText) but then at some point it suddenly began to, perhaps after a hot reload? I haven't been able to achieve that again though.

[saltycrane]

hi @slapbox thanks for trying to provide a repro example and trying to debug the code. You mentioned trying to use the minified version of the source code. You could also try copy and pasting the source code here into your project so you can debug the non-minified source code. So did you see that the useFitText hook was invoked but onStart was not? I would look at this code: https://github.com/saltycrane/use-fit-text/blob/master/index.ts#L64-L95 I think that code can actually be refactored to be simpler, but I'm not sure if that would affect your issue.

[nicolabosco87]

Maybe I have replicated this bug. Could not handle correctly conditioanl rendering. Added infos into a MR #29