8
16
Fork 22

Update textarea overflow to scroll #609

Zusammengeführt
marek-lach hat 3 Commits von scrollable-textarea nach master vor 5 Jahren zusammengeführt
marek-lach hat vor 5 Jahren kommentiert (Migriert von github.com)

Partial fix for #604.

Partial fix for #604.
codecov[bot] hat vor 5 Jahren kommentiert (Migriert von github.com)

Codecov Report

Merging #609 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #609      +/-   ##
=========================================
+ Coverage   34.56%   34.6%   +0.03%     
=========================================
  Files          67      67              
  Lines        7840    7840              
  Branches     1883    1883              
=========================================
+ Hits         2710    2713       +3     
  Misses       4367    4367              
+ Partials      763     760       -3
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/609?src=pr&el=h1) Report > Merging [#609](https://codecov.io/gh/Plume-org/Plume/pull/609?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/bffce041d76b04e0daee38f915909fc35f159ae4?src=pr&el=desc) will **increase** coverage by `0.03%`. > The diff coverage is `n/a`. ```diff @@ Coverage Diff @@ ## master #609 +/- ## ========================================= + Coverage 34.56% 34.6% +0.03% ========================================= Files 67 67 Lines 7840 7840 Branches 1883 1883 ========================================= + Hits 2710 2713 +3 Misses 4367 4367 + Partials 763 760 -3 ```
elegaanz (Migriert von github.com) hat vor 5 Jahren überprüft
elegaanz (Migriert von github.com) hat einen Kommentar hinterlassen

I didn't tested (for some reason it is not deployed on pr-609.joinplu.me 😕), but it seems reasonable.

I didn't tested (for some reason it is not deployed on pr-609.joinplu.me :confused:), but it seems reasonable.
marek-lach hat vor 5 Jahren kommentiert (Migriert von github.com)

I didn't tested (for some reason it is not deployed on pr-609.joinplu.me 😕), but it seems reasonable.

It does... I mean testing it live would probably be a good idea still, but there's really no reason why it should cause any problems. Pretty basic stuff overall...

> I didn't tested (for some reason it is not deployed on pr-609.joinplu.me 😕), but it seems reasonable. It does... I mean testing it live would probably be a good idea still, but there's really no reason why it should cause any problems. Pretty basic stuff overall...
elegaanz (Migriert von github.com) hat vor 5 Jahren überprüft
elegaanz (Migriert von github.com) hat vor 5 Jahren kommentiert

This line should be removed, otherwise there is a weird and useless scrollbar on the right of the whole form:

image

This line should be removed, otherwise there is a weird and useless scrollbar on the right of the whole form: ![image](https://user-images.githubusercontent.com/16254623/58764689-57552d80-8562-11e9-8db3-0e7e41e44198.png)
marek-lach hat vor 5 Jahren kommentiert (Migriert von github.com)

Right. Done.

Right. Done.
elegaanz (Migriert von github.com) hat die Änderungen vor 5 Jahren genehmigt

Reviewer

Der Pull Request wurde als b2312d7f1b gemergt.

Schritt 1:

Wechsle auf einen neuen Branch in deinem lokalen Repository und teste die Änderungen.
git checkout -b scrollable-textarea master
git pull origin scrollable-textarea

Schritt 2:

Führe die Änderungen zusammen und aktualisiere den Stand online auf Forgejo.
git checkout master
git merge --no-ff scrollable-textarea
git push origin master
Anmelden, um an der Diskussion teilzunehmen.
Keine Reviewer
Kein Meilenstein
Niemand zuständig
1 Beteiligte
Nachrichten
Fällig am
Das Fälligkeitsdatum ist ungültig oder außerhalb des zulässigen Bereichs. Bitte verwende das Format „jjjj-mm-tt“.

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Keine Abhängigkeiten gesetzt.

Referenz: Plume/Plume#609
Laden…
Hier gibt es bis jetzt noch keinen Inhalt.