shpotify icon indicating copy to clipboard operation
shpotify copied to clipboard

Clean up output messages and make them consistent

Open hnarayanan opened this issue 9 years ago • 4 comments

Especially:

  • [ ] The output that's shown after play-by-search
  • [ ] The output that's shown when toggling playback features
  • [ ] The status option can show more things and it could be renamed to info. Think about whether share could be folded into it?

hnarayanan avatar Jan 30 '15 08:01 hnarayanan

I think share should not be integrated into either info or status. Since the clipboard interaction will delete every other content in the clipboard. In my opinion this is not the behavior you would expect from an option that displays status information. The clipboard interaction should be explicitly requested by the user.

snipem avatar Apr 20 '15 08:04 snipem

I agree. The above is out-of-date and what I meant to say was:

  1. The interface for what we call status and info today should be merged intelligently. Something like status providing more information than it does today (few extra attributes from info) and something like status detailed for many more (but not all, e.g. I don't see the utility of thumbnail URL) attributes from info.
  2. The interface for share could be merged into 1. e.g. status share as opposed to simply share as it is today.

hnarayanan avatar Apr 20 '15 11:04 hnarayanan

I agree with you from a functional point of view.

But I wouldn't extend the shell options to a two dimensional structure like shpotify status detailed or shpotify status share. The beauty of shell commands is in their short and normalized structure. And shpotify is most likely used to type Spotify commands quickly on the command line.

I would prefer shpotify for a short info, with shpotify status pointing to the same output, shpotify info for the detailed view and shpotify share for the aforementioned clipboard copying.

snipem avatar Apr 21 '15 07:04 snipem

I don't disagree with you. I am just thinking out loud here.

hnarayanan avatar Apr 21 '15 08:04 hnarayanan