Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Structure of app and app navigation entries to have #146

Open
4 of 7 tasks
karlitschek opened this issue Apr 19, 2019 · 17 comments
Open
4 of 7 tasks

Structure of app and app navigation entries to have #146

karlitschek opened this issue Apr 19, 2019 · 17 comments
Labels
2. developing Work in progress design Related to the design enhancement New feature or request high High priority overview Overview of other issues

Comments

@karlitschek
Copy link
Member

karlitschek commented Apr 19, 2019

The Gallery app should have a left sidebar to find and view photos similar to other photo apps on Android, iOS, Mac, Windows, Gnome, KDE, ...

Sidebar entries could be:

  • All photos (in all folders as timeline)
  • Albums (grouped by folders as today)
  • Recently imported
  • Places (worldmap with pins of photos based on exif data) – half-done with a link to Maps
  • People (Different people with face detection (once we have the technology)) → Face Recognition #144
  • Memories (automatic grouping of photos from similar location and at a similar time)
  • more based on inspiration from other photo apps.
@karlitschek
Copy link
Member Author

@jancborchardt @MorrisJobke @oparoz What do you think?

@jancborchardt
Copy link
Member

Yep, absolutely think that’s the way to go for Gallery. 👍 Currently it’s just a weird other folder-based view, but if we make it a timeline, sort everything by date, and add these further enhancements which other photo apps also have, this is way more proper. (And then it won’t be just a "slightly different grid view".)

I guess the main point is that currently the Gallery is quite outdated and doesn’t follow a lot of the design standards we developed, and has the folder-based view. So it would be (don’t shoot me :D) a good point in time to actually do it properly with Vue.js and the design guidelines. What do you think @skjnldsv @juliushaertl?

There’s also some more work being done on the Maps app by @eneiluj including showing pictures on the map – so we could use that vice versa in the Gallery app too to show a map for the pictures.

@juliushaertl
Copy link
Member

I absolutely agree with Jan that this is a good point in time to rework the gallery app. There is a lot of code in it that is basically copied from the files_sharing app and therefore utterly outdated, because we changed a lot there recently for a better user experience.

For the sidebar and proper filtering, we somehow depend on nextcloud/server#5106

All photos (in all folders as timeline)

Not sure if we can achieve that already? Is there a dav search that we can perform to get all images in all folders?

Recently imported

Same as for all photos.

Places (worldmap with pins of photos based on exif data)

Not sure how the maps app looks for photos and caches the exif data, but we should probably just integrate the view here then.

Memories (automatic grouping of photos from similar location and at a similar time)
more based on inspiration from other photo apps.

A candidate once we have nextcloud/server#5106

@skjnldsv
Copy link
Member

Once we have the new sidebar standard, we might clone the thing into a OCA.Gallery.Sidebar with the same behaviour?

It should simplify our process a lot and allow further updates and security fixes in a single location :)

@jakobroehrl

This comment has been minimized.

@jancborchardt

This comment has been minimized.

@jancborchardt
Copy link
Member

So should we put that high on the list for Nextcloud 18 directly? We can do some planning in this cycle and see at the Contributor Week what we need to set it up.

@karlitschek
Copy link
Member Author

sure. let's plan this

@janis91
Copy link

janis91 commented Aug 1, 2019

I like that idea. A few weeks ago I started with some small frontend prototype (with react, because I'm not that familiar with vue yet). That perhaps gives some ideas. The prototype already includes things like a lightbox, lazy loading, responsive features, etc. But it is definitely a prototype so far. I did not have the time to connect that to the existing gallery API or to integrate Open Street Map yet.
sc1
sc2

The repo is private, because I don't want to publish this project (I think I won't have the time for it, to fulfill my own standards), but I can give you access to it, if you like.

@jancborchardt
Copy link
Member

@janis91 very cool, looks nice! :)
In Vue we do have the benefit of a lot of components already being available: https://github.com/nextcloud/nextcloud-vue – for example a Viewer which is basically the lightbox, or the left navigation which uses the Nextcloud styleguide.
cc @nextcloud/designers about that.

Also seeing you are in Stuttgart, maybe come over at the next Nextcloud Contributor Week (we just had one a month ago, but doing it every ~4 months)? :)

@janis91
Copy link

janis91 commented Aug 9, 2019

Also seeing you are in Stuttgart, maybe come over at the next Nextcloud Contributor Week (we just had one a month ago, but doing it every ~4 months)? :)

I will look out for it.

In Vue we do have the benefit of a lot of components [...]

I understand the benefit, it's just that I did not have the opportunity to learn Vue yet ;-)

@Cyber1000
Copy link

Besides from the renewal of the gallery app, which is a good thing by the way, would it make sense for you to create own albums (you may share with others) within the gallery app? (I've asked in https://help.nextcloud.com/t/is-there-a-good-app-to-organize-photos/57742 and was told to read this issue). It can't be only me who would like this feature, or are there already solutions I didn't find so far?

@karlitschek
Copy link
Member Author

@janis91 This is super cool! It would be great if you could help to push this forward. vue is prefered as @jancborchardt mentioned :-)

@janis91
Copy link

janis91 commented Aug 28, 2019

@janis91 This is super cool! It would be great if you could help to push this forward. vue is prefered as @jancborchardt mentioned :-)

My feeling is, that this will require a lot changes, @jancborchardt does it make sense to create a new branch and start from scratch in it, or what do you think?

@Spartachetto
Copy link

Now we have a functioning maps app, thanks to @eneiluj and the others who worked on it.

It would be really interesting to have an interaction between gallery and maps: if I have both apps active I'd love to have the possibility to switch from one view to the other...!

I think that the request is NOT simple but it would be a terrific improvement...

@jancborchardt
Copy link
Member

jancborchardt commented Aug 31, 2019

@janis91 @skjnldsv since we are doing everything in Vue now, it would absolutely make sense to do the Gallery in Vue too if we are making changes anyway.

Also design-wise, things to look at are:

(e.g. our variable tiling style is not really something other photo apps still use, it is either square cropped, or a square space for a photo and then shown smaller to show it in full.)

@jancborchardt jancborchardt changed the title Photo specific sidebar in Gallery Structure of app and app navigation entries to have Jan 21, 2020
@jancborchardt jancborchardt transferred this issue from nextcloud/gallery Jan 21, 2020
@jancborchardt jancborchardt added 2. developing Work in progress design Related to the design enhancement New feature or request high High priority overview Overview of other issues labels Jan 21, 2020
@jancborchardt
Copy link
Member

Moved this to the repository of the all-new Photos app → https://github.com/nextcloud/photos/

@janis91 did you see that? Come and get involved :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing Work in progress design Related to the design enhancement New feature or request high High priority overview Overview of other issues
Projects
None yet
Development

No branches or pull requests

8 participants