Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Scrollbalken verdeckt die letzte Zeile von HTML/Markdown Elementen #8328

Closed
J-J-A opened this issue Apr 29, 2016 · 14 comments
Closed

Scrollbalken verdeckt die letzte Zeile von HTML/Markdown Elementen #8328

J-J-A opened this issue Apr 29, 2016 · 14 comments
Assignees
Labels
Milestone

Comments

@J-J-A
Copy link

J-J-A commented Apr 29, 2016

Man kann die letzte Zeile nur noch umständlich anklicken.

Scrollt man im Chrome auf der Höhe des Scrollbalken, blendet dieser sich aus irgendwelchen Gründen auch nicht mehr automatisch aus. In Safari ist das ähnlich mühsam, im Firefox geht es besser. Unter Mac OS mit Contao 3.5.0 getestet.

bildschirmfoto 2016-04-29 um 11 32 21

Ich habe diesen Feldern im BE einfach ein padding nach unten gegeben. Vielleicht könnte man das schon im Core integrieren, falls es auch anderen so geht.

@leofeyer
Copy link
Member

Wie lässt sich das in der Onlinedemo reproduzieren?

@asaage
Copy link

asaage commented Apr 29, 2016

Das scheint Browserspezifisch zu sein.
win-chrome vergrößert das Feld wenn es Focus bekommt aber verdeckt zunächst auch die letzte Zeile.

@Aybee
Copy link
Contributor

Aybee commented Apr 29, 2016

Zum Testen in einem CE Typ HTML dieses hier eingeben:

<p>
  Ich bin eine lange Textzeile. Ich bin eine lange Textzeile. Ich bin eine lange Textzeile. Ich bin eine lange Textzeile. Ich bin eine lange Textzeile.
</p>

Es erscheint eine vertikale Scrollbar und der Quelltext (die 3 Zeilen) sind nicht auf einen Blick zu sehen. Da müsste hauptsächlich unten etwas Luft geschaffen werden. FF, Chrome, Edge, IE11 Opera.
Hier direkt bei ACE sieht das besser aus https://ace.c9.io/#nav=about. Gleichen Code in den Editor einfügen.

@J-J-A
Copy link
Author

J-J-A commented Apr 30, 2016

Im Prinzip bei allen Typen HTML/Code/Markdown.

@leofeyer
Copy link
Member

Vielleicht wäre es das beste, wenn wir angesichts der Cross-Browser-Problematik mit Soft-Wraps arbeiten würden, so dass gar nicht erst ein vertikaler Scrollbalken entsteht.

  var editor = ace.edit('<?= $this->selector ?>_div');
  editor.$blockScrolling = Infinity;
  editor.setTheme("ace/theme/clouds");
  editor.renderer.setScrollMargin(3, 3, 0, 0);
  editor.renderer.scrollBy(0, -6);
  editor.getSession().setValue(ta.value);
  editor.getSession().setMode("ace/mode/<?= Backend::getAceType($this->type) ?>");
  editor.getSession().setUseSoftTabs(false);
  editor.getSession().setUseWrapMode(true);  // <----------- diese Zeile einfügen
  editor.setAutoScrollEditorIntoView(true);

Könnt ihr bitte mal testen, ob das das Problem behebt?

leofeyer added a commit to contao/core-bundle that referenced this issue May 17, 2016
@Aybee
Copy link
Contributor

Aybee commented May 17, 2016

Für mich wäre das so ok - Windows 10, FF, Chrome, Edge, IE11. Vielleicht ein wenig viel Luft unten, aber nicht unangenehm mit diesem Platz.

@leofeyer
Copy link
Member

Was meinst Du mit "ein wenig viel Luft unten"? So sieht es bei mir aus:

@Aybee
Copy link
Contributor

Aybee commented May 17, 2016

artikel_ test inhaltselement_id_537_bearbeiten-contao_open_source_cms_3 5-_2016-05-17_17 34 05
Abgesicherter Modus, keine dcaconfig oder initconfig, Theme default, C 3.5.12, FF 46.01

@leofeyer
Copy link
Member

Was passiert, wenn Du Zeile 62 löschst?

@Aybee
Copy link
Contributor

Aybee commented May 17, 2016

artikel_ inhaltselement_id_537_bearbeiten-contao_open_source_cms_3 5-_2016-05-17_17 43 53

@Aybee
Copy link
Contributor

Aybee commented May 17, 2016

Ups, sorry, das war nicht mit 62 gelöscht, sondern wieder der Ursprungszustand ohne die neue Zeile von oben.

@Aybee
Copy link
Contributor

Aybee commented May 17, 2016

Ok, setUseWrapMode wieder entfernt und + editor.renderer.scrollBar.getWidth() entfernt, dann sieht es so aus, wie bei dir.

@Aybee
Copy link
Contributor

Aybee commented May 17, 2016

Sorry, jetzt dreh ich hier langsam am Rad: Neue Zeile setUseWrapMode und + editor.renderer.scrollBar.getWidth() entfernt - damit war es so wie bei dir.

leofeyer added a commit to contao/core-bundle that referenced this issue May 17, 2016
@leofeyer leofeyer added this to the 3.5.13 milestone May 28, 2016
@leofeyer leofeyer self-assigned this Jun 9, 2016
@leofeyer
Copy link
Member

leofeyer commented Jun 9, 2016

Behoben in 8a2afd9.

@leofeyer leofeyer closed this as completed Jun 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants