-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116559/
-----------------------------------------------------------
(Updated March 5, 2014, 10:01 p.m.)
Review request for Telepathy.
Changes
-------
This patch reverts back to the rev 3 version that was just attempting to solve
the problem of the not rendering chat preview tabs, but adds a big blob of
comment describing the RegExp. (Also I reordered a variable to make it easier
to put the comment next to the thing being described.
Since this is basically what David said to ship. I'm going to assume that I
should still commit it.
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 (updated)
-----
config/appearance-config-tab.cpp ffef4f6
lib/adium-theme-view.h 8d17f9d
lib/adium-theme-view.cpp 635a877
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
