#609 Update textarea overflow to scroll

Merged
marek-lach merged 3 commits from scrollable-textarea into master 1 year ago
marek-lach commented 1 year ago (Migrated from github.com)

Partial fix for #604.

Partial fix for #604.
codecov[bot] commented 1 year ago (Migrated from github.com)
Owner

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 (Migrated from github.com) reviewed 1 year ago
elegaanz (Migrated from github.com) left a comment

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

marek-lach commented 1 year ago (Migrated from github.com)
Owner

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 (Migrated from github.com) reviewed 1 year ago
Plume_migration_agent commented 1 year ago

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 commented 1 year ago (Migrated from github.com)
Owner

Right. Done.

Right. Done.
elegaanz (Migrated from github.com) approved these changes 1 year ago

Reviewers

Plume_migration_agent approved these changes 1 year ago
The pull request has been merged as b2312d7f1b.
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.