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

Changes_GOP-877,GOP-876,GOP-847-BUG-minimum_height_for_textarea #880

Merged
merged 3 commits into from
Apr 5, 2023

Conversation

adhiyan-tangoe
Copy link
Contributor

@adhiyan-tangoe adhiyan-tangoe commented Sep 26, 2022

Fixed GOP-876 uniform card margins, GOP-877 set minimum_height_for_textarea and GOP-847 file uploading component bug fix.

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our guidelines
  • Tests for the changes have been added
  • Docs have been added or updated

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Style Update (CSS)
  • Other... Please describe:

What is the current behavior?

Resolves

#877
#876
#847

What is the new behavior?

#877 - Textarea will have the minimum width as 70px.
#876 - Last child of the card content will have no margin bottom.
#847 - Reselecting the same file after clearing is working fine.

Does this PR introduce a breaking change?

  • Yes
  • No

Fixed GOP-876 uniform card margins, GOP-877 set minimum_height_for_textarea and GOP-847 file uploading component bug fix.
@adhiyan-tangoe adhiyan-tangoe added bug Something isn't working UX Review Waiting on review from UX labels Sep 26, 2022
@adhiyan-tangoe adhiyan-tangoe changed the title GOP-877-BUG-minimum_height_for_textarea Changes_GOP-877,GOP-876,GOP-847-BUG-minimum_height_for_textarea Sep 26, 2022
Copy link
Contributor

@jiaweicai92 jiaweicai92 left a comment

Choose a reason for hiding this comment

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

The fix looks good. Left some comments on changes on _forms.scss. Great job 👍

projects/go-lib/styles/_forms.scss Outdated Show resolved Hide resolved
projects/go-lib/styles/_forms.scss Outdated Show resolved Hide resolved
projects/go-lib/styles/_forms.scss Outdated Show resolved Hide resolved
projects/go-lib/styles/_forms.scss Outdated Show resolved Hide resolved
projects/go-lib/styles/_forms.scss Outdated Show resolved Hide resolved
projects/go-lib/styles/_forms.scss Outdated Show resolved Hide resolved
@adhiyan-tangoe adhiyan-tangoe self-assigned this Oct 11, 2022
swimtech20
swimtech20 previously approved these changes Oct 11, 2022
Copy link
Collaborator

@swimtech20 swimtech20 left a comment

Choose a reason for hiding this comment

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

Nice work!

jiaweicai92
jiaweicai92 previously approved these changes Oct 11, 2022
Copy link
Contributor

@jiaweicai92 jiaweicai92 left a comment

Choose a reason for hiding this comment

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

👍 Looks great!

@stolve
Copy link

stolve commented Jan 27, 2023

@kpmurphh and myself have no way to provide approval/feedback for this issue. We do not have an instance of goponents on our systems in order to look at the visual fix and we are not in a position to give feedback on coding approaches. Please attach a screenshot of the update or block time on one of our calendars for a review session.

@Saravanan-Tangoe Saravanan-Tangoe changed the base branch from dev to main February 22, 2023 13:58
@Saravanan-Tangoe Saravanan-Tangoe dismissed stale reviews from jiaweicai92 and swimtech20 February 22, 2023 13:58

The base branch was changed.

@Saravanan-Tangoe Saravanan-Tangoe changed the base branch from main to dev February 22, 2023 13:58
stolve
stolve previously approved these changes Mar 31, 2023
Copy link

@stolve stolve left a comment

Choose a reason for hiding this comment

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

Based on the netlify link the change looks good to me.

swimtech20
swimtech20 previously approved these changes Mar 31, 2023
@swimtech20 swimtech20 dismissed stale reviews from stolve and themself via db1b2c5 April 5, 2023 20:30
@swimtech20 swimtech20 self-requested a review April 5, 2023 20:35
@swimtech20 swimtech20 merged commit 50bb3a2 into dev Apr 5, 2023
@swimtech20 swimtech20 deleted the GOP-877-BUG-minimum_height_for_textarea branch April 5, 2023 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working UX Review Waiting on review from UX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants