> On April 13, 2014, 6:25 p.m., David Edmundson wrote: > > config/behavior-config.ui, line 159 > > <https://git.reviewboard.kde.org/r/117543/diff/3/?file=265234#file265234line159> > > > > I think it's generally bad to have a checkbox label which is phrased in > > negative. > > > >
On the other hand is weird to have a checkbox that you have to uncheck in order to enable a feature. - Dan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117543/#review55612 ----------------------------------------------------------- On April 13, 2014, 4:53 p.m., Dan Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117543/ > ----------------------------------------------------------- > > (Updated April 13, 2014, 4:53 p.m.) > > > Review request for Telepathy. > > > Repository: ktp-text-ui > > > Description > ------- > > When "Don't leave chat room when window is closed" settings is enabled, the > channel is not closed when user closes the tab or window, but is maintained > by TelepathyChatUi. The channel can be left via Conversation -> Leave room > action. > > > Diffs > ----- > > app/chat-window.h 72bbd1d > app/chat-window.cpp a7da574 > app/telepathy-chat-ui.h 97bc4b7 > app/telepathy-chat-ui.cpp 33150b8 > config/behavior-config.h d57fd90 > config/behavior-config.cpp eeb3597 > config/behavior-config.ui c8e731c > lib/chat-widget.cpp 3682742 > lib/notify-filter.h f929ce3 > lib/notify-filter.cpp 6807dac > lib/text-chat-config.h e0ba24f > lib/text-chat-config.cpp 57c7c0c > > Diff: https://git.reviewboard.kde.org/r/117543/diff/ > > > Testing > ------- > > > Thanks, > > Dan Vrátil > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
