<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<div class="moz-cite-prefix">Am 26.08.22 um 12:27 schrieb Aditya
Mehra:<br>
</div>
<blockquote type="cite"
cite="mid:ME3P282MB4196DFE1E5FF321132F0FD67EF759@ME3P282MB4196.AUSP282.PROD.OUTLOOK.COM">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<style type="text/css" style="display:none;">P {margin-top:0;margin-bottom:0;}</style>
<div style="font-family: Calibri, Helvetica, sans-serif;
font-size: 12pt; color: rgb(0, 0, 0); background-color: rgb(255,
255, 255);" class="elementToProof">
Hi,
<div><br>
</div>
<div>Aura Browser is in KDE Review and would like to release it
along with Plasma Bigscreen</div>
<div><br>
</div>
<div>Aura browser is a web browser specifically designed to work
with remote controls and key navigation for Plasma Bigscreen,
it supports a virtual mouse cursor and multiple tabs it also
includes a third party AdBlock from brave browser based on
easy list.</div>
<div><br>
</div>
<div>You can find the repository here:
<a class="moz-txt-link-freetext" href="https://invent.kde.org/plasma-bigscreen/aura-browser">https://invent.kde.org/plasma-bigscreen/aura-browser</a></div>
<div><br>
</div>
<div>Request to please review Aura Browser.</div>
<div><br>
</div>
<div>Regards,</div>
Aditya<br>
</div>
</blockquote>
<p><br>
</p>
<p>Hi,</p>
<p><br>
</p>
<p>- There is no Gitlab CI, please add that</p>
<p>- There is no support for Qt6. Please look into that and report
any blockers (e.g. missing API in Qt) you find. (not a blocker for
kdereview)</p>
<p>- In
<a class="moz-txt-link-freetext" href="https://invent.kde.org/plasma-bigscreen/aura-browser/-/blob/master/app/CMakeLists.txt#L36">https://invent.kde.org/plasma-bigscreen/aura-browser/-/blob/master/app/CMakeLists.txt#L36</a>
the desktop file is configured but only static values are passed
in. Why?</p>
<p>- According to the desktop file the app can open .deb and .rmp
files. Why would a browser be able to open those?</p>
<p>- The executable name (AuraBrowser) is somewhat unconventional.
Usually we use lowercase names, i.e. aurabrowser or aura-browser</p>
<p><br>
</p>
<p>I also have some nitpicks about the CMake code, but it's probably
more productive if I just make a MR instead of listing every
single one.</p>
<p><br>
</p>
<p>Cheers</p>
<p><br>
</p>
<p>Nico<br>
</p>
</body>
</html>