<table><tr><td style="">sitter marked an inline comment as done.<br />sitter added a comment.</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D2478" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Dump from IRC</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>[12:18] <sitter> ochurlaud: why is kcolorschemeeditor a standalone application if you don't use it as such?<br />
 [12:25] <ochurlaud> sitter: It can be used at such<br />
 [12:26] <ochurlaud> but you might want to access it from the colors kcm, and until now, i didn't find a way to reload the kcm after creating a new scheme<br />
 [12:27] <ochurlaud> maybe i would need to send a DBus signal or something...<br />
 [12:27] <sitter> ochurlaud: so the plan is to not use SchemeEditorDialog directly but run the standalone application from the KCM?<br />
 [12:28] <ochurlaud> sitter, if it  is possible it is the plan<br />
 [12:28] <ochurlaud> the idea is also that notmart is creating a theme creation tool<br />
 [12:28] <ochurlaud> and a color scheme creation tool would be needed: it should finally move to plasma-sdk or something<br />
 [12:29] <sitter> ah yes, cool then<br />
 [12:30] <sitter> ochurlaud: also, yeah you'll probably need to send a dbus signal to the editor to have it load a different scheme<br />
 [12:30] <ochurlaud> sitter: well but don't expect it too early... :S<br />
 [12:31] <sitter> ochurlaud: no no. I was just perplexed by there being an application but the application not actually being used by the KCM :)<br />
 [12:36] <ochurlaud> sitter: see Thomas comments on <a href="https://phabricator.kde.org/D2409" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D2409</a> about the context<br />
 [12:36] <sKreamer> Diff 2409 "Update Color KCM" [Closed] <a href="https://phabricator.kde.org/D2409" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D2409</a><br />
 [12:36] <ochurlaud> It was the first iteration. Now that we have a standalone app, we can play with it so that it becomes better without breaking the kcm<br />
 [12:37] <ochurlaud> sitter: ok, then I accept your change, and we'll undo the nodisplay (maybe) in next iteration<br />
 [12:37] <sitter> ochurlaud: right now I think nodisplay is the way to go. That being said, I think even with it being a proper application it would be weird to have it in the menu. Even if it doesn't look like a KCM anymore, it still is effectively a settings app, which we all bundle into systemsettings<br />
 [12:38] <sitter> but yeah, that's a discussion to be had at a later time</p></blockquote>

</div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMADESKTOP Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2478" rel="noreferrer">https://phabricator.kde.org/D2478</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>sitter, bshah, ochurlaud<br /><strong>Cc: </strong>ochurlaud, bshah, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>