Skip to content

Commit

Permalink
Slideout: Converted space from v3.0 to i18n interword space.
Browse files Browse the repository at this point in the history
  • Loading branch information
Paul Jackson authored and Paul Jackson committed Sep 20, 2013
1 parent 2ea5703 commit fcdd5e4
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/js/workers/slideout.js
Expand Up @@ -19,7 +19,9 @@
var borderWidth = 10,
tocText = _pe.dic.get('%table-contents'),
hideText = _pe.dic.get('%hide'),
closeLink = hideText + '<span class="wb-invisible"> ' + tocText + '</span>',
space = _pe.dic.get('%interword-space'),
%interword-space

This comment has been minimized.

Copy link
@nschonni

nschonni Sep 20, 2013

Member

Missing comma

This comment has been minimized.

Copy link
@pjackson28

pjackson28 Sep 20, 2013

Member

Oops, looks like I accidentally hit undo after deleting that (and after JSHinting). Will fix.

This comment has been minimized.

Copy link
@pjackson28

pjackson28 Sep 20, 2013

Member

Fixed now. Thanks for catching that.

closeLink = hideText + '<span class="wb-invisible">' + space + tocText + '</span>',
focusOutlineAllowance = 2,
opened = false,
reposition,
Expand Down Expand Up @@ -302,7 +304,7 @@
slideoutClose = elm.find('#slideoutClose');

// Add the slideout toggle
innerWrapper.css('padding', (focusOutlineAllowance / 2) + 'px').prepend('<div id="slideoutToggle" class="slideoutToggle"><a id="toggleLink" role="button" aria-controls="slideout-body" aria-label="' + opts.txtShow + '" href="#" onclick="return false;">' + opts.txtShow + '</a></div>');
innerWrapper.css('padding', (focusOutlineAllowance / 2) + 'px').prepend('<div id="slideoutToggle" class="slideoutToggle"><a id="toggleLink" role="button" aria-controls="slideout-body" href="#" onclick="return false;">' + opts.txtShow + '</a></div>');
tab = innerWrapper.find('#slideoutToggle');
toggleLink = innerWrapper.find('#toggleLink');

Expand Down

0 comments on commit fcdd5e4

Please sign in to comment.