Skip to content

Conversation

mhchen
Copy link
Contributor

@mhchen mhchen commented Nov 27, 2018

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

Disable textarea resize and window scroll method when doing anything that does not actually modify the textarea contents (e.g. arrow keys, pgup/pgdn).

Note: needed to make formatting changes since it looks like linting-related precommit hooks were added after this file was created. Particularly, needed to disable a no-unused-vars lint rule since the function is exposed and consumed globally.

Related Tickets & Documents

Fixes #1109

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

scroll is working

Added to documentation?

  • docs.dev.to
  • readme
  • no documentation needed

Disable textarea resize and window scroll method when doing anything
that does not actually modify the textarea contents (e.g. arrow keys,
pgup/pgdn).

Fixes #1109
@pr-triage pr-triage bot added the PR: unreviewed bot applied label for PR's with no review label Nov 27, 2018
@CLAassistant
Copy link

CLAassistant commented Nov 27, 2018

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@benhalpern benhalpern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@pr-triage pr-triage bot added PR: reviewed-approved bot applied label for PR's where reviewer approves changes and removed PR: unreviewed bot applied label for PR's with no review labels Nov 28, 2018
@benhalpern benhalpern merged commit 7ce3118 into forem:master Nov 28, 2018
@pr-triage pr-triage bot added PR: merged bot applied label for PR's that are merged and removed PR: reviewed-approved bot applied label for PR's where reviewer approves changes labels Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: merged bot applied label for PR's that are merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants