> On March 3, 2014, 9:24 p.m., David Edmundson wrote: > > lib/adium-theme-view.cpp, line 267 > > <https://git.reviewboard.kde.org/r/116559/diff/2/?file=251761#file251761line267> > > > > If we're going to throw a massive error we may as well just remove the > > code to load the custom templates. > > > > At least they then have a chance of working; and it will be a similar > > user experience.
BTW: Does changing from watching loadFinished to initialLayoutCompleted fix this issue? - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116559/#review51825 ----------------------------------------------------------- On March 3, 2014, 8:47 p.m., Diane Trout wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116559/ > ----------------------------------------------------------- > > (Updated March 3, 2014, 8:47 p.m.) > > > Review request for Telepathy. > > > Bugs: 325183 > http://bugs.kde.org/show_bug.cgi?id=325183 > > > Repository: ktp-text-ui > > > Description > ------- > > What I discovered is that occasionally the AdiumThemeView will fire its > loadFinished event before the body has finished loading. (which also means > that some of the javascript functions don't get defined as they're created > from the onLoad property of body. > > What this patch does is injects a small proxy QObject (AdiumThemeViewProxy) > into the javascript environment in > AdiumThemeView::injectProxyIntoJavascript(). The proxy gives javascript > access to a signal that it can call at the end of a new > "configureEnvironment" javascript function. > > I added the new function instead of adding the signal to the end of the > current initStyles functions because that appears to be overridable by some > of the themes. > > Also since this now lets you definitavely know that javascript is ready to > append your messages, I removed some extra sendDemoMessages calls. > > I'm not sure what happens if a theme tries to override data/Template.html > though. > > Also I stuck AdiumThemeViewProxy in with the AdiumThemeView because it seemed > small and tightly coupled with AdiumThemeView. However it seems like the > convention is every class gets its own .h/.cpp file. So that might need to be > fixed. > > > Diffs > ----- > > lib/adium-theme-view.cpp 635a877 > lib/adium-theme-view.h 8d17f9d > data/Template.html 6549479 > config/appearance-config-tab.cpp ffef4f6 > > Diff: https://git.reviewboard.kde.org/r/116559/diff/ > > > Testing > ------- > > Set message style to Renkoo for both normal chat and group chat. > > closed ktp-text-ui. > reloaded ktp-text-ui. > loaded configuration dialog. Clicked on both tabs. checked that they both > rendered > close configuration dialog. > repeat opening the configuration dialog box and checking both tabs a few > times. > > Before the patch usually at least one of the tabs wouldn't render. It was > somewhat random which one wouldn't render, and occasionally they both would > render correctly. > > I tried multiple times post patch and have yet to see it fail. > > > Thanks, > > Diane Trout > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
