> On Oct. 31, 2012, 2:12 p.m., Dan Vrátil wrote: > > I'd prefer to have a separate component right away, rather then hacking it > > now and waiting for someone else to clean it up. > > > > One issue I've run to: I set Renkoo as group chat theme, reopened the > > Settings dialog, went to the Group chat tab again and the preview was not > > populated (this does not happen with SimKete though). I don't know whether > > this is caused by the Renkoo theme, but it might indicate problem in our > > code (I don't have any other themes to test right now). This does not > > happen in the "Single chat" tab.
I didn't have any issue like that, does it happen every time and/or with any other theme? I *think* that the Renkoo theme is broken for group chats though... - Daniele Elmo ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107132/#review21224 ----------------------------------------------------------- On Oct. 30, 2012, 4:19 p.m., Daniele Elmo Domenichelli wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107132/ > ----------------------------------------------------------- > > (Updated Oct. 30, 2012, 4:19 p.m.) > > > Review request for Telepathy. > > > Description > ------- > > This is a quick and dirty way to fix it, basically I duplicated the code for > the single user chat and added a tabbed view. > Perhaps it could be a widget in order to avoid code duplication, but since we > have just 2, I'd leave it like this for now, so that we can configure this in > 0.6, and perhaps open a junior-job bug for this > > > This addresses bug 295843. > http://bugs.kde.org/show_bug.cgi?id=295843 > > > Diffs > ----- > > config/appearance-config.ui 7fe6ce43a952652edcca229b918283b05268b240 > lib/adium-theme-view.cpp 24b9c3570699f64e1b751183430e4975dc6a449d > config/appearance-config.h 5b6c83d6ac4001f1d28b3ce1cb8e18a38e5269c5 > config/appearance-config.cpp 7f42857998b509a03dc8168181850a7a9a2a477d > > Diff: http://git.reviewboard.kde.org/r/107132/diff/ > > > Testing > ------- > > Config works, group chat works > > > Thanks, > > Daniele Elmo Domenichelli > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
