<table><tr><td style="">dfaure added inline comments.
</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/D29871">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D29871#inline-172051">View Inline</a><span style="color: #4b4d51; font-weight: bold;">rrosch</span> wrote in <span style="color: #4b4d51; font-weight: bold;">index.docbook:2141</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Do you mean that I should edit the text to add the stuff in the parentheses?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Yes, this was a suggestion for improving the docu.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D29871#inline-172055">View Inline</a><span style="color: #4b4d51; font-weight: bold;">rrosch</span> wrote in <span style="color: #4b4d51; font-weight: bold;">index.docbook:2141</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Actually, konqueror is compiled for Windows too, right? Does KDirModel make that distinction in "setShowingDotFiles"?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">You know it's opensource, right? ;-)</p>

<p style="padding: 0; margin: 8px;">KCoreDirLister uses it to check KFileItem::isHidden() which is says "starts with dot = hidden" (on all platforms) and also supports kioslaves that set UDS_HIDDEN (lxr says smb:, fonts:, and desktop: for .desktop files that shouldn't be shown in KDE or due to TryExec).</p>

<p style="padding: 0; margin: 8px;">OK so it's technically more than "starts with dot", though for the common case of local files, that's all it's about indeed.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D29871#inline-172052">View Inline</a><span style="color: #4b4d51; font-weight: bold;">rrosch</span> wrote in <span style="color: #4b4d51; font-weight: bold;">sidebar_widget.cpp:339</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">I haven't used Q_ASSERT before, and when I looked it up just now it says that all it does is print a warning message if the boolean is false. Is that what you mean?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">A failed assertion aborts the program, in debug mode. It's a way for developers to catch logic errors.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D29871#inline-172054">View Inline</a><span style="color: #4b4d51; font-weight: bold;">rrosch</span> wrote in <span style="color: #4b4d51; font-weight: bold;">sidebar_widget.cpp:580</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Ok, I'm going to look up how to do the checkbox. Hopefully it's straightforward and doesn't require too much code.</p>

<p style="padding: 0; margin: 8px;">The race condition I had in mind was when you have two windows open and you change the toggle in one, and then again in the other. Stefano says that his Konq does the change live, but that doesn't seem to be working for me (not sure if my Konq, KF5 or Qt is to blame), but in any case if the change happens live, then that solves it and I guess there is no race condition.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">OK, this isn't what I call a race condition (because it can be triggered very very slowly, that would be a snails race...). I call this a synchronization issue. At least konqueror is a unique service (single process) these days so it's easier than before, no DBus needed.</p>

<p style="padding: 0; margin: 8px;">If you have multiple windows open, I don't see how checking the action in one would immediately affect the other, you would have to implement that.<br />
I doubt Dolphin does that though?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R226 Konqueror</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29871">https://phabricator.kde.org/D29871</a></div></div><br /><div><strong>To: </strong>rrosch, dfaure<br /><strong>Cc: </strong>kde-doc-english, gennad, fbampaloukas, skadinna<br /></div>