From 83eee97e950155f2cfdcf151c4f10814822c1519 Mon Sep 17 00:00:00 2001 From: Jonathan Frederic Date: Fri, 6 Jun 2014 16:32:10 -0700 Subject: [PATCH] Review comments --- IPython/html/static/base/js/dialog.js | 5 +- IPython/html/static/base/less/variables.less | 23 +++++++++- IPython/html/static/notebook/js/quickhelp.js | 2 +- IPython/html/static/notebook/js/savewidget.js | 2 +- IPython/html/static/notebook/less/cell.less | 2 +- .../static/notebook/less/celltoolbar.less | 46 +++---------------- .../static/notebook/less/renderedhtml.less | 4 +- IPython/html/static/style/ipython.min.css | 19 ++++---- IPython/html/static/style/style.min.css | 45 +++++++++--------- IPython/html/static/tree/css/override.css | 2 +- IPython/html/static/tree/less/tree.less | 4 +- 11 files changed, 73 insertions(+), 81 deletions(-) diff --git a/IPython/html/static/base/js/dialog.js b/IPython/html/static/base/js/dialog.js index 9be280a64..e81a4be71 100644 --- a/IPython/html/static/base/js/dialog.js +++ b/IPython/html/static/base/js/dialog.js @@ -128,7 +128,7 @@ IPython.dialog = (function (IPython) { autoIndent: true, mode: 'application/json', }); - IPython.dialog.modal({ + var modal = IPython.dialog.modal({ title: "Edit " + name + " Metadata", body: dialogform, buttons: { @@ -149,7 +149,8 @@ IPython.dialog = (function (IPython) { Cancel: {} } }); - editor.refresh(); + + modal.on('shown.bs.modal', function(){ editor.refresh(); }); }; return { diff --git a/IPython/html/static/base/less/variables.less b/IPython/html/static/base/less/variables.less index 74fc789be..9e0708e3d 100644 --- a/IPython/html/static/base/less/variables.less +++ b/IPython/html/static/base/less/variables.less @@ -1,15 +1,31 @@ // Our customizations to bootstrap go here. -@black: @gray-darker; +@black: #000; @text-color: @black; @font-size-base: 13px; @font-family-monospace: monospace; // to allow user to customize their fonts @navbar-height: 36px; +@breadcrumb-color: darken(@border_color, 30%); +@blockquote-font-size: inherit; +// Override the modal footer padding that BS3 uses. +.modal-footer { + padding: 7px; +} + +// Disable modal slide-in from top animation. +.modal { + &.fade .modal-dialog { + .translate(0, 0); + } +} + +// Set the default code color. code { color: @black; // default code color in bootstrap is #d14 (crimson / amaranth) } +// Override bootstrap pre element styling. pre { // bootstrap has pre defaults that we don't want to inherit. // start pre tag defaults based on the surrounding context instead. @@ -17,5 +33,10 @@ pre { line-height: inherit; } +// Disable bold labels in BS3 +label { + font-weight: normal; +} + // Our own global variables for all pages go here diff --git a/IPython/html/static/notebook/js/quickhelp.js b/IPython/html/static/notebook/js/quickhelp.js index 76c4ae438..1b80698ce 100644 --- a/IPython/html/static/notebook/js/quickhelp.js +++ b/IPython/html/static/notebook/js/quickhelp.js @@ -77,7 +77,7 @@ var IPython = (function (IPython) { var element = $('
'); // The documentation - var doc = $('
').addClass('alert'); + var doc = $('
').addClass('alert alert-warning'); doc.append( $('