mobile_pagination 0.0.8 → 0.0.9

data/README.md

@@ -26,7 +26,7 @@ Or install it yourself as:
 
 
 ## Usage
-Initializes with 4 options: `current_page, total_pages, query, path`
+Initializes with 4 options: `current_page, total_pages, query, path`.
 
 ```ruby
 # in your view, you may have something like this:
@@ -49,15 +49,28 @@ def opts
 end
 
 def pagination
-  @pagination ||= MobilePagination::Paginate.new(opts).html
+  @pagination ||= MobilePagination::Paginate.new(opts)
 end
 
 def pagination_html
-  pagination.html
+  pagination.html # => returns paginated elements
 end
 
+# Other public methods
+
+previous_page_link  # => path for the previous page
+next_page_link      # => path for the next page
+
+previous_page?      # => true or false
+next_page?          # => true or false
+should_paginate?    # => true or false
+
+# Readers
+current_page, total_pages query, path
 ```
 
+Take a look at lib/mobile_pagination/paginate.rb for more information.
+
 ### Overrides
 
 If you don't like list items, you will need to override `MobilePagination::Templates`
@@ -69,7 +82,7 @@ module MobilePagination
   module Templates
 
     def first_page_html
-      "<p'>
+      "<p>
         <a title='First Page' href='#{first_page_link}'>Luke's Page</a> |
       <p>"
     end

data/lib/mobile_pagination/paginate.rb

@@ -16,42 +16,50 @@ module MobilePagination
       @path         = opts[:path] || '/'
     end
 
-    def first_page_link
-      "#{page_url}"
-    end
-
     def previous_page_link
       previous == 1 ? "#{first_page_link}" : "#{page_url(previous)}"
     end
 
     def next_page_link
-      "#{page_url(@current_page + 1)}" if @current_page < @total_pages
+      "#{page_url(@current_page + 1)}"
+    end
+
+    def first_page_link
+      "#{page_url}"
     end
 
     def last_page_link
       "#{page_url(@total_pages)}"
     end
 
-    private
+    def previous_page?
+      @current_page > 1
+    end
 
-      def should_paginate?
-        @total_pages > 1
-      end
+    def next_page?
+      @current_page < @total_pages
+    end
+
+    def should_paginate?
+      @total_pages > 1
+    end
+
+    private
 
-      def first_page?
+      def display_first_page?
         @current_page != 1
       end
 
-      def previous_page?
-        @current_page > 2
+      def display_next_page?
+        display_last_page? && !second_to_last?
       end
 
-      def next_page?
-        last_page? && !second_to_last?
+      def display_last_page?
+        @current_page != @total_pages
       end
 
-      def last_page?
-        @current_page != @total_pages
+      def display_previous_page?
+        @current_page > 2
       end
 
       def second_to_last?

data/lib/mobile_pagination/templates.rb

@@ -2,38 +2,44 @@ module MobilePagination
   module Templates
 
     def first_page_html
-      "<li>
-        <a title='First Page' href='#{first_page_link}'>First Page</a> |
-      </li>"
+      generate_pagination_link title: 'First Page',
+                               href: first_page_link,
+                               text: 'First Page'
     end
 
     def previous_page_html
-      "<li>
-        <a title='Previous Page' href='#{previous_page_link}'>Previous Page</a> |
-      </li>"
+      generate_pagination_link title: 'Previous Page',
+                               href: previous_page_link,
+                               text: 'Previous Page'
     end
 
     def next_page_html
-      "<li>
-        <a title='Next Page' href='#{next_page_link}'>Next Page</a> |
-      </li>"
+      generate_pagination_link title: 'Next Page',
+                               href: next_page_link,
+                               text: 'Next Page'
     end
 
     def last_page_html
-      "<li>
-        <a title='Last Page' href='#{last_page_link}'>Last Page</a>
-      </li>"
+      generate_pagination_link title: 'Last Page',
+                               href: last_page_link,
+                               text: 'Last Page'
     end
 
     def html
       return '' unless should_paginate?
       ''.tap do |markup|
-        markup << first_page_html    if first_page?
-        markup << previous_page_html if previous_page?
-        markup << next_page_html     if next_page?
-        markup << last_page_html     if last_page?
+        markup << first_page_html    if display_first_page?
+        markup << previous_page_html if display_previous_page?
+        markup << next_page_html     if display_next_page?
+        markup << last_page_html     if display_last_page?
       end
     end
 
+    def generate_pagination_link(opts)
+      "<li>
+        <a title='#{opts[:title]}' href='#{opts[:href]}'>#{opts[:text]}</a>
+      </li>"
+    end
+
   end
-end
+end

data/lib/mobile_pagination/version.rb

@@ -1,3 +1,3 @@
 module MobilePagination
-  VERSION = "0.0.8"
+  VERSION = "0.0.9"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mobile_pagination
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.9
   prerelease: 
 platform: ruby
 authors: