Skip to content
This repository

Totally remove pager when read only (notebook) #1126

Closed
wants to merge 1 commit into from

2 participants

Matthias Bussonnier Min RK
Matthias Bussonnier
Collaborator

I don't see any use of having a pager in Read only mode, so I added it to the removed element.
Not 'hidden' because otherwise there is still 2 blank pixel at the bottom of the screen.

Min RK minrk closed this in aa46457
Brian E. Granger ellisonbg referenced this pull request from a commit
Commit has since been removed from the repository and is no longer available.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

Dec 08, 2011
Matthias Bussonnier totally remove pager when read only 107e082
This page is out of date. Refresh to see the latest.
2  IPython/frontend/html/notebook/static/js/notebookmain.js
@@ -99,6 +99,8 @@ $(document).ready(function () {
99 99
         // hide various elements from read-only view
100 100
         IPython.save_widget.element.find('button#save_notebook').addClass('hidden');
101 101
         IPython.quick_help.element.addClass('hidden'); // shortcuts are disabled in read_only
  102
+        $('div#pager').remove();
  103
+        $('div#pager_splitter').remove();
102 104
         $('button#new_notebook').addClass('hidden');
103 105
         $('div#cell_section').addClass('hidden');
104 106
         $('div#config_section').addClass('hidden');
Commit_comment_tip

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.