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

Tweaks for better small screen layout and darker theme #86

Closed
wants to merge 7 commits into from
Closed

Tweaks for better small screen layout and darker theme #86

wants to merge 7 commits into from

Conversation

southafricanrob
Copy link

  1. volume slider movies to header on small screens for easy access
  2. now playing moves to side panel on larger screens so always visible
  3. added right side settings panel in header to free up some space from Now Playing panel
  4. dark theme option similar to Spotify Web Player
  5. independently scrollable playlist and track list divs

southafricanrob added 7 commits March 10, 2015 23:28
- Volume slider moves to header on small screens for easier access
- Now Playing moves to side panel on large screens so always visible
-added right side settings panel in header to free space in the
left panel for the Now Playing info
-links to other Musicbox's on the same network (Work In Progress!)
-option to choose a Dark Theme similar to Spotify's Web Player
- remove all the duplicate parts of the css that appears in both light
and dark theme style sheets to reflect only the changes needed
- this css was inline and so couldn't be overridden in dark theme style
sheet
- stops playlists from scrolling off screen when you scroll the tracks
in a long playlist
- may as well move them here to clean up the Left Side Panel a bit more
Remove links to other musicbox's until ability to configure their IP's
in settings.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 32.69% when pulling c11d252 on southafricanrob:develop into a4b6d78 on woutervanwijk:develop.

@kingosticks
Copy link
Member

I think we will hold off on 1) and 2), for now at least. Sadly my attempts at cherry-picking the rest are meeting with little cooperation from git.

@jcass77
Copy link
Member

jcass77 commented Mar 28, 2016

Closing this PR as these changes seem to have been taken further in #89 instead.

@jcass77 jcass77 closed this Mar 28, 2016
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.

4 participants