<table><tr><td style="">ngraham 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/D23542">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/D23542#inline-132951">View Inline</a><span style="color: #4b4d51; font-weight: bold;">ndavis</span> wrote in <span style="color: #4b4d51; font-weight: bold;">module.cpp:39</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Why are we changing the title to just "Audio"? Weren't we supposed to use descriptions for the page titles to make them useful? Am I mistaken?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Our conclusion before was to make the page title match the title listed in the desktop file, and the comment has a description. You can see this in all the KCMs in the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Appearance section</span></span></span> This seems semantically correct, and if we want to display the descriptive string, we should change KPageDialog or something in KCMUtils to parse the comment in the desktop file and display it underneath the title. Frankly that should happen anyway so we don't have to define the same string for the title in both the desktop file and also the kaboutdata.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R115 Plasma Audio Volume Applet</div></div></div><br /><div><strong>BRANCH</strong><div><div>promote-audio-kcm (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23542">https://phabricator.kde.org/D23542</a></div></div><br /><div><strong>To: </strong>ngraham, VDG, Plasma, drosca, nicolasfella, sitter<br /><strong>Cc: </strong>ndavis, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>