Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added queue's total time & case insensitive sorting; ignore empty search string #168

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

embhd
Copy link
Contributor

@embhd embhd commented Apr 5, 2018

  • Added queue's total time to panel header (closes Cumulative time for Queue #47)
  • Enable case insensitive sorting of database entries while browsing
  • Since an empty search string always seems to just time out, it is now simply ignored

Unfortunately, I had to "pre-merge" the original three branches, since I had to make some final changes for everything to work as expected (and to be auto-mergeable), c.f. 71ec2cd.

Additional changes:

  • Removed compiler warning "comparison of constant -1 with expression of type 'enum mpd_cmd_ids' is always false"
  • Fixed main table layout in case of an empty queue and search result, respectively
  • Split-up of a few very long code lines
  • Update link behind navbar's "Browse Database" after every page reload (i.e. after following a link)
  • Make text input in "Save Queue" behave like the one in "Add Stream"
  • Some clean-up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cumulative time for Queue
1 participant