Add horizontal popup offset

This commit is contained in:
Klemen Košir 2018-10-02 23:27:59 +09:00
parent 39cf302eef
commit 99a231a773
4 changed files with 14 additions and 8 deletions

View File

@ -192,7 +192,8 @@ function optionsSetDefaults(options) {
showAdvanced: false, showAdvanced: false,
popupWidth: 400, popupWidth: 400,
popupHeight: 250, popupHeight: 250,
popupOffset: 10, popupHorizontalOffset: 0,
popupVerticalOffset: 10,
showGuide: true, showGuide: true,
compactTags: false, compactTags: false,
compactGlossaries: false, compactGlossaries: false,

View File

@ -33,7 +33,8 @@ async function formRead() {
optionsNew.general.maxResults = parseInt($('#max-displayed-results').val(), 10); optionsNew.general.maxResults = parseInt($('#max-displayed-results').val(), 10);
optionsNew.general.popupWidth = parseInt($('#popup-width').val(), 10); optionsNew.general.popupWidth = parseInt($('#popup-width').val(), 10);
optionsNew.general.popupHeight = parseInt($('#popup-height').val(), 10); optionsNew.general.popupHeight = parseInt($('#popup-height').val(), 10);
optionsNew.general.popupOffset = parseInt($('#popup-offset').val(), 10); optionsNew.general.popupHorizontalOffset = parseInt($('#popup-horizontal-offset').val(), 0);
optionsNew.general.popupVerticalOffset = parseInt($('#popup-vertical-offset').val(), 10);
optionsNew.scanning.middleMouse = $('#middle-mouse-button-scan').prop('checked'); optionsNew.scanning.middleMouse = $('#middle-mouse-button-scan').prop('checked');
optionsNew.scanning.selectText = $('#select-matched-text').prop('checked'); optionsNew.scanning.selectText = $('#select-matched-text').prop('checked');
@ -163,7 +164,8 @@ async function onReady() {
$('#max-displayed-results').val(options.general.maxResults); $('#max-displayed-results').val(options.general.maxResults);
$('#popup-width').val(options.general.popupWidth); $('#popup-width').val(options.general.popupWidth);
$('#popup-height').val(options.general.popupHeight); $('#popup-height').val(options.general.popupHeight);
$('#popup-offset').val(options.general.popupOffset); $('#popup-horizontal-offset').val(options.general.popupHorizontalOffset);
$('#popup-vertical-offset').val(options.general.popupVerticalOffset);
$('#middle-mouse-button-scan').prop('checked', options.scanning.middleMouse); $('#middle-mouse-button-scan').prop('checked', options.scanning.middleMouse);
$('#select-matched-text').prop('checked', options.scanning.selectText); $('#select-matched-text').prop('checked', options.scanning.selectText);

View File

@ -93,8 +93,11 @@
</div> </div>
<div class="form-group options-advanced"> <div class="form-group options-advanced">
<label for="popup-offset">Popup offset (in pixels)</label> <label>Popup offset (horizontal, vertical; in pixels)</label>
<input type="number" min="0" id="popup-offset" class="form-control"> <div class="row">
<div class="col-xs-6"><input type="number" min="0" id="popup-horizontal-offset" class="form-control"></div>
<div class="col-xs-6"><input type="number" min="0" id="popup-vertical-offset" class="form-control"></div>
</div>
</div> </div>
</div> </div>

View File

@ -50,7 +50,7 @@ class Popup {
const limitX = document.body.clientWidth; const limitX = document.body.clientWidth;
const limitY = window.innerHeight; const limitY = window.innerHeight;
let x = elementRect.left; let x = elementRect.left + options.general.popupHorizontalOffset;
let width = Math.max(containerWidth, options.general.popupWidth); let width = Math.max(containerWidth, options.general.popupWidth);
const overflowX = Math.max(x + width - limitX, 0); const overflowX = Math.max(x + width - limitX, 0);
if (overflowX > 0) { if (overflowX > 0) {
@ -64,8 +64,8 @@ class Popup {
let y = 0; let y = 0;
let height = Math.max(containerHeight, options.general.popupHeight); let height = Math.max(containerHeight, options.general.popupHeight);
const yBelow = elementRect.bottom + options.general.popupOffset; const yBelow = elementRect.bottom + options.general.popupVerticalOffset;
const yAbove = elementRect.top - options.general.popupOffset; const yAbove = elementRect.top - options.general.popupVerticalOffset;
const overflowBelow = Math.max(yBelow + height - limitY, 0); const overflowBelow = Math.max(yBelow + height - limitY, 0);
const overflowAbove = Math.max(height - yAbove, 0); const overflowAbove = Math.max(height - yAbove, 0);
if (overflowBelow > 0 || overflowAbove > 0) { if (overflowBelow > 0 || overflowAbove > 0) {