Discussions and requests related to new CMSimple features, plugins, templates etc. and how to develop.
Please don't ask for support at this forums!
-
lck
- Posts: 3211
- Joined: Wed Mar 23, 2011 11:43 am
-
Contact:
Post
by lck » Wed Aug 11, 2021 10:20 am
manu wrote: ↑Wed Aug 11, 2021 9:48 am
Ging früher nicht anders, aber jetzt gibts im filebrowser.js eine Option "Large" (Fenster etwa so gross wie im Tiny4). Werde es nächstens implementieren.
Das habe ich gerade mal getestet. Das löst das Problem auch nicht. Das Fenster des File Managers wird zwar größer, aber das iframe nicht. Immer noch mit Scrollbalken.
Code: Select all
// FIXME: avoid the following two global variables!
filebrowsercallback = callback;
filebrowserwindow = tinymce.activeEditor.windowManager.open({
title: "File Manager",
size: 'large',
-
manu
- Posts: 1117
- Joined: Wed Jun 04, 2008 12:05 pm
- Location: St. Gallen - Schweiz
-
Contact:
Post
by manu » Wed Aug 11, 2021 11:42 am
lck wrote: ↑Wed Aug 11, 2021 10:20 am
manu wrote: ↑Wed Aug 11, 2021 9:48 am
Ging früher nicht anders, aber jetzt gibts im filebrowser.js eine Option "Large" (Fenster etwa so gross wie im Tiny4). Werde es nächstens implementieren.
Das habe ich gerade mal getestet. Das löst das Problem auch nicht. Das Fenster des File Managers wird zwar größer, aber das iframe nicht. Immer noch mit Scrollbalken.
Code: Select all
// FIXME: avoid the following two global variables!
filebrowsercallback = callback;
filebrowserwindow = tinymce.activeEditor.windowManager.open({
title: "File Manager",
size: 'large',
Liegt wohl an der filebrowser.js resizeIframe() Zeile 32. Ich habe dort mal probeweise offsetHeight + 140px versucht, dann passt die iFrame Höhe (mit den Standard Thumbnail Höhen). Aber wie schon im Code Kommentar. Diesen Code resp. Berechnung durchschaue ich nicht ganz. Der Wert 140 ist beim Pröbeln rausgekommen. Ideen von den CSS Spezies?
-
lck
- Posts: 3211
- Joined: Wed Mar 23, 2011 11:43 am
-
Contact:
Post
by lck » Wed Aug 11, 2021 11:47 am
manu wrote: ↑Wed Aug 11, 2021 11:42 am
Liegt wohl an der filebrowser.js resizeIframe() Zeile 32. Ich habe dort mal probeweise offsetHeight + 140px versucht, dann passt die iFrame Höhe (mit den Standard Thumbnail Höhen). Aber wie schon im Code Kommentar. Diesen Code resp. Berechnung durchschaue ich nicht ganz. Der Wert 140 ist beim Pröbeln rausgekommen. Ideen von den CSS Spezies?
Folgendes habe ich angepasst.
plugins/tinymce5/filebrowser.js
Die function resizeIframe(obj) wäre dann eigentlich hinfällig.
plugins/tinymce5/tinymce/skins/ui/oxide/skinmin.css
am Ende derweil mal angefügt:
Code: Select all
.tox .tox-form__group{height:100%;}
.tox .tox-form__group iframe{min-height:100% !important;}
plugins/filebrowser/css/stylesheet.css
derweil am Ende
Code: Select all
#filebrowser_files.xhfb_Bg li:last-of-type {
padding-bottom: 10em !important;
}
Sieht schon mal gut aus. Aber ist halt Flickerei.
-
frase
- Posts: 5085
- Joined: Thu Apr 21, 2016 6:32 am
- Location: Saxony
-
Contact:
Post
by frase » Wed Aug 11, 2021 12:00 pm
manu wrote: ↑Wed Aug 11, 2021 11:42 am
Liegt wohl an der filebrowser.js resizeIframe()
Genau.
Das JS ist irgendwie Murks.
Versucht das mal bitte (filebrowser.js komplett, keine CSS-Änderungen):
Code: Select all
function wrFilebrowser (callback, value, meta) {
var cmsURL = "%URL%";
var type = meta.filetype;
if (type == "file") {
type = "downloads"
};
if (cmsURL.indexOf("?") < 0) {
cmsURL = cmsURL + "?type="+ type;
} else {
cmsURL = cmsURL + "&type=" + type;
}
// FIXME: avoid the following two global variables!
filebrowsercallback = callback;
filebrowserwindow = tinymce.activeEditor.windowManager.open({
title: "File Manager",
size: 'medium',
body: {
type: "panel",
items: [{
type: "htmlpanel",
html: '<iframe src="' + cmsURL + '" style="width:100%" onload="resizeIframe(this)"></iframe>'
}]
},
buttons: []
});
return false;
}
function resizeIframe(obj) {
obj.style.height = self.innerHeight + 'px';
}
Könnte aber auch Murks sein
Aber bei mir klappt das - auch responsive.
-
manu
- Posts: 1117
- Joined: Wed Jun 04, 2008 12:05 pm
- Location: St. Gallen - Schweiz
-
Contact:
Post
by manu » Wed Aug 11, 2021 12:14 pm
lck wrote: ↑Wed Aug 11, 2021 11:47 am
manu wrote: ↑Wed Aug 11, 2021 11:42 am
Liegt wohl an der filebrowser.js resizeIframe() Zeile 32. Ich habe dort mal probeweise offsetHeight + 140px versucht, dann passt die iFrame Höhe (mit den Standard Thumbnail Höhen). Aber wie schon im Code Kommentar. Diesen Code resp. Berechnung durchschaue ich nicht ganz. Der Wert 140 ist beim Pröbeln rausgekommen. Ideen von den CSS Spezies?
Folgendes habe ich angepasst.
plugins/tinymce5/filebrowser.js
Die function resizeIframe(obj) wäre dann eigentlich hinfällig.
plugins/tinymce5/tinymce/skins/ui/oxide/skinmin.css
am Ende derweil mal angefügt:
Code: Select all
.tox .tox-form__group{height:100%;}
.tox .tox-form__group iframe{min-height:100% !important;}
plugins/filebrowser/css/stylesheet.css
derweil am Ende
Code: Select all
#filebrowser_files.xhfb_Bg li:last-of-type {
padding-bottom: 10em !important;
}
Sieht schon mal gut aus. Aber ist halt Flickerei.
Das ist schon doch mal was. Mit dem filebrowser CSS passt es schon fast mit dem bestehenden resizeIframe(). Am skin ändere ich nur ungern etwas, weil er beim nächsten Tiny update potentiell überschrieben wird. Dann müssten wir einen eigenständigen skin anlegen. Auch nicht so prickelnd, den müssten wir selber unterhalten.
Dass padding 10em gerade 150px ergibt ist wohl Glück. Bei hochkantigen Bildern stimmt dieser Wert dann nicht mehr (TN width fix 150px, height dann im Verhältnis). Aber damit lässt sich wohl leben.
Lange Rede meine Meinung: Deine filebrowser CSS Angaben anpassen und gut ist.
-
frase
- Posts: 5085
- Joined: Thu Apr 21, 2016 6:32 am
- Location: Saxony
-
Contact:
Post
by frase » Wed Aug 11, 2021 12:28 pm
Übrigens:
Tiny5 - dropdown
Da gibt es keine Möglichkeit einen Link zu entfernen.
Oder sehe ich das nicht?
-
manu
- Posts: 1117
- Joined: Wed Jun 04, 2008 12:05 pm
- Location: St. Gallen - Schweiz
-
Contact:
Post
by manu » Wed Aug 11, 2021 12:33 pm
frase wrote: ↑Wed Aug 11, 2021 12:28 pm
Übrigens:
Tiny5 - dropdown
Da gibt es keine Möglichkeit
einen Link zu entfernen.
Oder sehe ich das nicht?
Hast Recht, muss ich nachbessern.
-
frase
- Posts: 5085
- Joined: Thu Apr 21, 2016 6:32 am
- Location: Saxony
-
Contact:
Post
by frase » Wed Aug 11, 2021 12:37 pm
manu wrote: ↑Wed Aug 11, 2021 12:14 pm
Lange Rede meine Meinung: Deine filebrowser CSS Angaben anpassen und gut ist.
Ich bin der Meinung, dass die Höhe im JS richtig berechnet werden sollte.
Vielleicht finden wir ja jemanden, der JS richtig kann
Keine CSS-Änderungen in den Skins!
lck wrote: ↑Wed Aug 11, 2021 11:47 am
plugins/filebrowser/css/stylesheet.css
derweil am Ende
Code: Select all
#filebrowser_files.xhfb_Bg li:last-of-type {
padding-bottom: 10em !important;
}
Sieht schon mal gut aus. Aber ist halt Flickerei.
Die 10em sollten eigentlich genau passen, weil:
Code: Select all
a.xhfbfile img,
span.xhfbfile img {
...
max-height: 10em !important;
max-width: 10em !important;
}
-
manu
- Posts: 1117
- Joined: Wed Jun 04, 2008 12:05 pm
- Location: St. Gallen - Schweiz
-
Contact:
Post
by manu » Wed Aug 11, 2021 12:44 pm
frase wrote: ↑Wed Aug 11, 2021 12:00 pm
manu wrote: ↑Wed Aug 11, 2021 11:42 am
Liegt wohl an der filebrowser.js resizeIframe()
Genau.
Das JS ist irgendwie Murks.
Versucht das mal bitte (filebrowser.js komplett, keine CSS-Änderungen):
Code: Select all
function wrFilebrowser (callback, value, meta) {
var cmsURL = "%URL%";
var type = meta.filetype;
if (type == "file") {
type = "downloads"
};
if (cmsURL.indexOf("?") < 0) {
cmsURL = cmsURL + "?type="+ type;
} else {
cmsURL = cmsURL + "&type=" + type;
}
// FIXME: avoid the following two global variables!
filebrowsercallback = callback;
filebrowserwindow = tinymce.activeEditor.windowManager.open({
title: "File Manager",
size: 'medium',
body: {
type: "panel",
items: [{
type: "htmlpanel",
html: '<iframe src="' + cmsURL + '" style="width:100%" onload="resizeIframe(this)"></iframe>'
}]
},
buttons: []
});
return false;
}
function resizeIframe(obj) {
obj.style.height = self.innerHeight + 'px';
}
Könnte aber auch Murks sein
Aber bei mir klappt das - auch responsive.
Size 'medium' funktioniert nach dem filebrowser padding auch wieder. Aber Dein self.innerHeight wird einiges zu gross.
-
manu
- Posts: 1117
- Joined: Wed Jun 04, 2008 12:05 pm
- Location: St. Gallen - Schweiz
-
Contact:
Post
by manu » Wed Aug 11, 2021 12:48 pm
cmb wrote: ↑Tue Aug 10, 2021 9:09 pm
lck wrote: ↑Tue Aug 10, 2021 11:50 am
Du hast recht. Also folgende Lösung. Wir packen eine leere en.js in den Ordner langs und schon ist in "en" auch die Tollbar auf English
.
Mir scheint, wir machen da irgendwas falsch, aber wenn es so einfach geht, prima!
Geht mir eigentlich gegen den Strich, aber stimmt, es ist die einfachste Lösung. Und ein Kommentar rein "Just leave it here, because of sub languages".