GitLab

The appstream spec was updated a while ago, so install our appdata
to the new location[0] instead of the legacy path.

[0] https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#spec-component-location

It's useless for styling, but makes it much easier to identify a widget
corresponding to a particular user, room or account in the inspector :-)

As outlined in the previous commit, there is no real need for
keeping hidden popovers for every room around all the time.
Just share a single popover between all entryAreas to cut down
on the number of widgets (and the resulting overhead of GTK+
updating their state etc.).

The popover for changing the nick is currently handled entirely as
part of the entryArea, which means we have to keep one popover for
every room around, even though they don't carry any room-specific
state and we never show more than one at any given time.
Splitting out the NickPopover decouples it from the entryArea,
which will allow us to share a single one in the future.

The intended behavior when changing the nick via the popover is to
update the button label immediately, and only revert to the actually
used nick when the server doesn't react in a reasonable time (or
simply rejects the update).
However 911b9558 broke that, as the account's nickname property
changes right away , but isn't actually used to set the nick label
while connected. In that case the label is set from the connection's
self-contact, which will only pick up the new nick after it changed
on the server.
So to avoid resetting the newly requested nick immediately, only
consider changes to the nickname while disconnected.

We stopped bundling font-config in commit 84b8c04b, so this is now
unused.

On gitlab (or -hub), the file is displayed prominently on the project
page, so put together some useful information for new contributors.

While F26 satisfies all our build dependencies (not least because
there's little to build), rawhide is much closer to the platform
we are actually developing for, in particular when it comes to gjs
and Spidermonkey.

Also rawhide's mozjs52-devel package now includes the js52 binary,
so we can run the basic syntax check added in commit 65a20993 during
continuous integration.

Application::prepare-shutdown is emitted when the window is destroyed,
after all its children has been destroyed as well - any signal connected
to an AppNotification has been disconnected by then, so even if we called
close as intended it wouldn't have any effect. So to get the intended
behavior, run close() when the notification is destroyed without being
closed before.

In order to allow for keyboard-friendly network selection, we currently
activate the first one when the filter entry is activated. While this
is generally what we want, it's not very obvious that this will happen.
So instead, keep the first row selected while filtering, and activate
the selected entry when hitting enter.

Daniel Boles authored Sep 12, 2017 and Florian Müllner committed Oct 28, 2017

https://bugzilla.gnome.org/show_bug.cgi?id=787154

Daniel Boles authored Sep 02, 2017 and Florian Müllner committed Oct 28, 2017

Use the same string as the accessible name, to aid translators.

https://bugzilla.gnome.org/show_bug.cgi?id=787154

Augusto Cesar authored Dec 07, 2016 and Florian Müllner committed Oct 28, 2017

The "Add Network" section of Polari's join dialog doesn't give any
feedback when the search does not return any results.

To fix this, add a GTK box with the looking glass icon and a label
giving the proper feedback as a placeholder in the connections list.

https://bugzilla.gnome.org/show_bug.cgi?id=762892

GJS added API for defining GObject classes with ES6 class syntax
last cycle, use it to port the remaining Lang.Class classes to
the new syntax.

Fixes GNOME/polari#14

ES6 finally adds standard class syntax to the language, so we can
replace our custom Lang.Class framework with the new syntax. Any
classes that inherit from GObject will need some special treatment,
so limit the port to regular javascript classes for now.

Fixes GNOME/polari#14

Modern javascript has a short-hand for function properties, embrace
it for better readability and to prepare for porting to ES6 classes.

Fixes GNOME/polari#14

Our helper method takes less arguments than the GTK+ method it's based
on - remove the excess argument to avoid warnings.

Fixes GNOME/polari#14

All rows have the same height, so with some minor tweaks we can set
the colum's sizing to fixed and enable fixed-height-mode to speed
up the TreeView.

Fixes GNOME/polari#13

With the previous commit, we now leave enough time each frame for
drawing under normal circumstances. However as filtering requires
some extra resources, we still run into responsiveness issues when
the list is filtered while we are still loading it. Address this by
limiting the time available for inserting rows some more while
filtering is active.

Fixes GNOME/polari#13

The number of available rooms can be fairly large (*cough* Freenode
*cough*), so we currently insert rooms in chunks of 100 to make sure
that we don't block the mainloop while loading the list. We still suffer
from significant slow-down though, and Bastien Nocera pointed out that
instead of inserting a fixed number of rows, we should limit the time
spent in the idle handler to make sure that each frame still has enough
cycles for drawing.

Fixes GNOME/polari#13

Zandr Martin authored Feb 10, 2017 and Florian Müllner committed Oct 27, 2017

Increase the right margin of the username box so that the overlay
scrollbars do not cover up part of the arrow.

https://bugzilla.gnome.org/show_bug.cgi?id=728756

Mark polari as a GNOME project and include links to online help
and translation documentation.

While the old bugzilla URL redirects to gitlab, it makes sense to
just request the correct URL in the first place ...

We will currently always open the main application window after the
initial setup window has been closed. This behavior is correct if
the window was completed or dismissed, but not when the app itself
was quit - cancel continuing to the main window in that case to
get the expected behavior.

Eventually we want continuous integration based on flatpak-builder,
but for now use a Fedora docker image.

Fixes GNOME/polari#11

As most of our sources are written in Javascript, they are currently
not even compile-tested when building. We can at least get some minimal
testing by performing some basic syntax checking when SpiderMonkey's JS
shell is available.

Fixes GNOME/polari#11

The list of javascript sources moved from the build system to the
GResource XML when we  started to compile sources into a resource.
However to get at least some minimal testing of non-compiled sources,
we want to leverage meson's unit test support to perform some basic
syntax checking. That will require that the build system again has
access to the list of javascript files, so move it back there. In order
to avoid duplicating the list of files, generate the resource XML
at configure time.

Fixes GNOME/polari#11

Daniel Boles authored Aug 25, 2017 and Florian Müllner committed Oct 19, 2017

If Dennis messaged you from #room, the title was:

    room Dennis

It seems clearer and more grammatical to say

    Dennis in room

https://bugzilla.gnome.org/show_bug.cgi?id=786709

Update NEWS.