KurtDeGreeff / pirate

Automatically exported from code.google.com/p/pirate
0 stars 0 forks source link

pirate UX, uploader name after the tracktitle #24

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I checked the new UX version, nice program! it's definitely an improvement from 
pirate , but now it puts the uploader name after the tracktitle,
Maybe this can be fixed with a regular expression removing everything from the 
last '(' in the returned title. (it looks like something is already cutting of 
the last ')' )

Some other improvement points I found for UX are:
-auto-advance track for the playlist,
-after sorting a track in the playlist you have to click the track you want to 
sort again, because the focus is stolen by the sort buttons (and gives an index 
out of bound error)

Original issue reported on code.google.com by evertsch...@gmail.com on 7 Nov 2012 at 12:27

GoogleCodeExporter commented 9 years ago
Did you test with the latest version, which is 1002 at the time of me writing 
this?
I made a fix and released 1002 and someone reported it to be good.
We then applied the same fix in Pirate 1010 that was released earlier today.

We need someone to verify if this bug is indeed fixed or if it's still a 
problem in Pirate UX 1002 and Pirate 1010.

By auto-advance, do you mean that it should automatically play the next song in 
the playlist? If yes, then this feature should already be present.

I will fix the focus problem in the next version :)

Original comment by amplificator on 7 Nov 2012 at 8:10

GoogleCodeExporter commented 9 years ago
Ok, yes, 
it's kind of fixed because you can search on tracktitles now, because those are 
in the results, but still not like it should be.

so the tracktitle changed from:
before:        '(uploader_name)' and sometimes 'tracktitle'
now:           'tracktitle (uploader_name' and sometimes 'tracktitle'
and should be: 'tracktitle' all the time (like in pirate 1008)

the results that do correctly just show 'tracktitle' are the same in 1009 and 
1010 and ux 1002

Original comment by evertsch...@gmail.com on 7 Nov 2012 at 8:53

GoogleCodeExporter commented 9 years ago
I will release a fix for the song title within a couple of days so you can 
verify.

Regarding the auto-advance in the playlist, can you confirm that this feature 
(as I described above) is actually working?

Original comment by amplificator on 7 Nov 2012 at 8:59

GoogleCodeExporter commented 9 years ago
and yes, I mean that it automatically plays the next song, but it doesn't work..
after track is finished playing the play-position indicator goes back to the 
start and greys-out, and no track starts playing.
you have to press play on the next track to get it to play the next one.

Thanx for the support, I really love your program, and i think the playlist is 
a great addition if it does auto-advance.

Original comment by evertsch...@gmail.com on 7 Nov 2012 at 9:08

GoogleCodeExporter commented 9 years ago
It's supposed to auto-advance.. it seems to be working fine for me, but I'll 
look into it :)

Original comment by amplificator on 7 Nov 2012 at 9:12

GoogleCodeExporter commented 9 years ago
Hello j its walking(step) not c has a question with me is to mark(to stand out) 
no default logins available at this time
do you want to open the settings menu and use a custom credentials ? 

Original comment by lilololo...@gmail.com on 9 Nov 2012 at 1:03

GoogleCodeExporter commented 9 years ago
lilolololo71: You get that message because there is no default logins at this 
time. You must create your own account at VK.com and enter the login details in 
the settings menu.

Original comment by amplificator on 9 Nov 2012 at 1:45

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
I just released Pirate UX 1003 with the following changelog (Changelog is also 
attached to each file in the download section):

Changelog since version 1002:
- The display bug with track titles should now be fixed for good.
- Fixed the auto-advance of the playlist so when a track is done playing it 
will start the next one automatically.
- When you move a song up or down in the playlist it will now stay selected so 
you don't have to select it again to move it.
- Fixed occurrences where the song details would appear as 0 in the application 
title and in the saved playlists.

Please verify the fix for the track titles so I can merge it with Pirate 1011 :)

Original comment by amplificator on 10 Nov 2012 at 10:31

GoogleCodeExporter commented 9 years ago
hey, yes all fixes work for me.
the track titles all look good
and the playlist works as well.
great work, thanks!

Original comment by evertsch...@gmail.com on 10 Nov 2012 at 10:10

GoogleCodeExporter commented 9 years ago
Great, I'll be passing on the fix so it can be merged with Pirate then :)

Original comment by amplificator on 10 Nov 2012 at 10:15

GoogleCodeExporter commented 9 years ago
nice!
glad I could be of help to make one of my favorite programs even better:)

Original comment by evertsch...@gmail.com on 10 Nov 2012 at 10:18

GoogleCodeExporter commented 9 years ago
If you or anyone else got suggestions specifically for Pirate UX you can send 
me a mail and I'll look at it :)

Original comment by amplificator on 10 Nov 2012 at 10:24