Skip to content
This repository has been archived by the owner on Nov 1, 2020. It is now read-only.

Create custom detailsview for the slideshow #119

Open
3 tasks
oparoz opened this issue Sep 4, 2016 · 9 comments
Open
3 tasks

Create custom detailsview for the slideshow #119

oparoz opened this issue Sep 4, 2016 · 9 comments
Labels
design Related to the design enhancement New feature or request

Comments

@oparoz
Copy link
Member

oparoz commented Sep 4, 2016

From @oparoz on March 1, 2016 22:18

Feature request

User type: All

User level: All

Description

We can't re-use the sidebar designed for Files as it only works in Files, especially since plugins used in tabs target the Files app (#548), so we'll have to design our own.
We can probably re-use the tabs logic to be able to easily show additional information and in order to not interfere with the slideshow itself, the information should be kept in a separate container, next to the slideshow.

Like this:

detailsview_slideshow

Basic info

  • filename
  • favourite star
  • File info
  • Read-only tags

Additional info

  • Meta data (GPS, decription, etc.)
  • Sharing
  • Read-only comments

Benefit / value

  • Users will appreciate being able to get to see everything there is to know about a particular image right next to it
  • Users will be able to share individual media files

Risk / caveats

  • It will look different than the Files sidebar
  • It will definitely offer less features than the Files sidebar, namely tags and comments management will never happen.

Sponsorship

None

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Copied from original issue: owncloud/gallery#549

@oparoz oparoz added coder wanted design Related to the design enhancement New feature or request labels Sep 4, 2016
@oparoz
Copy link
Member Author

oparoz commented Sep 4, 2016

@Bugsbane @jancborchardt @blizzz If you have ideas on what this should look like.

@oparoz
Copy link
Member Author

oparoz commented Sep 4, 2016

I've added a screenshot.
Also @setnes @deMattin

@oparoz
Copy link
Member Author

oparoz commented Sep 4, 2016

From @jancborchardt on March 4, 2016 21:6

Is there absolutely no way we can just use exactly the same? The sidebar should be a component, and especially since pictures are all files it should be the same.

cc @PVince81 @DeepDiver1975

@oparoz
Copy link
Member Author

oparoz commented Sep 4, 2016

No way in its current state. It's a component in Files and sidebar plugins target Files. It's the same problem with the upload class (owncloud/core#19563) and the current position of core is to let apps rewrite everything. That would require an additional Gallery maintainer focusing exclusively on keeping everything in sync with changes in core.

@oparoz
Copy link
Member Author

oparoz commented Sep 4, 2016

From @jancborchardt on March 6, 2016 18:36

Would really be cool if we can find a way to have the core one reusable. Otherwise it will result in a really strange UX. @karlitschek @PVince81 @DeepDiver1975

@oparoz
Copy link
Member Author

oparoz commented Sep 4, 2016

Otherwise it will result in a really strange UX.

Or complaints by users because of the lack of core features in apps. One cool alternative is being discussed in the grid view issue where apps would directly render their content in the Files app.

@oparoz
Copy link
Member Author

oparoz commented Sep 4, 2016

From @PVince81 on March 7, 2016 9:14

I'm not sure whether it's a good idea to make that class public. But maybe there could be another mechanism to let apps inject a sidebar into their apps, for example a method var detailsView = OC.Apps.enableDetailsView();. Then they can register their own panels there.

However, note that the details view is a more complicated topic. It's not just a panel, we also need to figure out whether the panel should automatically display all default tabs (sharing, comments, etc) if a file is given as input. And it becomes even more complicated when people are using the details view for non-files, in which case they'd have to implement their own sharing, comments, etc views.

A ticket was raised here to discuss this: owncloud/core#18170.

Or is this only about having an empty panel on the right that has the same styling as in core ?

@oparoz
Copy link
Member Author

oparoz commented Sep 4, 2016

Or is this only about having an empty panel on the right that has the same styling as in core ?

The business case for using the Files sidebar was opened here: owncloud/gallery#548
The idea is to be able to use any tabs available in Files, although we might remove some of the tabs, and to add our own.

The issue here was created to discuss about an alternative way of doing things where we would probably offer non-interactive views for things like comment and re-use the sharingview component in order to get rid of the dropdown.

I'll share my views on possible implementations on owncloud/core#18170

Thanks @PVince81

@oparoz
Copy link
Member Author

oparoz commented Sep 4, 2016

While we wait we could have a very simple sidebar implementation:

  1. User clicks on action button
  2. Sidebar slides out containing only the information relevant to the pressed button i.e. comments, sharing panel, meta data

If the user presses on a different button while the sidebar is open, the proper content can slide-out covering the previous one, which would be removed/hidden afterwards.

In the end, I don't think that it would be too much of a problem to have a different sidebar in the slideshow as that's another component all together, presenting a detailed view of an image or video.
The header would be different anyway, since there wouldn't be any thumbnail, the star for favourite doesn't need to be repeated there if it's already in the slideshow UI, etc.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
design Related to the design enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants