diff --git a/app/assets/stylesheets/participacion.scss b/app/assets/stylesheets/participacion.scss index 88ed47d65..d4442e11c 100644 --- a/app/assets/stylesheets/participacion.scss +++ b/app/assets/stylesheets/participacion.scss @@ -12,6 +12,7 @@ // 10. User account // 11. Filters // 12. Official levels +// 13. Pagination // // 01. Variables @@ -673,3 +674,14 @@ img.initialjs-avatar { .level-5 { background: $level-5; } + +// 13. Pagination +// - - - - - - - - - - - - - - - - - - - - - - - - - + +ul.pagination { + margin-top: rem-calc(24); + + li { + color: $text-medium; + } +} diff --git a/app/views/kaminari/_first_page.html.erb b/app/views/kaminari/_first_page.html.erb index 2682d3415..b9802389d 100644 --- a/app/views/kaminari/_first_page.html.erb +++ b/app/views/kaminari/_first_page.html.erb @@ -1,11 +1,3 @@ -<%# Link to the "First" page - - available local variables - url: url to the first page - current_page: a page object for the currently displayed page - total_pages: total number of pages - per_page: number of items to fetch per page - remote: data-remote --%> - - <%= link_to_unless current_page.first?, t('views.pagination.first').html_safe, url, :remote => remote %> - +
  • + <%= link_to t("views.pagination.first").html_safe, url, :remote => remote %> +
    • diff --git a/app/views/kaminari/_gap.html.erb b/app/views/kaminari/_gap.html.erb index bbb0f9839..288e7c69b 100644 --- a/app/views/kaminari/_gap.html.erb +++ b/app/views/kaminari/_gap.html.erb @@ -1,8 +1,3 @@ -<%# Non-link tag that stands for skipped pages... - - available local variables - current_page: a page object for the currently displayed page - total_pages: total number of pages - per_page: number of items to fetch per page - remote: data-remote --%> -<%= t('views.pagination.truncate').html_safe %> +
  • + <%= t("views.pagination.truncate").html_safe %> +
    • diff --git a/app/views/kaminari/_last_page.html.erb b/app/views/kaminari/_last_page.html.erb index 53d613074..179f6e7fc 100644 --- a/app/views/kaminari/_last_page.html.erb +++ b/app/views/kaminari/_last_page.html.erb @@ -1,11 +1,3 @@ -<%# Link to the "Last" page - - available local variables - url: url to the last page - current_page: a page object for the currently displayed page - total_pages: total number of pages - per_page: number of items to fetch per page - remote: data-remote --%> - - <%= link_to_unless current_page.last?, t('views.pagination.last').html_safe, url, :remote => remote %> - +
  • + <%= link_to t("views.pagination.last").html_safe, url, :remote => remote %> +
    • diff --git a/app/views/kaminari/_next_page.html.erb b/app/views/kaminari/_next_page.html.erb index 4fc20712c..05695cd6f 100644 --- a/app/views/kaminari/_next_page.html.erb +++ b/app/views/kaminari/_next_page.html.erb @@ -1,11 +1,3 @@ -<%# Link to the "Next" page - - available local variables - url: url to the next page - current_page: a page object for the currently displayed page - total_pages: total number of pages - per_page: number of items to fetch per page - remote: data-remote --%> - - <%= link_to_unless current_page.last?, t('views.pagination.next').html_safe, url, :rel => 'next', :remote => remote %> - +
  • + <%= link_to t("views.pagination.next").html_safe, url, :rel => "next", :remote => remote %> +
    • diff --git a/app/views/kaminari/_page.html.erb b/app/views/kaminari/_page.html.erb index 582af7bc9..329f36b3f 100644 --- a/app/views/kaminari/_page.html.erb +++ b/app/views/kaminari/_page.html.erb @@ -1,12 +1,3 @@ -<%# Link showing page number - - available local variables - page: a page object for "this" page - url: url to this page - current_page: a page object for the currently displayed page - total_pages: total number of pages - per_page: number of items to fetch per page - remote: data-remote --%> - - <%= link_to_unless page.current?, page, url, {:remote => remote, :rel => page.next? ? 'next' : page.prev? ? 'prev' : nil} %> - +
  • + <%= link_to page, url, {:remote => remote, :rel => page.next? ? "next" : page.prev? ? "prev" : nil} %> +
    • diff --git a/app/views/kaminari/_paginator.html.erb b/app/views/kaminari/_paginator.html.erb index 4fb445fd3..6e15a545d 100644 --- a/app/views/kaminari/_paginator.html.erb +++ b/app/views/kaminari/_paginator.html.erb @@ -1,23 +1,21 @@ -<%# The container tag - - available local variables - current_page: a page object for the currently displayed page - total_pages: total number of pages - per_page: number of items to fetch per page - remote: data-remote - paginator: the paginator that renders the pagination tags inside --%> <%= paginator.render do -%> - +
    + +
    <% end -%> diff --git a/app/views/kaminari/_prev_page.html.erb b/app/views/kaminari/_prev_page.html.erb index 9c4aff49c..f4c5c4041 100644 --- a/app/views/kaminari/_prev_page.html.erb +++ b/app/views/kaminari/_prev_page.html.erb @@ -1,11 +1,3 @@ -<%# Link to the "Previous" page - - available local variables - url: url to the previous page - current_page: a page object for the currently displayed page - total_pages: total number of pages - per_page: number of items to fetch per page - remote: data-remote --%> - - <%= link_to_unless current_page.first?, t('views.pagination.previous').html_safe, url, :rel => 'prev', :remote => remote %> - +
  • + <%= link_to t("views.pagination.previous").html_safe, url, :rel => "prev", :remote => remote %> +
    • diff --git a/config/i18n-tasks.yml b/config/i18n-tasks.yml index 14d8b82a7..cb06a3360 100644 --- a/config/i18n-tasks.yml +++ b/config/i18n-tasks.yml @@ -26,6 +26,7 @@ data: - config/locales/moderation.%{locale}.yml - config/locales/devise_views.%{locale}.yml - config/locales/responders.%{locale}.yml + - config/locales/kaminari.%{locale}.yml # Locale files to write new keys to, based on a list of key pattern => file rules. Matched from top to bottom: # `i18n-tasks normalize -p` will force move the keys according to these rules @@ -98,6 +99,8 @@ ignore_unused: - 'unauthorized.*' - 'simple_captcha.*' - 'admin.officials.level_*' + - 'helpers.page_entries_info.*' # kaminari + - 'views.pagination.*' # kaminari # - '{devise,kaminari,will_paginate}.*' # - 'simple_form.{yes,no}' # - 'simple_form.{placeholders,hints,labels}.*' diff --git a/spec/features/comments_spec.rb b/spec/features/comments_spec.rb index 60f06f169..b820fffcd 100644 --- a/spec/features/comments_spec.rb +++ b/spec/features/comments_spec.rb @@ -27,7 +27,7 @@ feature 'Comments' do visit debate_path(debate) expect(page).to have_css('.comment', count: per_page) - within("nav.pagination") do + within("ul.pagination") do expect(page).to have_content("1") expect(page).to have_content("2") expect(page).to_not have_content("3") diff --git a/spec/features/debates_spec.rb b/spec/features/debates_spec.rb index cd0706ca8..eade338f5 100644 --- a/spec/features/debates_spec.rb +++ b/spec/features/debates_spec.rb @@ -24,7 +24,7 @@ feature 'Debates' do expect(page).to have_selector('#debates .debate', count: per_page) - within("nav.pagination") do + within("ul.pagination") do expect(page).to have_content("1") expect(page).to have_content("2") expect(page).to_not have_content("3")