<table><tr><td style="">sredman 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/D17036">View Revision</a></tr></table><br /><div><div><p>This is good. Thank you!<br />
I have added a few comments for making the English sound a little more natural, as well as some little fixes</p></div></div><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/D17036#inline-92820">View Inline</a><span style="color: #4b4d51; font-weight: bold;">README.md:13</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);"><span class="n">All</span> <span style="color: #aa4000">this</span> <span class="n">without</span> <span class="n">wires</span><span class="p">,</span> <span class="n">over</span> <span class="bright"></span><span class="n"><span class="bright">the</span></span> <span class="n">already</span> <span class="n">existing</span> <span class="n">WiFi</span> <span class="n">network</span><span class="p">,</span> <span class="n">and</span> <span class="n">using</span> <span class="n">TLS</span> <span class="n">encryption</span><span class="p">.</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span class="n">All</span> <span style="color: #aa4000">this</span> <span class="n">without</span> <span class="n">wires</span><span class="p">,</span> <span class="n">over</span> <span class="bright"></span><span class="n"><span class="bright">your</span></span> <span class="n">already</span> <span class="n">existing</span> <span class="n">WiFi</span> <span class="n">network</span><span class="p">,</span> <span class="n">and</span> <span class="n">using</span> <span class="n">TLS</span> <span class="n">encryption</span><span class="p">.</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">"already existing" is redundant -- I would just change it to "over your WiFi network" or even just "over WiFi"</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/D17036#inline-92822">View Inline</a><span style="color: #4b4d51; font-weight: bold;">README.md:16</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; "><span class="p">##</span> <span class="n">Supported</span> <span class="n">platforms</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);"><span class="p">-</span> <span class="n">Computers</span> <span class="n">running</span> <span class="n">Linux</span> <span style="color: #aa4000">with</span> <span class="n">Plasma</span> <span style="color: #601200">5</span><span class="p">,</span> <span class="n">Gnome</span> <span style="color: #601200">3</span><span class="p">,</span> <span class="n">Elementary</span> <span class="n">OS</span><span class="p">...</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);"><span class="p">-</span> <span class="n">Android</span><span class="p">,</span> <span class="n">by</span> <span class="n">installing</span> <span class="n">the</span> <span class="bright"></span><span class="p"><span class="bright">[</span></span><span class="bright"></span><span class="n"><span class="bright">KDE</span></span><span class="bright"> </span><span class="n"><span class="bright">Connect</span></span><span class="bright"> </span><span class="n"><span class="bright">Android</span></span><span class="bright"> </span><span class="n"><span class="bright">app</span></span><span class="bright"></span><span class="p"><span class="bright">](</span></span><span class="bright"></span><span class="n"><span class="bright">https</span></span><span class="bright"></span><span class="p"><span class="bright">:</span></span><span class="bright"></span><span style="color: #74777d"><span class="bright">//play.google.com/store/apps/details?id=org.kde.kdeconnect_tp) (also available on [F-Droid](https://f-droid.org/repository/browse/?fdid=org.kde.kdeconnect_tp)</span>).</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span class="p">-</span> <span class="n">Computers</span> <span class="n">running</span> <span class="n">Linux</span> <span style="color: #aa4000">with</span> <span class="n">Plasma</span> <span style="color: #601200">5</span><span class="p">,</span> <span class="n">Gnome</span> <span style="color: #601200">3</span><span class="p">,</span> <span class="n">Elementary</span> <span class="n">OS</span><span class="p">...<span class="bright"></span></span><span class="bright"> </span><span class="n"><span class="bright">any</span></span><span class="bright"> </span><span class="n"><span class="bright">distro</span></span><span class="bright"> </span><span style="color: #aa4000"><span class="bright">with</span></span><span class="bright"> </span><span class="n"><span class="bright">a</span></span><span class="bright"> </span><span class="n"><span class="bright">little</span></span><span class="bright"> </span><span class="n"><span class="bright">work</span></span><span class="bright"> </span><span class="p"><span class="bright">:)</span></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span class="p">-</span> <span class="n">Android</span><span class="p">,</span> <span class="n">by</span> <span class="n">installing</span> <span class="n">the</span> <span class="bright"></span><span class="n"><span class="bright">KDE</span></span><span class="bright"> </span><span class="n"><span class="bright">Connect</span></span><span class="bright"> </span><span class="n"><span class="bright">app</span></span><span class="bright"> </span><span class="n"><span class="bright">from</span></span><span class="bright"> </span><span class="n"><span class="bright">the</span></span><span class="bright"> </span><span class="p"><span class="bright">[</span></span><span class="bright"></span><span class="n"><span class="bright">Play</span></span><span class="bright"> </span><span class="n"><span class="bright">Store</span></span><span class="bright"></span><span class="p"><span class="bright">](</span></span><span class="bright"></span><span class="n"><span class="bright">https</span></span><span class="bright"></span><span class="p"><span class="bright">:</span></span><span class="bright"></span><span style="color: #74777d"><span class="bright">//play.google.com/store/apps/details?id=org.kde.kdeconnect_tp) or [F-Droid](https://f-droid.org/repository/browse/?fdid=org.kde.kdeconnect_tp</span>).</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">I don't know if we need to add a comment about distro support, but I would change the addition to something like "Any disto which supports Qt 5"<br />
Better to be specific about what "little work" is needed :)</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/D17036#inline-92814">View Inline</a><span style="color: #4b4d51; font-weight: bold;">README.md:20</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; "><span class="p">##</span> <span class="n">How</span> <span class="n">to</span> <span class="n">install</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);"><span class="n">This</span> <span class="n">explains</span> <span class="n">how</span> <span class="n">to</span> <span class="n">install</span> <span class="n">KDE</span> <span class="n">Connect</span> <span class="n">on</span> <span class="n">your</span> <span class="n">computer</span><span class="p">.</span> <span class="n">You</span> <span class="n">will</span> <span class="n">also</span> <span class="n">need</span> <span class="n">to</span> <span class="n">install</span> <span class="n">it</span> <span style="color: #aa4000">in</span> <span class="n">your</span> <span class="bright"></span><span class="n"><span class="bright">phon</span>e</span> <span class="n">and</span> <span class="n">pair</span> <span class="bright"></span><span class="n"><span class="bright">both</span></span><span class="bright"> </span><span class="n"><span class="bright">devices</span></span> <span style="color: #aa4000">in</span> <span class="n">the</span> <span class="n">app</span> <span style="color: #aa4000">if</span> <span class="n">you</span> <span class="n">want</span> <span class="n">it</span> <span class="n">to</span> <span class="n">be</span> <span class="n">any</span> <span class="n">useful</span><span class="p">.</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span class="n">This</span> <span class="n">explains</span> <span class="n">how</span> <span class="n">to</span> <span class="n">install</span> <span class="n">KDE</span> <span class="n">Connect</span> <span class="n">on</span> <span class="n">your</span> <span class="n">computer</span><span class="p">.</span> <span class="n">You</span> <span class="n">will</span> <span class="n">also</span> <span class="n">need</span> <span class="n">to</span> <span class="n">install</span> <span class="n">it</span> <span style="color: #aa4000">in</span> <span class="n">your</span> <span class="bright"></span><span class="n"><span class="bright">Andriod</span></span><span class="bright"> </span><span class="n"><span class="bright">devic</span>e</span> <span class="n">and</span> <span class="n">pair</span> <span class="bright"></span><span class="n"><span class="bright">it</span></span><span class="bright"> </span><span class="n"><span class="bright">to</span></span><span class="bright"> </span><span class="n"><span class="bright">your</span></span><span class="bright"> </span><span class="n"><span class="bright">computer</span></span> <span style="color: #aa4000">in</span> <span class="n">the</span> <span class="n">app</span> <span style="color: #aa4000">if</span> <span class="n">you</span> <span class="n">want</span> <span class="n">it</span> <span class="n">to</span> <span class="n">be</span> <span class="n">any</span> <span class="n">useful</span><span class="p">.</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">These typos were already here:<br />
"This explains how to install KDE Connect on your computer. You will also need to install it <del>in</del> <strong>on</strong> your Andriod device and pair it to your computer in the app if you want it to be <del>any</del> useful."</p>

<p style="padding: 0; margin: 8px;">But I would actually reword this to:<br />
"These instructions explain how to install KDE Connect on your computer. You will also need to install it on your Andriod device and pair both devices before you can use this app"</p>

<p style="padding: 0; margin: 8px;">(I prefer "pair both devices" vs. "pair it to your computer" because it avoids mirroring "your computer" in both sentences. It sounds more natural to me)</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/D17036#inline-92818">View Inline</a><span style="color: #4b4d51; font-weight: bold;">README.md:31</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; "><span class="p">###</span> <span class="n">On</span> <span class="n">BSD</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);"><span class="n">It</span> <span class="bright"></span><span class="n"><span class="bright">should</span></span> <span class="n">work</span><span class="p">,</span> <span class="n">but</span> <span class="n">no</span> <span class="n">promises</span> <span class="p">:)</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span class="n">It</span> <span class="bright"></span><span class="n"><span class="bright">might</span></span> <span class="n">work</span><span class="p">,</span> <span class="n">but</span> <span class="n">no</span> <span class="n">promises</span> <span class="p">:)</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">I would leave this sentence as it originally was. "might" means something like "Oh, well, we don't really know" but "should" means "We expect that it will work even though we don't officially support it" (And I think it does work?)</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/D17036#inline-92817">View Inline</a><span style="color: #4b4d51; font-weight: bold;">README.md:61</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);"><span class="n">To</span> <span class="n">contribute</span> <span class="n">patches</span><span class="p">,</span> <span class="n">use</span> <span class="p">[</span><span class="n">KDE</span> <span class="n">Connect</span><span class="err">'</span><span class="n">s</span> <span class="n">Phabricator</span><span class="p">](</span><span class="n">https</span><span class="p">:</span><span style="color: #74777d">//phabricator.kde.org/project/profile/159/). There you can also find a task list with stuff to do, and links to other relevant resources. It is a good idea to also subscribe to the [KDE Connect mailing list](https://mail.kde.org/mailman/listinfo/kdeconnect).</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span class="n">To</span> <span class="n">contribute</span> <span class="n">patches</span><span class="p">,</span> <span class="n">use</span> <span class="p">[</span><span class="n">KDE</span> <span class="n">Connect</span><span class="err">'</span><span class="n">s</span> <span class="n">Phabricator</span><span class="p">](</span><span class="n">https</span><span class="p">:</span><span style="color: #74777d">//phabricator.kde.org/project/profile/159/). There you can also find a task list with stuff to do, and links to other relevant resources. It is a good idea to also subscribe to the [KDE Connect mailing list](https://mail.kde.org/mailman/listinfo/kdeconnect).<span class="bright"> We can also be reached on IRC at #kdeconnect or on [telegram](https://t.me/joinchat/AOS6gA37orb2dZCLhqbZjg).</span></span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Don't you have to specify that we are channel #kdeconnect <u>on freenode</u>? I know almost nothing about IRC 😂</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R224 KDE Connect</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17036">https://phabricator.kde.org/D17036</a></div></div><br /><div><strong>To: </strong>varunp, albertvaka<br /><strong>Cc: </strong>sredman, kdeconnect, shivanshukantprasad, skymoore, wistak, dvalencia, rmenezes, julioc, Leptopoda, timothyc, jdvr, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, mikesomov, tctara, apol<br /></div>