Tags list on Edit page is not seperated by spaces and hard to read

[Jef van Schendel]

On the Edit page for any media, the "Tags" field look like this, no
matter if you used commas, spaces or both:

::

    tag1,tag2,tag3,tag4

Instead, it could use some space between the tags to make it more
readable:

::

    tag1, tag2, tag3, tag4

I've also changed the tag list on media pages to "Tagged with tag1,
tag2, tag3, etc", so if we change this they will be consistent in
appearance.

[Manolinux aka Mu]

It is easy to add a whitespace:

::

    -        media_tag_string = mg_globals.app_config['tags_delimiter'].join(
    +        media_tag_string = (mg_globals.app_config['tags_delimiter']+u' ').join(
                                           [tag['name'] for tag in media_entry_tags])

but I am unsure about something.

'tags\_delimeter' is parametrized but, do you want to make the
whitespace "hardcoded"? I think it is ok, but I want to be sure.

[Christopher Allan Webber]

You're right mano, shouldn't be too hard.

However I think we agreed on IRC that we were going to drop the
custom-delimiters. I marked `#678 </issues/678>`_ as related.
Someone interested could probably wrap these up in one simple swoop
by doing ', '.join() instead and removing the custom delimiter
stuff.

[Manolinux aka Mu]

That would be easy because I think 'tags\_delimiter' is only used
twice in all code.

But, are you sure you want to put only ', ', it is similar to a
magic number. Suppose you want to change the delimiter tomorrow,
you would need to search for commas inside the code (madness)
instead of searching for tags\_delimiter, wich would be more
reasonable.

Maybe we could use a constant if you don't want it to be
customizable.

Anyway, could you please assign this bug to me?

[Elrond]

Added you as Hacker,Reporter so that we (and you yourself) can
assign you to some issues.
And assigned this issue to you.

[Manolinux aka Mu]

I pushed the fix to my repository:
`git@gitorious.org <mailto:git@gitorious.org>`_:~manolinux/mediagoblin/manolinuxs-mediagoblin.git

Finally I defined a constant in the same text.py.

I modified a test that expected the tags as 'yin,yang' instead of
'yin, yang'.

I also took care of
`http://bugs.foocorp.net/issues/678 <http://bugs.foocorp.net/issues/678>`_

[Manolinux aka Mu]

I forgot to say the branch name is
671\_spaces\_in\_tag\_list\_edit

[Elrond]

Merged.
Thanks!

[Will Kahn-Greene]

The original url for this bug was ​http://bugs.foocorp.net/issues/671 .
Relations:
#329: related