Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make comment box shrink on submit #13686

Merged
merged 3 commits into from May 10, 2021

Conversation

oyeyipo
Copy link
Contributor

@oyeyipo oyeyipo commented May 6, 2021

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

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

The main comment box on the articles detail page does not reset to default height after successful submission. The inline style height set on the textarea element is not being removed.

Related Tickets & Documents

Fixes #13298

Added tests?

  • Yes
  • No, and this is why: this is a UI bug fix
  • I need help with writing tests

[optional] What gif best describes this PR or how it makes you feel?

Thinking Bob

@oyeyipo oyeyipo requested a review from a team May 6, 2021 20:06
@pr-triage pr-triage bot added the PR: unreviewed bot applied label for PR's with no review label May 6, 2021
@oyeyipo oyeyipo requested review from ludwiczakpawel and nickytonline and removed request for a team May 6, 2021 20:07
@github-actions
Copy link
Contributor

github-actions bot commented May 6, 2021

Thank you for opening this PR! We appreciate you!

For all pull requests coming from third-party forks we will need to
review the PR before we can process it through our CI pipelines.

A Forem Team member will review this contribution and get back to
you as soon as possible!

Copy link
Contributor

@rhymes rhymes left a comment

Choose a reason for hiding this comment

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

Great catch @oyeyipowale!

I tested it and it works! I added a small request :)

Copy link

@Farhaduneci Farhaduneci left a comment

Choose a reason for hiding this comment

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

👍

@rhymes rhymes requested a review from aitchiss May 7, 2021 12:57
Copy link
Contributor

@aitchiss aitchiss left a comment

Choose a reason for hiding this comment

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

I tested this locally and it works great - thanks for fixing it! I just have a small change request to make things hopefully clearer for anyone coming to the code in future 🙂

@pr-triage pr-triage bot added PR: reviewed-changes-requested bot applied label for PR's where reviewer requests changes PR: unreviewed bot applied label for PR's with no review and removed PR: unreviewed bot applied label for PR's with no review PR: reviewed-changes-requested bot applied label for PR's where reviewer requests changes labels May 7, 2021
@oyeyipo oyeyipo force-pushed the oyeyipowale/shrink-commet-box branch from 5b27118 to 40ba833 Compare May 7, 2021 20:10
@pr-triage pr-triage bot added PR: partially-approved bot applied label for PR's where a single reviewer approves changes and removed PR: unreviewed bot applied label for PR's with no review labels May 8, 2021
@rhymes rhymes requested a review from aitchiss May 8, 2021 07:29
@aitchiss aitchiss merged commit 5107844 into forem:main May 10, 2021
@pr-triage pr-triage bot added PR: merged bot applied label for PR's that are merged and removed PR: partially-approved bot applied label for PR's where a single reviewer approves changes labels May 10, 2021
@aitchiss
Copy link
Contributor

Thanks for the fix @oyeyipowale! ✨

@aitchiss aitchiss added the changelog: rollup Items that will be communicated in our monthly rollup changelogs label May 10, 2021
@cmgorton
Copy link
Contributor

Great job @oyeyipowale ! Thanks for helping out and smashing this bug!

@cmgorton
Copy link
Contributor

cmgorton commented Jun 1, 2021

@oyeyipowale would you mind making a quick post on DEV using this template. That way we can make sure you get the Bug Smash badge and stickers 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: rollup Items that will be communicated in our monthly rollup changelogs PR: merged bot applied label for PR's that are merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Comment box not shrinking on submit
5 participants