Skip to content

Conversation

@eknoes
Copy link
Contributor

@eknoes eknoes commented Apr 8, 2016

Licence: MIT

Adapted owncloud/core@d386168 to make work on #547 easier.

Description

This pull request makes it possible, to create a .nomedia file from the album itself via the [+] button. Furthermore it adapts owncloud/core@d386168, so that there is a standardized way to extend the [+]-menu.

Features

  • Possibility to extend the [+] buttons menu
  • Creation of a .nomedia file from the album itself

Tests

Test plan

  • "Hide Album" Submenu is displayed
  • Clicking Hide-Album creates a .nomedia file
  • Gallery will not show up the album anymore

Tested on

  • Linux/Firefox
  • Android 5.1/Chrome

TODO

  • Implement extending [+] button menu
  • Document extending
  • Implement creation of .nomedia file
  • Implement what follows after hiding an album and you are "inside"
  • Test

Check list

  • Code is properly documented
  • Code is properly formatted
  • Commits have been squashed
  • Tests (unit, integration, api and/or acceptance) are included
  • Documentation (manuals or wiki) has been updated or is not required

This is my first pull request, so if you have any advice for me please feel welcome.

…ted "Hide Album" function

Adapted owncloud/core@d386168
Hide Album function will create a .nomedia file in the current album and refreshs after creating .nomedia file
@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @oparoz, @imjalpreet and @PVince81 to be potential reviewers

@imjalpreet
Copy link
Member

@eknoes I have tested this and the main feature of this piece of code is working. On clicking Hide Album, The album gets hidden and the .nomedia file also gets created. But I have a few suggestions:

  • Firstly, when we hide an album, it gets hidden but after that the + button at the top becomes invisible until I go back to files and come back again. This happens when there was only one album and that is hidden. I have attached the screenshot.

pr643

  • Secondly, in my opinion, I think we should not have the hide album option in the home of Gallery App as it is not an album by itself. I think Hide Album should only be visible when we are inside some album.

I will give more suggestions if needed.

Cheers!

@oparoz
Copy link
Contributor

oparoz commented Apr 16, 2016

I agree with both suggestions

@oparoz oparoz added this to the 9.1-current milestone Apr 16, 2016
@oparoz
Copy link
Contributor

oparoz commented Apr 16, 2016

@eknoes Let me know if you have some time in the coming days, otherwise we can merge this as-is and implement the suggestions at a later time.

@eknoes
Copy link
Contributor Author

eknoes commented Apr 18, 2016

Sorry, the semester started again and so I don't really have the time to do this right now.

@oparoz
Copy link
Contributor

oparoz commented Apr 18, 2016

OK, thank you @eknoes

@oparoz oparoz modified the milestones: 9.2-next, 9.1-current May 31, 2016
@oparoz oparoz mentioned this pull request Jun 17, 2016
@oparoz
Copy link
Contributor

oparoz commented Jun 18, 2016

Transferred locally via #677 and will be part of the next release of Gallery+

@oparoz oparoz closed this Jun 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants