service2.js: Revise

This commit is contained in:
Simon Rozman 2024-05-23 12:35:12 +02:00
parent 947d5048e6
commit 94f55e0d7f
2 changed files with 7 additions and 4 deletions

View File

@ -17,5 +17,6 @@
} }
.ck-editor__editable { .ck-editor__editable {
height: 400px; min-height: 100px;
max-height: 500px;
} }

View File

@ -309,7 +309,7 @@ class BesService {
const scrollLeft = window.scrollX || document.documentElement.scrollLeft const scrollLeft = window.scrollX || document.documentElement.scrollLeft
const scrollTop = window.scrollY || document.documentElement.scrollTop const scrollTop = window.scrollY || document.documentElement.scrollTop
this.statusDiv.style.left = `${rect.right - 40 + scrollLeft}px` this.statusDiv.style.left = `${rect.right - 40 + scrollLeft}px`
this.statusDiv.style.top = `${rect.top + rect.height - 30 + scrollTop}px` this.statusDiv.style.top = `${rect.bottom - 30 + scrollTop}px`
} }
/** /**
@ -911,12 +911,14 @@ class BesCKService extends BesTreeService {
* Repositions status DIV element. * Repositions status DIV element.
*/ */
setStatusDivPosition() { setStatusDivPosition() {
// TODO: New position needs to be tested on better displays (4K, etc.) // TODO: The position returned by this.hostElement.getBoundingClientRect() appears to be absolute,
// while this.statusDiv.style.left and .top require relative position. Either compensate by subtracting
// CKEditor frame position, or change statusDiv placement to absolute.
const rect = this.hostElement.getBoundingClientRect() const rect = this.hostElement.getBoundingClientRect()
const scrollLeft = window.scrollX || document.documentElement.scrollLeft const scrollLeft = window.scrollX || document.documentElement.scrollLeft
const scrollTop = window.scrollY || document.documentElement.scrollTop const scrollTop = window.scrollY || document.documentElement.scrollTop
this.statusDiv.style.left = `${rect.right - 50 + scrollLeft}px` this.statusDiv.style.left = `${rect.right - 50 + scrollLeft}px`
this.statusDiv.style.top = `${rect.top + rect.height - 120 + scrollTop}px` this.statusDiv.style.top = `${rect.bottom - 120 + scrollTop}px`
} }
} }