@anthony-nicholls I have some more test results re: resizing performance. DO you want it here or a new thread? I feel like I’m derailing this one.
new thread here:
Bumping as I’ve also noticed some pretty rough stutters with TextEditor as of JUCE 8.
I tried to follow along everything going on above so hopefully I didn’t miss this being addressed.
I’m doing a function like such
//In an overriden version of TextEditor
void insertTextAtCaret(const String& text, const Font& textFont, const Colour& textColour)
{
setFont(textFont);
setColour(TextEditor::textColourId, textColour);
TextEditor::insertTextAtCaret(text);
};
And then in another function like such.
text.insertTextAtCaret("Lead Programmer / Creator", lf, tc);
text.insertTextAtCaret("\n\n -Text\n\n\n", sf, tc);
text.insertTextAtCaret("Interface Graphic Design", lf, tc);
text.insertTextAtCaret("\n\n -Text", sf, tc);
text.insertTextAtCaret("\n\n -Text\n\n\n", sf, tc);
text.insertTextAtCaret("Beta Testers (Much Love!)", lf, tc);
text.insertTextAtCaret("\n\n -Text", sf, tc);
text.insertTextAtCaret("\n\n -Text", sf, tc);
text.insertTextAtCaret("\n\n -Text", sf, tc);
text.insertTextAtCaret("\n\n -Text", sf, tc);
text.insertTextAtCaret("\n\n -Text", sf, tc);
text.insertTextAtCaret("\n\n -Text", sf, tc);
I don’t have benchmark times from juce 7, but since upgrading to 8.0.6 this function takes almost 5 seconds which is WAY more than previously. It seems each insert text causes the next call to be even longer. I don’t know if thats just due to text size or caching or something.
This is a known issue. The current TextEditor’s implementation details aren’t a good fit for the new, more powerful text shaping building blocks. A new implementation is being worked on which addresses the performance regression.
Just wondering if there’s been any update on this or an estimated date we might see a fix? I see there was a commit with some text editor changes yesterday but seems like just a bug fix
Over the last week we released significant improvements to the TextEditor both in terms of performance and behaviour with regards to working with ligatures and bidi text.
With today’s bugfix on develop, I can recommend that you give it a try
