Ticket #714 (closed enhancement: fixed)

Opened 4 years ago

Last modified 16 months ago

filter for audio

Reported by: erikos Owned by: kushal
Priority: Unspecified by Maintainer Milestone: 0.86
Component: Jukebox Version: 0.84.x
Severity: Major Keywords:
Cc: cjl Distribution/OS: Unspecified
Bug Status: New

Description

chooser = ObjectChooser(_('Choose image'), self._parent,
                                gtk.DIALOG_MODAL | gtk.DIALOG_DESTROY_WITH_PARENT,
                                what_filter=mime.GENERIC_TYPE_AUDIO)

Change History

  Changed 4 years ago by kushal

If I do this , then it is not picking the different playlist files, is this the correct behaviour ?

  Changed 4 years ago by tomeu

I guess we need to tell Sugar that .pls files are audio files, see GENERIC_TYPE_AUDIO in sugar-base/src/sugar/mime.py.

  Changed 2 years ago by RafaelOrtiz

this .pls mimetype is in sugar by audio/x-scpls'
 http://git.sugarlabs.org/sugar-base/mainline/blobs/master/src/sugar/mime.py

	
	
    'id': GENERIC_TYPE_AUDIO,
	
    'name': _('Audio'),
	
    'icon': 'audio-x-generic',
	
    'types': [
	
        'audio/ogg', 'audio/x-wav', 'audio/wav', 'audio/x-vorbis+ogg',
	
        'audio/x-mpegurl', 'audio/mpegurl', 'audio/mpeg', 'audio/x-scpls'],

follow-up: ↓ 5   Changed 2 years ago by RafaelOrtiz

jukebox has the corrected chooser in

248 chooser = ObjectChooser(_('Choose document'), self,
249 gtk.DIALOG_MODAL |
250 gtk.DIALOG_DESTROY_WITH_PARENT,
251 what_filter=mime.GENERIC_TYPE_AUDIO)

I'd would only change 'Choose document' for 'Choose media'

in reply to: ↑ 4   Changed 22 months ago by godiard

  • cc cjl added

Replying to RafaelOrtiz:

.....

I'd would only change 'Choose document' for 'Choose media'


I don't know, will break all the translations and "media" is not very clear for a child. CC to cjl to see if he has any idea about what do here.

  Changed 22 months ago by cjl

Change any UI string that is needed for clarity. Then regenerate and commit the POT. We'll update the POT on Pootle from VCS and localizers will correct the one string. Not a big deal.

  Changed 16 months ago by godiard

  • status changed from new to closed
  • resolution set to fixed

Changing the title in the ObjectChooser is deprecated for a long time.

This ticket can be closed.

Note: See TracTickets for help on using tickets.