Skip to content

Commit

Permalink
add reduced view for js viewer.
Browse files Browse the repository at this point in the history
  • Loading branch information
siegy22 committed Jul 28, 2016
1 parent e6780e9 commit 6164550
Show file tree
Hide file tree
Showing 8 changed files with 37 additions and 4 deletions.
10 changes: 9 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ gem 'pdfjs_viewer-rails'

## Viewer Styles

This gem ships with two viewer styles:
This gem ships with three viewer styles:

**full**

Expand All @@ -24,6 +24,14 @@ This gem ships with two viewer styles:

**minimal**

<p align="center">
<img
src="https://raw.githubusercontent.com/senny/pdfjs_viewer-rails/master/doc/files/viewer_reduced.png"
alt="reduced style"/>
</p>

**minimal**

<p align="center">
<img
src="https://raw.githubusercontent.com/senny/pdfjs_viewer-rails/master/doc/files/viewer_minimal.png"
Expand Down
1 change: 1 addition & 0 deletions app/assets/stylesheets/pdfjs_viewer/application.css
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
*
*= require pdfjs_viewer/pdfjs/viewer
*= require pdfjs_viewer/minimal
*= require pdfjs_viewer/reduced
*= require pdfjs_viewer/full
*= require_self
*/
10 changes: 10 additions & 0 deletions app/assets/stylesheets/pdfjs_viewer/reduced.scss
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
#pdfjs_viewer-reduced {
.openFile { display: none; }
.bookmark { display: none; }
#viewThumbnail { display: none; }
#viewOutline { display: none; }
#viewAttachments { display: none; }
#download { display: none; }
#secondaryToolbarToggle { display: none; }
#toolbarViewerRight div.verticalToolbarSeparator { display: none; }
}
1 change: 1 addition & 0 deletions app/views/pdfjs_viewer/viewer/reduced.html.erb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
<%= render "viewer", style: "reduced" %>
1 change: 1 addition & 0 deletions config/routes.rb
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
PdfjsViewer::Rails::Engine.routes.draw do
get "minimal" => "viewer#minimal", as: :minimal
get "reduced" => "viewer#reduced", as: :reduced
get "full" => "viewer#full", as: :full
end
Binary file added doc/files/viewer_reduced.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
7 changes: 4 additions & 3 deletions test/dummy/app/views/sample/show.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@
</p>

<ul>
<li><%= link_to "Open sample.pdf using the full viewer", pdfjs.full_path(file: "/sample.pdf") %></li>
<li><%= link_to "Open sample.pdf using the minimal viewer", pdfjs.minimal_path(file: "/sample.pdf") %></li>
<li><%= link_to "Open sample.pdf using the pdfjs_viewer helper", helper_path %></li>
<li><%= link_to "Open sample.pdf using the full viewer", pdfjs.full_path(file: "/sample.pdf") %></li>
<li><%= link_to "Open sample.pdf using the reduced viewer", pdfjs.reduced_path(file: "/sample.pdf") %></li>
<li><%= link_to "Open sample.pdf using the minimal viewer", pdfjs.minimal_path(file: "/sample.pdf") %></li>
<li><%= link_to "Open sample.pdf using the pdfjs_viewer helper", helper_path %></li>
</ul>
11 changes: 11 additions & 0 deletions test/integration/viewer_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,17 @@ class ViewerTest < ActionDispatch::IntegrationTest
assert_rendered_pdf output, screenshot: SANDBOX_PATH + "full_viewer.png"
end

test "Reduced viewer" do
visit "/"
output = capture(:stdout) do
click_on "reduced viewer"
assert_equal 1, all("#pdfjs_viewer-reduced").size
sleep @time_to_render
end

assert_rendered_pdf output, screenshot: SANDBOX_PATH + "reduced_viewer.png"
end

test "Minimal viewer" do
visit "/"
output = capture(:stdout) do
Expand Down

0 comments on commit 6164550

Please sign in to comment.