Browse Source

Merge pull request #116 from ilya-konanykhin/issue-115

Issue #115: New top menu for desktop & mobile.
master
Jason Denney 8 years ago
committed by GitHub
parent
commit
9df4aa38d9
  1. 9
      app/assets/stylesheets/frontend.scss
  2. 22
      app/helpers/application_helper.rb
  3. 2
      app/views/bikes/new.html.haml
  4. 6
      app/views/bikes/show.html.haml
  5. 2
      app/views/task_lists/edit.haml
  6. 2
      app/views/time_entries/index.haml
  7. 2
      app/views/time_entries/new.haml

9
app/assets/stylesheets/frontend.scss

@ -58,3 +58,12 @@ fieldset, .fieldset {
padding-left: 6px; padding-left: 6px;
padding-right: 6px; padding-right: 6px;
} }
.navbar-default {
.navbar-text {
width: 100%;
font-size: 150%;
margin-top: 10px;
margin-bottom: 10px;
}
}

22
app/helpers/application_helper.rb

@ -8,21 +8,27 @@ module ApplicationHelper
link_to(name, '#', id: "add_#{association.to_s.singularize}" , class: "add_fields", data: {id: id, fields: fields.gsub("\n", "")}) link_to(name, '#', id: "add_#{association.to_s.singularize}" , class: "add_fields", data: {id: id, fields: fields.gsub("\n", "")})
end end
def top_menu(left, right = nil) def top_menu(right = nil)
content_tag :div do content_tag :nav, class: 'navbar navbar-default' do
content_tag :div, class: 'container-fluid' do
content_tag :div, class: 'row' do
[ [
content_tag(:div, left, class: 'pull-left'), content_tag(:div, link_to_dashboard, class: 'col-xs-3'),
content_tag(:div, right, class: 'pull-right'), content_tag(:div, header_logo, class: 'col-xs-6'),
content_tag(:div, nil, class: 'clearfix'), content_tag(:div, class: 'col-xs-3') { content_tag :div, right, class: 'pull-right' },
].join.html_safe ].join.html_safe
end end
end end
end
end
def link_to_dashboard def link_to_dashboard
content_tag :p do link_to root_path, class: 'navbar-brand', title: 'Back to dashboard' do
content_tag :a, href: root_path, class: 'btn btn-default', title: 'Back to dashboard' do content_tag :i, '', class: "glyphicon glyphicon-home"
content_tag :span, '', class: 'glyphicon glyphicon-home'
end end
end end
def header_logo
content_tag :div, 'Velocipede', class: 'navbar-text text-center'
end end
end end

2
app/views/bikes/new.html.haml

@ -1,4 +1,4 @@
= top_menu link_to_dashboard = top_menu
%h1 Add Bike %h1 Add Bike

6
app/views/bikes/show.html.haml

@ -1,8 +1,10 @@
- btn = if @show_add_bike - btn = if @show_add_bike
- capture do - capture do
= link_to 'Add Another Bike?', new_bike_path, class: 'btn btn-default' = link_to 'Add Another Bike?', new_bike_path, class: 'btn btn-default navbar-btn hidden-xs'
= link_to new_bike_path, class: 'navbar-brand visible-xs', title: 'Add Another Bike?' do
%i.glyphicon.glyphicon-plus
= top_menu link_to_dashboard, btn = top_menu btn
%h1 #{@bike.shop_id}: #{@bike.bike_brand} %h1 #{@bike.shop_id}: #{@bike.bike_brand}
%h4= @bike.model %h4= @bike.model

2
app/views/task_lists/edit.haml

@ -1,4 +1,4 @@
= top_menu link_to_dashboard = top_menu
%h1 Task List %h1 Task List

2
app/views/time_entries/index.haml

@ -1,4 +1,4 @@
= top_menu link_to_dashboard = top_menu
%h1 Your Timesheet %h1 Your Timesheet

2
app/views/time_entries/new.haml

@ -1,4 +1,4 @@
= top_menu link_to_dashboard = top_menu
%h1 Add Time Entry %h1 Add Time Entry
- time = Time.now - time = Time.now

Loading…
Cancel
Save