alex/yomichan
1
Fork 0
Code Releases Activity

Change how current entry is indicated

Browse Source
This commit is contained in:
toasted-nutbread 2019-09-15 18:44:49 -04:00
parent 3ca84e3a85
commit 8db830b468
2 changed files with 20 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ext/mixed/css/display.css
View File

@ -230,3 +230,7 @@ div.glossary-item.compact-glossary {
.info-output td { .info-output td {
text-align: right; text-align: right;
} }
.entry:not(.entry-current) .current {
display: none;
}

19
ext/mixed/js/display.js
View File

@ -412,16 +412,23 @@ class Display {
index = Math.min(index, this.definitions.length - 1); index = Math.min(index, this.definitions.length - 1);
index = Math.max(index, 0); index = Math.max(index, 0);
$('.current').hide().eq(index).show(); const entryPre = this.getEntry(this.index);
if (entryPre !== null) {
entryPre.classList.remove('entry-current');
}
const entry = this.getEntry(index);
if (entry !== null) {
entry.classList.add('entry-current');
}
this.windowScroll.stop(); this.windowScroll.stop();
const entry = $('.entry').eq(index);
let target; let target;
if (scroll) { if (scroll) {
target = scroll; target = scroll;
} else { } else {
target = index === 0 ? 0 : entry.offset().top; target = index === 0 || entry === null ? 0 : Display.getElementTop(entry);
} }
if (smooth) { if (smooth) {
@ -606,4 +613,10 @@ class Display {
addEventListeners(selector, ...args) { addEventListeners(selector, ...args) {
this.container.querySelectorAll(selector).forEach((node) => node.addEventListener(...args)); this.container.querySelectorAll(selector).forEach((node) => node.addEventListener(...args));
} }
static getElementTop(element) {
const elementRect = element.getBoundingClientRect();
const documentRect = document.documentElement.getBoundingClientRect();
return elementRect.top - documentRect.top;
}
} }