This patch adds a GitHub Actions workflow which will automatically run
through all steps to build the Android version of the app. This is an
easy way of catching major build errors in pull requests or pushed
commits.
Additionally, this uploads the Android APK as build artifact. This
allows anyone to easily download any build for a while. That will make
it much easier for non-developers to participate in testing a new
version or a specific test since the no longer have to wait for a new
release.
The build artifacts are a bit tricky to find, but they are easy to point
out if anyone asks.
The account setting list the connection name which is the server address
directly followed by the username. This is then followed by a field
repeating the username again.
This duplication of information is nor really helpful, which is why this
patch replaces this with two fields containing just the two information:
host address and username.
This patch improves word wrapping for the description text in book
details by justifying the text block and allowing for words to be
wrapped by automatically inserting hyphens if necessary.
This causes the description box to look far less ragged on the right
edge which I think helps the overall cleanliness of the look.
Unfortunately the app's/browser's aren't as good as e.g. the TeX
algorithm for hyphening and there are JavaScript libraries which seem to
reimplement that. But this is already a significant step up and far less
work for something which is not the main focus of this app.
This patch adds a heavily blurred version of the cover image as
background to the cover image area in the book details.
This looks especially nice on larger devices like tablets.
Running `npm ci` you always get a warning about an old lock-file format
being used:
```
npm WARN old lockfile
npm WARN old lockfile The package-lock.json file was created with an old version of npm,
npm WARN old lockfile so supplemental metadata must be fetched from the registry.
npm WARN old lockfile
npm WARN old lockfile This is a one-time fix-up, please be patient...
npm WARN old lockfile
```
This patch simply re-generates the file by removing the old one and
running`npm install`. That gets rid of the warning.
This patch is a follow up to the discussion from #559 about potentially
simplifying progress interactions.
This patch essentially reduces the interactions to two simple options
which allow users to always marks books as finished or discard the
progress and thus mark them as not yet started with a single
interaction:
- If users have already listed to the book, regardless of the state of
the progress (finished, half way through, …), they get an option to
discard the progress. This replaces the additional “mark as not
finished” option but it should still be clear to users what happens.
- If a book is not yet finished, regardless of the state of the progress
(not yet started, half way through), users get a “mark as finished”
option which sets the progress to 100%.
This is hopefully a bit simpler than what we have now but also doesn't
require a user to figure out that you need to first mark a book as
finished to be able to discard the progress.
This patch updates the account settings view, trying to make it look a
bit cleaner by:
- Making the logout button look more like a button. The full width
looked odd on large devices or in landscape mode.
- Removing the version number. It is already listed in the main menu
anyway.
- Moving the hint about reporting bugs and the GitHub link to the
bottom.
The hint is still a bit weird in this view. Maybe long-term, we want to
move all these information about the app (version, issues, …) to a
separate view. But that's something for another day.
This patch moves the discard progress functionality from the tiny icon
at the top right corner of the progress display to the options menu.
The reasoning for this is that resetting progress is an option which is
unlikely to be used very often and therefor does not need to be on the
main screen. In addition to that, the menu already holds the related
options to mark the book as finished and to discard the progress of a
finished book.
Finally, this removes the tiny icon which does not really match the rest
of the user interface of the app.
This patch moves the progress bar slightly up for a better separation
between playback controls and seeking.
The reasoning behind this is that I (and probably others as well) often
interact with this interface in a clumsy, half-asleep state. Due to the
closeness of the progress bar and the play/pause button, I have managed
it several times to accidentally seek instead of pause. That is really
annoying since you then have to manually find the previous playback
position again.
Long term, I would love to see more of the available space being used.
In particular in the player interface. There should be enough free space
available. Especially if you have just one of the progress bars active.
But for now, this is just a quick-fix to improve the current situation.
This patch fixes the separators between metadata list entries in the book
detail view. Items were separated only by comma. A space was missing.
This just adds a space to both the genre as well as to the narrator
separators.
Lars Kiesow