Opened 9 years ago

Closed 8 years ago

#390 closed defect (fixed)

"Separate tags by commas" should be unnecessary

Reported by: Jef van Schendel Owned by: spaetz
Priority: minor Milestone: 0.3.2
Component: programming Keywords: bitesized
Cc: sebastian@… Parent Tickets:


I think JanCBorchardt talked about this before. This text really shouldn't be necessary.

We should choose one method and stick with it. The user should be able to enter tags as desired, after which they will automatically be seperated.


Change History (13)

comment:1 Changed 9 years ago by joar

I like the expliicit comma separation of tags, it enables me to have tags that are more verbose, like "georgia 2011" or "dj spyhunter".

comment:2 Changed 9 years ago by Will Kahn-Greene

Milestone: 0.2.1

Bumping this out of 0.2.1. If someone wants to grab it, feel free to grab it and put it in 0.2.2.

comment:3 Changed 8 years ago by Christopher Allan Webber

Component: component1programming

comment:4 Changed 8 years ago by joar

Disregard my previous comment, the "Verbose names" should be handled by "sets" or "albums" not tags.

comment:5 Changed 8 years ago by Jakob Kramer

Priority: blockerminor

comment:6 Changed 8 years ago by Christopher Allan Webber

Keywords: bitesized added

I feel pretty strongly that we should keep comma separated tags.

The code mostly only provides one of these options, but there's some leftover crufty code in mediagoblin/tools/ that provides the option of a delimiter.. we should remove that and make it explicitly a comma. That's an easy task.. marking this as bitesized.

comment:7 Changed 8 years ago by ShawnRisk

Owner: somebody deleted
Status: newassigned

comment:8 Changed 8 years ago by Aleksej

Some new users enter tags as “#tag1 #tag2 #tag3”, despite the text. Maybe an example would work better for them?

Last edited 8 years ago by Aleksej (previous) (diff)

comment:9 Changed 8 years ago by Duncan

Fixed comma delimiter to fixed delimiter as per "comment 6". Changed some text formatting to fit pep8

branch is 390_delim

comment:10 Changed 8 years ago by spaetz

Cc: sebastian@… added
Keywords: needsreview added
Milestone: 0.3.3
Owner: set to spaetz
Status: newassigned

I'll see if I can review this durin this week.

comment:11 Changed 8 years ago by spaetz

I reviewed dunkyp's patch and it is exactly the right thing to do. The only thing I have done is to squash the patches/reverts and incremental improvements into a single patch by him, they touch only a few lines.

I also added a second cosmetic patch on top that get's rid of 2 more lines.
Branch can be seen here:

Please merge branch 390_delim of my git repo at git://

comment:12 Changed 8 years ago by spaetz

Keywords: needsreview removed

comment:13 Changed 8 years ago by Elrond

Resolution: fixed
Status: assignedclosed

Merged spaetz' branch.

Commit: fa2f6ba162

Note: See TracTickets for help on using tickets.