-
-
Notifications
You must be signed in to change notification settings - Fork 298
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
[Enhancement] Set album visibility recursively #1411
Comments
That could be done, but there are use cases where you don't want that behaviour. e.g. if some of your sub albums are hidden. |
Of course, but there are also are user cases when you do. So put a proper "do you really want to this?" option on it so you can't accidently do it. Or put it in the admin menu or something where it can't be done without proper thought. |
This issue is a duplicate. I will look for the duplicate ID as soon as I am in front of a proper PC. The idea by @mrpijey is actually rather old. For each album one can either select that the album inherits the settings of its parent album or the user can define independent settings. Then, when the user defines visibility settings, the user can either select a) the new settings shall only apply to the current album and all child albums which have been inheriting the previous settings will become independent This is basically the same approach as used by the NTFS filesystem and is probably the "gold plate solution". I would actually love to implement that and it is on my todo list after the unfortunate web installer, but until then there are still so many other things which need to be refactored. |
Well I guess this issue can be closed then if it's a duplicate. Unfortunately there are other issues with this software that prevents me from using it so I will have to look for alternatives, or wait until these issues are fixed. Thank you. |
Some thoughts & design proposal. Implementation
Defaults
Implementations thoughts:
Open questions
|
@mrpijey If you don't mind, go ahead and list your other issues as well. We may not be able to address them right away, but knowing what the big "pain points" are is helpful in driving our longer-term development. Ideally, we should probably have some poll on our website where users could vote on their "favorite" issues and such... |
@ildyria So What would that look like in the GUI? Would Should the |
Yes.
yes.
No. Not possible, our modals only allows 2 options: cancel and execute
That or a drop-down selection. |
After spinning it up this morning, Im surprised to see that this isn't the default. I expect that there are more complex things going on behind the scenes as the conversation in #1150 seems to indicate, but having to manually enter every album and set it to visible instead of setting the parent album to visible with it 'trickling down' seems like unfortunate UX. Im looking at alternatives for Piwigo which handles albums amazing, but has started getting very slow for me recently with only around 3k photos. |
I currently have a small problem with lychee, the change I made to the access policy actually slowed down Lychee significantly, so that is something I want to investigate. My problem is that I barely have time to work on Lychee. 2 of our best dev are sort of AWOL so we are pretty much only 2 active dev left on this project with very much time to spare. |
This is probably a totally different issue, but I'm curious on your thoughts on managing visibility at a photo level instead of an Album level. One of my issues that Ive run into is that I may have some photos from an album I want public, but not all of them. This doubly comes into play when looking at how 'Smart Albums' function by showing the images of only public albums. If 'publicity' were handled at a photo level, we would be able to 'trickle down' smart albums into full publicity based on their 'smart settings' instead of being required to share the entire album, just to get one photo from the album to show up in a smart album. I export my photos from lightroom with a star tag on it, so I can create 3/4/5 star albums, but that is highly dependent on the picture itself and not the album/folder it is currently in. Generally, I create an 'Album Highlights' style album of |
This is actually more tricky that it seems. Album visibility is better because it allows to immediately set the full set of pictures as visible. I do agree it is not convenient for your use-case though. Currently with the latest refactoring, this induced a large slowdown on the gallery, the more albums/pictures you have and the worst it becomes. I am currently investigating to avoid those problems further. |
At the moment you can set a single albums visibility to public or sensitive, but would be great to allow this to do this recursively to allow it to affect all sub-albums as well.
If you have a large gallery with lots of sub-galleries and what to change the visibility settings for all of them you have to do it manually one by one today, or go directly into the database which is impossible for hundreds of sub-albums.
The text was updated successfully, but these errors were encountered: