search

nextcloud / gallery

DEPRECATED Gallery app was replaced by Photos
https://github.com/nextcloud/photos
GNU Affero General Public License v3.0
109 stars 58 forks source link

Sort icon images are off-center #496

Open clcain opened 5 years ago

clcain commented 5 years ago

Bug report

I noticed that the sort icons in the Gallery app were off-center after upgrading from Nextcloud 14 to Nextcloud 15 and updating the Gallery app. I created a Nextcloud server from the official Docker image (nextcloud:latest) for testing and observed the same issue. Perhaps some necessary padding was removed?

Steps to reproduce

  1. Start a new Nextcloud server from the official nextcloud:lastet Docker image on DockerHub
  2. Log in
  3. Click the Gallery icon at the top left

Expected behaviour

Sort icons should be nicely centered as they were in the version of Gallery that came with Nextcloud 14.

Actual behaviour

Sort icons are shifted to the right such that they are no longer centered.

screen shot 2019-01-20 at 7 33 27 am screen shot 2019-01-20 at 7 42 31 am

I observed this issue on Nextcloud version 15.0.2 with Gallery version 18.2.0. I'm using the latest version of Chrome, but I can confirm the issue is present in Safari and Firefox as well.

AInteriorB commented 5 years ago

The whole gally app seems to be outdated. There are many "tiny mistakes" like this one and nobody cares about. It this gallery still maintained?

1ndahous3 commented 5 years ago

I fully support the previous post.

@rullzer @weeman1337 @nickvergessen is the project still alive? It looks like a dead project without a maintainer. It must be rewritten (or at least globally revised) or closed and excluded from the basic set of applications.

nickvergessen commented 5 years ago

There is currently work being put into a new viewer app ( https://github.com/nextcloud/viewer/ ) which will replace the app in the shipped package at one day.

The gallery is currently without maintainer and anyone with spare time can pick it up.

clcain commented 4 years ago

This specific issue is still present on 17.0.0, although some other issues have been fixed. I submitted a PR (#535).