<div><div dir="auto">As an uninformed user I would just like to say Horizon is seen  _as_ Openstack to new users and I appreciate ever effort to improve it. </div><div dir="auto"><br></div><div dir="auto">Without discounting past work, the Horizon experience leaves much to be desired and it colors the perspective on the entire platform. </div></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 6, 2019 at 05:01 Radomir Dopieralski <<a href="mailto:openstack@sheep.art.pl">openstack@sheep.art.pl</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 6, 2019 at 11:33 AM Marek Lyčka <<a href="mailto:marek.lycka@ultimum.io" target="_blank">marek.lycka@ultimum.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi,<br><br>> we need people familiar with Angular and Horizon's ways of using Angular (which seem to be very <br>> non-standard) that would be willing to write and review code. Unfortunately the people who originally <br>> introduced Angular in Horizon and designed how it is used are no longer interested in contributing, <br>> and there don't seem to be any new people able to handle this.<br><br>I've been working with Horizon's Angular for quite some time and don't mind keeping at it, but<br>it's useless unless I can get my code merged, hence my original message. <br><br>As far as attracting new developers goes, I think that removing some barriers to entry couldn't hurt -<br>seeing commits simply lost to time being one of them. I can see it as being fairly demoralizing.<br></div></blockquote><div><br></div><div>We can't review your patches, because we don't understand them. For the patches to be merged, we</div><div>need more than one person, so that they can review each other's patches.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">> Personally, I think that a better long-time strategy would be to remove all <br>> Angular-based views from Horizon, and focus on maintaining one language and one set of tools.<br><br>Removing AngularJS wouldn't remove JavaScript from horizon. We'd still be left with a home-brewish <br>framework (which is buggy as is). I don't think removing js completely is realistic either: we'd lose <div>functionality and worsen user experience. I think that keeping Angular is the better alternative:<br><br>1) A lot of work has already been put into Angularization, solving many problems<br>2) Unlike legacy js, Angular code is covered by automated tests<br>3) Arguably, improvments are, on average, easier to add to Angular than pure js implementations<br><br>Whatever reservations there may be about the current implementation can be identified and addressed, but<br>all in all, I think removing it at this point would be counterproductive.<br></div></div></blockquote><div><br></div><div>JavaScript is fine. We all know how to write and how to review JavaScript code, and there doesn't</div><div>have to be much of it — Horizon is not the kind of tool that has to bee all shiny and animated. It's a tool</div><div>for getting work done. AngularJS is a problem, because you can't tell what the code does just by looking</div><div>at the code, and so you can neither review nor fix it.</div><div><br></div><div>There has been a lot of work put into mixing Horizon with Angular, but I disagree that it has solved problems,</div><div>and in fact it has introduced a lot of regressions. Just to take a simple example, the translations are currently</div><div>broken for en.AU and en.GB languages, and date display is not localized. And nobody cares.</div><div><br></div><div>We had automated tests before Angular. There weren't many of them, because we also didn't have much JavaScript code.</div><div>If I remember correctly, those tests were ripped out during the Angularization.</div><div><br></div><div>Arguably, improvements are, on average, impossible to add to Angular, because the code makes no sense on its own.</div><div><br></div><div><br></div></div></div>
</blockquote></div></div>