Problem/Motivation

As a AN I want to have a Albums so I can view all media items in one place.

Steps to reproduce

  1. Install the "Social Album" module.
  2. When you log in as "Test User" and open the user menu then you should see the "My albums" item.
  3. When you click on that menu item then you should be redirected to the "Albums" tab of the user profile page.
  4. You should see the "Create new album" button but when you navigate to the same tab on the profile page of another user then you shouldn't see this button.
  5. When you open the home page or the user stream page and add at least one image to the post creation form then you should see the "Add image(s) to an album" field which appears in this form.
  6. When you select the "Create new album" item then you should see the "Album name" field.
  7. When you fill "My first album" in the "Album name" and add some text to the post text field and press "Post" then you should see "Images posted in album My first album" in the teaser of the created post.
  8. When you open the "My albums" page from the user menu then you should see a tile teaser for the "My first album" album.
  9. You should see the first post image in the teaser.
  10. When you click on the image or album name or the "View album" link then you should be redirected to the album node overview page.
  11. You should see the "Test User's Albums" link over the page title in the hero section.
  12. You should see the number of images and the "Add images" button between the hero section and the content section.
  13. You should see all images from the created post in the content section.
  14. You should see the votes widget in the main content section and you should see the comment section under the content section.
  15. When you click "Test User's Albums" then you should be redirected to the "Albums" page.
  16. When you log in as SM and navigate to the "Configuration >Open Social Settings" page from the administrative menu then you should see the "Album settings" link.
  17. When you click "Album settings" then you should be on the settings page and you should see the "Active" checkbox.
  18. Uncheck the "Active" checkbox and press "Save configuration".
  19. When you back to the home page and open the user menu then you shouldn't see the "My albums" item.

Proposed resolution

  • It should be possible to place images in albums
    • Only the creator of the Album can upload images to the album
  • AN can open the images and it will show in a lightbox
    • should be possible to click through all images of the album
    • the post on the right should change appropriately
    • should be possible to see and add comments there
    • when adding a comment it should not reload
  • Pictures can be removed by the creator of the album
    • Creator can remove the picture from the post
    • Creator can delete the whole post
    • Creator can remove the image from the album
    • CM+ can also delete those images (as in deleting a whole post )
    • If user should delete the last image the album stays
  • The Album should have a default image if no image is uploaded
  • Pictures can be placed in albums via
    • A post on a stream
    • The album itself
      • If placed via the album directly it should be still a post (with description and the possibility to comment)
      • An activity item is created for that
  • Albums should contain
    • Title
    • Visibility
    • Creator
    • Creation date
    • Referenced images from posts
    • number of images (on the teaser and detail page
    • add image button
    • navigational element to go back to the Album overview
  • There should be an overview page for all Albums of one user
    • should be a separate tab on a profile
    • each album should have a teaser
  • The album should be a new content type
  • SM can enable and disable the tab
  • should be a new module: social_album

User interface changes

  • Add a link to an album in the post teaser and post image pop-up when the post is linked to the album.
  • Add the "My albums" item to the user menu.
  • Add the "Albums" tab on the user profile page.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chmez created an issue. See original summary.

lexhouk’s picture

Status: Active » Needs work
bas123’s picture

I would like to expand on this concept to the extent that there can be a Community Album with all or at leasts many of the attributes described above!

The contributor (and Admins) would have access control to any images that they contribute to the Album.

I have attempted to create a new Content Type for this purpose but need to dig further into how to do so without the content type taking on all of the attributes associated with Open Social such as Group participation. The placeholder for said community Album ned not have Events and all of the taxonomy associated with Topics.

Perhaps as the development of this proposal proceeds, this too can be integrated...

lexhouk’s picture

Issue summary: View changes
lexhouk’s picture

Assigned: lexhouk » Unassigned
Issue summary: View changes
Status: Needs work » Needs review
ribel’s picture

All album related PRs were merged into one:
https://github.com/goalgorilla/open_social/pull/1989

bas123’s picture

FileSize
124.54 KB

I would like to suggest this type of formatting for Activity Streams as utilized on Facebook:
Facebook Photo Album

ronaldtebrake’s picture

Will be in 10.0.0

ronaldtebrake’s picture

Status: Needs review » Fixed

  • chmez committed 1a4e902 on 10.0.x
    Issue #3172477 by chmez: Delete images from the "Hero" section on the...
  • chmez committed 1d159d6 on 10.0.x
    Issue #3172477 by chmez: Create a new module with the "Album" content...
  • chmez committed 953175b on 10.0.x
    Issue #3172477 by chmez: Fix getting the name of the image field when...
  • chmez committed b8b6a44 on 10.0.x
    Issue #3172477 by chmez: Delete the visibility field from the main...
  • chmez committed bed657d on 10.0.x
    Issue #3172477 by chmez: Add content to the main section of the album...
  • chmez committed c2ea6f6 on 10.0.x
    Revert "Issue #3172477 by chmez: Delete images from the "Hero" section...
  • chmez committed 09c75b1 on 10.0.x
    Issue #3172477 by chmez: Replace the images field of an album by the...
  • chmez committed 0aa71bf on 10.0.x
    Issue #3172477 by chmez: Change the position of the images block on the...
  • chmez committed 1307ead on 10.0.x
    Issue #3172477 by chmez: Add a link to the post creation form of the...
  • chmez committed 180e4c9 on 10.0.x
    Issue #3172477 by chmez: Show images from posts on the album page
    
  • chmez committed 18edcdd on 10.0.x
    Issue #3172477 by chmez: Change the order of posts on the album page to...
  • chmez committed 1f7b423 on 10.0.x
    Issue #3172477 by chmez: Re-use styling of post creation form from a...
  • chmez committed 2230195 on 10.0.x
    Issue #3172477 by chmez: Allow for logged-in users viewing albums of...
  • chmez committed 2a854a1 on 10.0.x
    Issue #3172477 by chmez: Create a block on the album page for displaying...
  • chmez committed 2b389cc on 10.0.x
    Issue #3172477 by chmez: Update album teaser and overview page after...
  • chmez committed 2dbb1b7 on 10.0.x
    Issue #3172477 by chmez: Override the message about creating a post when...
  • chmez committed 318e097 on 10.0.x
    Issue #3172477 by chmez: Make the cover image in the album teaser...
  • chmez committed 3294835 on 10.0.x
    Issue #3172477 by chmez: Combine two views into one
    
  • 32d6d62 committed on 10.0.x
    Issue #3172477 by agami4: Add stylesand new icons for the album teaser
    
  • chmez committed 38e16c6 on 10.0.x
    Issue #3172477 by chmez: Make access to the albums page configurable by...
  • chmez committed 3beae03 on 10.0.x
    Issue #3172477 by chmez: Hide a button from the main content section for...
  • chmez committed 417f021 on 10.0.x
    Issue #3172477 by chmez: Implement image preview based on modal windows
    
  • chmez committed 4e5a6cf on 10.0.x
    Issue #3172477 by chmez: Add the likes field to the main section of the...
  • chmez committed 5280c3a on 10.0.x
    Issue #3172477 by chmez: Make text in the main section of the page for...
  • chmez committed 59f0b39 on 10.0.x
    Issue #3172477 by chmez: Check if image fields were found before...
  • chmez committed 614f7cd on 10.0.x
    Issue #3172477 by chmez: Disable content centering in the main section...
  • chmez committed 6b5befb on 10.0.x
    Issue #3172477 by chmez: Implement updating the number of albums/images...
  • chmez committed 70f16f5 on 10.0.x
    Issue #3172477 by chmez: Create an overview page of user albums
    
  • chmez committed 712ce1a on 10.0.x
    Issue #3172477 by chmez: Open the album page after submitting the form...
  • chmez committed 731a6ee on 10.0.x
    Issue #3172477 by chmez: Send a node identifier to function for getting...
  • chmez committed 7984f3e on 10.0.x
    Issue #3172477 by chmez: Add conditions to checking relationships...
  • chmez committed 7a6bdf0 on 10.0.x
    Issue #3172477 by chmez: Grant permissions for a logged-in user for...
  • chmez committed 7e046d4 on 10.0.x
    Issue #3172477 by chmez: Exclude other users' albums from the album...
  • chmez committed 8a01c07 on 10.0.x
    Issue #3172477 by chmez: Implement a process for deleting the image from...
  • chmez committed 9a3a79d on 10.0.x
    Issue #3172477 by chmez: Add module name as a prefix of the extra field...
  • chmez committed 9ebfe04 on 10.0.x
    Issue #3172477 by chmez: Replace image item index by image file...
  • chmez committed 9f8ba88 on 10.0.x
    Issue #3172477 by chmez: Unlink post from the album when the last image...
  • chmez committed a030a7c on 10.0.x
    Issue #3172477 by chmez: Open post creation form instead of the album...
  • chmez committed a4eac3a on 10.0.x
    Issue #3172477 by chmez: Switch to an uncompressed JavaScript-file which...
  • chmez committed a5fd417 on 10.0.x
    Issue #3172477 by chmez: Attach a label to the first image in an album...
  • chmez committed a864080 on 10.0.x
    Issue #3172477 by chmez: Make the images field on the first album post...
  • chmez committed ab3b76f on 10.0.x
    Issue #3172477 by chmez: Clear the album field in a post when all...
  • chmez committed b25b43b on 10.0.x
    Issue #3172477 by chmez: Add the possibility to create a new album from...
  • chmez committed b59e6b9 on 10.0.x
    Issue #3172477 by chmez: Show the album field on the post creation form...
  • chmez committed bf046f9 on 10.0.x
    Issue #3172477 by chmez: Re-use post operations drop-down of streams for...
  • chmez committed bf47ea1 on 10.0.x
    Issue #3172477 by chmez: Hide a button for adding an album/images when...
  • chmez committed c584901 on 10.0.x
    Issue #3172477 by chmez: Prohibit creating a new album during adding...
  • chmez committed c702731 on 10.0.x
    Issue #3172477 by chmez: Open the album post page when clicking a link...
  • chmez committed ccff7cd on 10.0.x
    Issue #3172477 by chmez: Create a directory for templates of content...
  • chmez committed cddf4c6 on 10.0.x
    Issue #3172477 by chmez: Use album visibility for visibility of album...
  • chmez committed d1af0ee on 10.0.x
    Issue #3172477 by chmez: Replace the "Social Extra Large" image style by...
  • chmez committed d1c094a on 10.0.x
    Issue #3172477 by chmez: Re-route the post deletion operation on the...
  • chmez committed d2e0858 on 10.0.x
    Issue #3172477 by chmez: Add a link to the "Hero" section of the album...
  • chmez committed d474cb9 on 10.0.x
    Issue #3172477 by chmez: Add a drop-down menu with post operations to...
  • chmez committed dd989be on 10.0.x
    Issue #3172477 by chmez: Create an option for the PhotoSwipe gallery to...
  • chmez committed e2be540 on 10.0.x
    Issue #3172477 by chmez: Use image from the first position in the images...
  • chmez committed f2d0e9b on 10.0.x
    Issue #3172477 by chmez: Add block on the albums page for displaying...
  • chmez committed f9ba8c4 on 10.0.x
    Issue #3172477 by chmez: Use tiled teasers for albums on the albums page
    
  • chmez committed 044bc92 on 10.0.x
    Issue #3172477 by chmez: Make the machine name of the post creation form...
  • chmez committed 06a2ca3 on 10.0.x
    Issue #3172477 by chmez: Add classes to the cancel link on the image...
  • chmez committed 0dd19d1 on 10.0.x
    Issue #3172477 by chmez: Use injected services in the controller
    
  • chmez committed 0e42517 on 10.0.x
    Issue #3172477 by chmez: Hide a button for creating a new album on the...
  • chmez committed 136d108 on 10.0.x
    Issue #3172477 by chmez: Fix structure of the paths set where a block...
  • chmez committed 1c14e7d on 10.0.x
    Issue #3172477 by chmez: Switch a button for submitting a comment form...
  • chmez committed 3725382 on 10.0.x
    Issue #3172477 by chmez: Prohibit access to the albums page for a not...
  • chmez committed 3cff35d on 10.0.x
    Issue #3172477 by chmez: Add post teaser on the page for deleting an...
  • chmez committed 42e24eb on 10.0.x
    Issue #3172477 by chmez: Show album name in post teaser
    
  • chmez committed 652ea0f on 10.0.x
    Revert "Issue #3172477 by chmez: Switch to an uncompressed JavaScript-...
  • chmez committed 6adea83 on 10.0.x
    Issue #3172477 by chmez: Add library file which extends images slider by...
  • chmez committed 75640f4 on 10.0.x
    Issue #3172477 by chmez: Navigate to the page of a post after creating a...
  • chmez committed 76626b9 on 10.0.x
    Issue #3172477 by chmez: Allow adding a comment to post from the album...
  • chmez committed 7eb1822 on 10.0.x
    Issue #3172477 by chmez: Update the name of the theme for showing...
  • chmez committed 9349748 on 10.0.x
    Issue #3172477 by chmez: Hide pager on the albums page when the number...
  • 945cbac committed on 10.0.x
    Issue #3172477 by agami4: Update template
    
  • chmez committed a37ad40 on 10.0.x
    Issue #3172477 by chmez: Hide cover image label on all album pages...
  • chmez committed b3e3793 on 10.0.x
    Issue #3172477 by chmez: Change method for adding the default blocks set...
  • chmez committed b9a87c5 on 10.0.x
    Issue #3172477 by chmez: Swap places of two last parts of the path to...
  • chmez committed c06a7e7 on 10.0.x
    Issue #3172477 by chmez: Extend the images slider for the album page by...
  • chmez committed c8be274 on 10.0.x
    Issue #3172477 by chmez: Use the same translatable phrase with album...
  • chmez committed d243302 on 10.0.x
    Issue #3172477 by chmez: Allow for logged-in users viewing albums with...
  • chmez committed d486b98 on 10.0.x
    Issue #3172477 by chmez: Replace a stream library with an album library...
  • dcf4086 committed on 10.0.x
    Issue #3172477 by agami4: Update styles for the album and album teasers
    
  • chmez committed e24324e on 10.0.x
    Revert "Issue #3172477 by chmez: Create an option for the PhotoSwipe...
  • ribel authored e7e17ef on 10.0.x
    Merge pull request #1995 from goalgorilla/feature/3172477-albums
    
    Issue...
  • chmez committed f44ccf5 on 10.0.x
    Issue #3172477 by chmez: Check if the visibility option is available...
  • fccb9d0 committed on 10.0.x
    Issue #3172477 by agami4: Fix UI bugs
    
  • chmez committed fd07c47 on 10.0.x
    Issue #3172477 by chmez: Update patch for Drupal core
    
  • chmez committed 08ffe5b on 10.0.x
    Issue #3172477 by chmez: Add empty text
    
  • chmez committed 1f0b9a3 on 10.0.x
    Issue #3172477 by chmez: Update "Image Effects" module from 2.3 to 3.1
    
  • chmez committed 434c3d3 on 10.0.x
    Issue #3172477 by chmez: Fix coding standards in the "Social Album"...
  • chmez committed 5c2e545 on 10.0.x
    Issue #3172477 by chmez: Extend access callback of pop-up routes by...
  • chmez committed 935242b on 10.0.x
    Issue #3172477 by chmez: Mark album module as experimental
    
  • chmez committed f8158f8 on 10.0.x
    Issue #3172477 by chmez: Exclude albums from search pages
    
  • ronaldtebrake authored cc55382 on 10.0.x
    Merge pull request #2166 from goalgorilla/feature/3172477-DS-7488-album-...
  • nachosalvador authored f00d81e on 10.0.x
    Issue #3172477 by nachosalvador: DS-7488 Modify add image button text.
    

  • 1a955db committed on 10.0.x
    Issue #3172477 by agami4: Improve album theming
    
  • chmez committed 1c723c5 on 10.0.x
    Issue #3172477 by chmez: Change a message after creating a new album
    
  • chmez committed 3f7639a on 10.0.x
    Issue #3172477 by chmez: Use simple title on the album post creation...
  • 5928da6 committed on 10.0.x
    Issue #3172477 by agami4: Add improvements for the single image on the...
  • 5cdc00e committed on 10.0.x
    Issue #3172477 by agami4: Regenerate back-album button styles
    
  • 718009a committed on 10.0.x
    Issue #3172477 by agami4: Add styles for the back-album link
    
  • ronaldtebrake committed 774ef3b on 10.0.x
    issue #3172477 by ronaldtebrake: Add create new album to the + dropdown
    
  • ronaldtebrake committed 96744ee on 10.0.x
    issue #3172477 by ronaldtebrake: Add create new album to the + dropdown
    
  • ronaldtebrake committed c7358a4 on 10.0.x
    Issue #3172477 by ronaldtebrake: Fix albums with one image, these are...
  • chmez authored c74e5fc on 10.0.x
    Issue #3172477 by chmez: Reporting an post album renders the pop-up full...
  • chmez committed d4b68e3 on 10.0.x
    Issue #3172477 by chmez: Refactor code of a class of the page title...
  • ronaldtebrake authored 18f6967 on 10.0.x
    Merge pull request #2202 from goalgorilla/feature/3172477-album-single-...
  • ronaldtebrake authored e52c7b3 on 10.0.x
    Merge pull request #2195 from goalgorilla/bugfix/3172477-creation-page-...
  • ronaldtebrake authored f2f735a on 10.0.x
    Merge pull request #2193 from goalgorilla/bugfix/3172477-album-theming...
bas123’s picture

See below #13

bas123’s picture

Congratulations to all who contributed to this wonderful new (and needed) feature!

However having updated from social-8.x-9.8 to social-10.0.0, I must report the following with reference to the Album feature version that accompanied that update.

First and foremost, in all browsers, their is a dialogue box covering the bottom Post comment box on each of the streams.
photo album issue
I have no idea what purpose this serves nor why it appears here!

Revised: I have determined that the overlapping element pictured in the screenshot occurred following my enabling of the Social Album (Experimental) Content type. Which after realizing that it crashed once a node was saved, I uninstalled, thus leaving this behind on all Streams.
Note: I attempted to re-enable the module to see if that might resolve the issue, but it wouldn't do so citing:

Error message

    Unable to install Social Album, core.base_field_override.node.album.promote, core.entity_form_display.node.album.default, core.entity_view_display.node.album.default, core.entity_view_display.node.album.teaser, field.field.node.album.field_album_comments, field.field.node.album.field_album_creators, field.field.node.album.field_content_visibility, field.field.post.photo.field_album, field.storage.node.field_album_comments, field.storage.node.field_album_creators, field.storage.post.field_album, group.content_type.closed_group-group_node-album, group.content_type.open_group-group_node-album, group.content_type.public_group-group_node-album, node.type.album already exist in active configuration.

---------------------
Secondly, as I am sure you are aware, the Album Content type (when enabled), and then an Album is created crashes upon saving, plus has no input field for adding images! Perhaps there is some step I have missed?

Additional opinion: Respectfully, the Post comments should remain in the stream to discuss the Album in general (as it is assumed to be thematic), but not in the lightbox as each of the images should have their own comments/discussions. Having the Album's discussions carry over into the lightbox slides is a redundancy, IMHO!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.