Opened 11 years ago
Closed 9 years ago
#795 closed enhancement (fixed)
Improve adding media to categories
Reported by: | GDR! | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 0.9.0 |
Component: | graphic design/interface | Keywords: | usability, gallery, collection |
Cc: | Parent Tickets: |
Description
So I uploaded some photos from an event, and wanted to add them all to a collection.
The current work flow is as following:
- Create collection
- Upload pictures
- Go to media processing panel
- Click on a title of photo
- Assign to collection
- Save
- GOTO 4
It's very tedious. I'm not an UI expert, but I see two potential solutions:
- When clicking Add Media while in a collection, add media to this collection by default
- In the Add Media screen, add a dropdown with list of categories
Attachments (3)
Change History (18)
comment:1 by , 11 years ago
Keywords: | usability gallery collection added |
---|
comment:2 by , 11 years ago
comment:3 by , 10 years ago
Totally agree here, especially on:
In the Add Media screen, add a dropdown with list of categories
And in the collection view, there should be a function to add media to this collection as well.
comment:5 by , 9 years ago
Hi,
I've just setup a mediagoblin instance, and my wife was annoyed by this. I'm happy to implement a simple collection dropdown for the add media page (I'll attach a basic patch), which I think would be useful independent of other bulk uploading solutions.
What's the next step here? If this ticket gets accepted, and the patch is along the right lines then I can write tests and code to take care of edge cases, i.e the collection already contains media of the same title.
by , 9 years ago
Attachment: | ticket795.patch added |
---|
comment:6 by , 9 years ago
Heya tofay! I think this patch is on the right track, but it doesn't seem to work. I tried running it and got the following error:
File '/home/cwebber/devel/mediagoblin/mediagoblin/submit/views.py', line 80 in submit_start "add", media, request.user, target=collection) NameError: global name 'collection' is not defined
And indeed, collection is not defined! Could you fix that issue?
Additionally, I think it would be ideal if the collections dropdown was shown *if* the user has collections to add it to. If not, maybe we should leave it out. Would you be interested in adding that enhancement?
Tests and edge cases fixes would be good to handle as well.
By the way, nice job adding the "add to collection activity"! That would have been easy to miss! :)
comment:7 by , 9 years ago
Oh yes, and if you submit another patch, could you use git format-patch
? It will make merging your patch much easier and preserve attribution information better.
comment:8 by , 9 years ago
Owner: | set to |
---|---|
Status: | new → in_progress |
comment:9 by , 9 years ago
w.r.t only showning the dropdown if the user has collections, yes I'll do that (it was the only edge case I could think of anyway!)
by , 9 years ago
Attachment: | 0001-Add-collection-drop-down-to-submit-page.patch added |
---|
comment:10 by , 9 years ago
Changes are:
mediagoblin/submit/forms.py - add the collection dropdown field
mediagoblin/submit/views.py - remove the collection dropdown if the user has no collections. If a collection was chosen then add the media to the collection
mediagoblin/tests/test_submission.py - test that the submit form works with/without the user having/specifying collections.
by , 9 years ago
Attachment: | 0002-Add-test-to-check-that-an-activity-is-created-when-a.patch added |
---|
comment:11 by , 9 years ago
Owner: | removed |
---|---|
Status: | in_progress → review |
Second patch adds a test to check that the activity was created. Sorry for submitting it as two patches/thanks for your patience (I'm pretty new to git)!
comment:12 by , 9 years ago
Milestone: | → 0.9.0 |
---|
Great work on this, tofay! I merged and pushed it. It's definitely a big improvement :)
Btw, how would you prefer to be attributed in the AUTHORS file? Just tom, or tofay, or something longer? Thanks!
comment:13 by , 9 years ago
Glad that I can help as I'm really liking mediagoblin. Re AUTHORS file: "Tom Fay"
comment:15 by , 9 years ago
Resolution: | → fixed |
---|---|
Status: | review → closed |
Please refer the workflow at a wordpress instance to create a gallery, this is very nice: