Merge branch 'master' of github.com:magnolia-fan/BeatHaven

Gemfile

@@ -14,8 +14,8 @@ gem 'barista'
 
 gem 'awesome_print', :require => 'ap'
 gem 'delayed_job'
-gem 'lastfm-client', :git => 'http://github.com/magnolia-fan/lastfm-client.git'
-gem 'musicbrainz', '~> 0.4.3'
+gem 'lastfm-client', :git => 'http://github.com/pch/lastfm-client.git'
+gem 'musicbrainz'
 
 gem 'bitmask_attributes'
 

Gemfile.lock

@@ -1,6 +1,6 @@
 GIT
-  remote: http://github.com/magnolia-fan/lastfm-client.git
-  revision: 26fec825943d0bb77606bf058386e786ee97572b
+  remote: http://github.com/pch/lastfm-client.git
+  revision: d17806722c1da685f7d2f283b0064cb92eb96965
   specs:
     lastfm-client (0.0.2)
       json (>= 1.4.6)
@@ -128,7 +128,7 @@ DEPENDENCIES
   haml
   json
   lastfm-client!
-  musicbrainz (~> 0.4.3)
+  musicbrainz
   nokogiri
   pg
   rails (= 3.1.0)

app/assets/javascripts/ajax.coffee

@@ -9,11 +9,11 @@ class window.Ajax
   
   loadIndexPage: ->
     $('#content').load '/greetings/'
+    _ajax.setTitle ''
     false
   
   loadAboutPage: ->
-    $.get '/templates/about.html', (data) ->
-      _page.renderTextpage data
+    $('#content').load '/about/'
     _ajax.setTitle 'About'
     false
   
@@ -34,14 +34,12 @@ class window.Ajax
   detectPage: () ->
     if m = _ajax.getAnchor().match /\/artist\/(.+)\//
       _search.loadArtistData m[1]
-    else if _ajax.getAnchor() == ''
-      _ajax.loadIndexPage();
     else if _ajax.getAnchor().match /\/settings\//
       _settings.loadSettingsPage()
     else if _ajax.getAnchor().match /\/about\//
       _ajax.loadAboutPage()
     else
-      #_ajax.loadSearchPage()
+      _ajax.loadIndexPage();
     false
 
 $(window).bind 'hashchange', ->

app/assets/javascripts/page.coffee

@@ -29,6 +29,9 @@ class window.Page
     false
 
 $ ->
+  $('#logo').live 'click', ->
+    _ajax.go '/'
+    false
   $('.about').live 'click', ->
     _ajax.go '/about/'
     false

app/assets/javascripts/player.coffee

@@ -147,10 +147,10 @@ class window.Player
     false
   
   onShuffle: ->
-    #return $('#shuffle').hasClass 'active'
+    return $('.shuffle').hasClass 'on'
   
   onRepeat: ->
-    #return $('#repeat').hasClass 'active'
+    return $('.repeat').hasClass 'on'
   
   updateNowListening: (track) ->
     if _session.getUser().lastfm_username
@@ -184,11 +184,11 @@ $('.player .progress').live 'click', (e) ->
 
 # Player Additional Controls
 
-$('#repeat, #shuffle').live 'click', ->
-  $(this).toggleClass 'active'
+$('.repeat, .shuffle').live 'click', ->
+  $(this).toggleClass 'on'
   false
 
-$('#empty-playlist').live 'click', ->
+$('.do_empty').live 'click', ->
   if confirm('Are you sure?')
     $('.playlist li').remove()
     $('#jplayer').jPlayer 'clearMedia'

app/assets/javascripts/search.coffee

@@ -19,8 +19,10 @@ class window.Search
       if data.status in ['ok', 'loading']
         _ajax.setArchor '/artist/' +name+ '/'
         _page.print data.html
+        if _session.getUser().id
           for album in data.albums
             _player.albums[album.id] = album
+          $('.button-container').show()
         _search.hideSpinner()
         if data.status is 'loading'
           setTimeout () ->

app/assets/javascripts/vkontakte.coffee

@@ -132,8 +132,8 @@ $('#login').live 'click', ->
   , 8
   false
 $('#logout').live 'click', ->
-  _ajax.go '/search/';
   VK.Auth.logout (response) ->
     _vkontakte.authInfo(response)
+    $('.button-container').hide()
     false
   false

app/assets/stylesheets/albums.scss

@@ -19,6 +19,7 @@
       max-width: 220px;
     }
     .button-container {
+      display: none;
       text-align: center;
       margin-top: 10px;
     }

app/assets/stylesheets/player.scss

@@ -34,6 +34,24 @@
       }
     }
   }
+  .secondary-buttons {
+    margin: 10px 0;
+    text-align: center;
+    a {
+      font-size: 11px;
+      color: #cacaca;
+      font-weight: bold;
+      text-decoration: none;
+      cursor: pointer;
+      
+      &:hover {
+        color: #b0b0b0;
+      }
+      &.on {
+        color: #606060;
+      }
+    }
+  }
 }
 
 .playlist {

app/controllers/application_controller.rb

@@ -11,6 +11,10 @@ class ApplicationController < ActionController::Base
     render :partial => 'greetings'
   end
   
+  def about
+    render :partial => 'about'
+  end
+  
   protected
   
   def authorize

app/views/application/_about.en.html.haml

@@ -0,0 +1,13 @@
+%h2 Write us a letter!
+%br
+.row
+  .span3
+    %address
+      %strong Site issues?
+      %br
+      %a{ :href => "mailto:support@beathaven.org" } support@beathaven.org
+  .span6
+    %address
+      %strong Everything else (suggestions, compliments, threats)
+      %br
+      %a{ :href => "mailto:contact@beathaven.org" } contact@beathaven.org

app/views/application/_about.ru.html.haml

@@ -0,0 +1,13 @@
+%h2 Напишите нам письмо!
+%br
+.row
+  .span3
+    %address
+      %strong Проблемы с сайтом?
+      %br
+      %a{ :href => "mailto:support@beathaven.org" } support@beathaven.org
+  .span6
+    %address
+      %strong Все остальное (пожелания, благодарности, угрозы)
+      %br
+      %a{ :href => "mailto:contact@beathaven.org" } contact@beathaven.org

app/views/application/_greetings.en.html.haml

@@ -0,0 +1 @@
+%h2 Halló!

app/views/application/_greetings.ru.html.haml

@@ -1 +1 @@
-%h1 Привет!
+%h2 Привет!

app/views/layouts/application.html.haml

@@ -15,10 +15,10 @@
         .topbar-inner
           .container
             %h3
-              %a{ :href => "#/" } BeatHaven
+              %a#logo{ :href => "#/" } BeatHaven
             %ul.nav
               %li
-                %a{ :href => "http://blog.beathaven.org/" }= I18n.t 'global.news'
+                %a{ :href => "http://blog.beathaven.org/", :target => "_blank" }= I18n.t 'global.news'
               %li
                 %a.about{ :href => "#/about/" }= I18n.t 'global.about' 
             %form#search-form{ :action => "" }
@@ -61,6 +61,10 @@
               %a.btn.large.play ►
               %a.btn.large.pause II
               %a.btn.small-round.next »
+            .secondary-buttons
+              %a.shuffle= (I18n.t 'player.shuffle').upcase
+              %a.repeat= (I18n.t 'player.repeat').upcase
+              %a.do_empty= (I18n.t 'player.do_empty').upcase
 
           %ul.playlist
     

config/locales/en.yml

@@ -7,9 +7,6 @@ en:
     logout: "Log out"
     do_add: "Add some music to playlist"
     do_login: "Don't forget to log in, please. It's simple."
-    repeat: "Repeat"
-    shuffle: "Shuffle"
-    do_empty: "Empty playlist"
     hello: "Hi there"
     settings: "Settings"
     your_ad_here: "Your ad couldn't<br/>be here"
@@ -49,3 +46,6 @@ en:
   
   player:
     add: "Add to Now Playing"
+    repeat: "Repeat"
+    shuffle: "Shuffle"
+    do_empty: "Empty"

config/locales/ru.yml

@@ -7,9 +7,6 @@ ru:
     logout: "Выйти"
     do_add: "Добавьте музыку в плей-лист"
     do_login: "Авторизуйтесь, пожалуйста. Это действительно просто."
-    repeat: "Повторять"
-    shuffle: "Перемешать"
-    do_empty: "Очистить"
     hello: "Привет"
     settings: "Настройки"
     your_ad_here: "Здесь не могла бы<br/>быть Ваша реклама"
@@ -49,3 +46,6 @@ ru:
   
   player:
     add: "Добавить в плей-лист"
+    repeat: "Повторять"
+    shuffle: "Перемешать"
+    do_empty: "Очистить"

config/routes.rb

@@ -58,6 +58,7 @@ Beathaven::Application.routes.draw do
   
   match '/' => 'application#index'
   match '/greetings/' => 'application#greetings'
+  match '/about/' => 'application#about'
   
   match 'user/auth' => 'user#auth'
   match 'user/update' => 'user#update'