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

fix(textfield): textarea-issue-3432 #3452

Merged
merged 1 commit into from Jul 18, 2023
Merged

fix(textfield): textarea-issue-3432 #3452

merged 1 commit into from Jul 18, 2023

Conversation

blunteshwar
Copy link
Contributor

@blunteshwar blunteshwar commented Jul 14, 2023

Description

fixes #3432

I changed some conditions in textfield.css file so that while styling "grows" and "quite" property does not interfere/

Related issue(s)

Motivation and context

Now the textarea with properties "grows" and "quite" is properly functioning when focused.

How has this been tested?

using dev tools

  • Test case 1
    1. Go here
    2. Do this
  • Test case 2
    1. Go here
    2. Do this

Screenshots (if appropriate)

Screen.Recording.2023-07-14.at.2.03.54.PM.mov

Types of changes

  • [ x] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (minor updates related to the tooling or maintenance of the repository, does not impact compiled assets)

Checklist

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • If my change required a change to the documentation, I have updated the documentation in this pull request.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have reviewed at the Accessibility Practices for this feature, see: Aria Practices

Best practices

This repository uses conventional commit syntax for each commit message; note that the GitHub UI does not use this by default so be cautious when accepting suggested changes. Avoid the "Update branch" button on the pull request and opt instead for rebasing your branch against main.

@blunteshwar blunteshwar marked this pull request as ready for review July 14, 2023 08:36
@github-actions
Copy link

github-actions bot commented Jul 14, 2023

Tachometer results

Chrome

number-field permalink

Version Bytes Avg Time vs remote vs branch
npm latest 625 kB 227.32ms - 231.21ms - unsure 🔍
-0% - +2%
-0.32ms - +4.99ms
branch 627 kB 225.12ms - 228.73ms unsure 🔍
-2% - +0%
-4.99ms - +0.32ms
-

search permalink

Version Bytes Avg Time vs remote vs branch
npm latest 469 kB 117.12ms - 118.60ms - unsure 🔍
-1% - +1%
-0.64ms - +1.60ms
branch 470 kB 116.54ms - 118.23ms unsure 🔍
-1% - +1%
-1.60ms - +0.64ms
-

slider permalink

Version Bytes Avg Time vs remote vs branch
npm latest 452 kB 163.55ms - 166.93ms - unsure 🔍
-2% - +1%
-3.92ms - +1.33ms
branch 453 kB 164.52ms - 168.55ms unsure 🔍
-1% - +2%
-1.33ms - +3.92ms
-

textfield permalink

Version Bytes Avg Time vs remote vs branch
npm latest 416 kB 69.63ms - 71.20ms - unsure 🔍
-0% - +3%
-0.33ms - +1.75ms
branch 417 kB 69.02ms - 70.39ms unsure 🔍
-2% - +0%
-1.75ms - +0.33ms
-
Firefox

number-field permalink

Version Bytes Avg Time vs remote vs branch
npm latest 625 kB 718.41ms - 759.83ms - unsure 🔍
-6% - +1%
-48.49ms - +7.41ms
branch 627 kB 740.90ms - 778.42ms unsure 🔍
-1% - +7%
-7.41ms - +48.49ms
-

search permalink

Version Bytes Avg Time vs remote vs branch
npm latest 469 kB 318.75ms - 349.77ms - faster ✔
3% - 14%
10.53ms - 52.79ms
branch 470 kB 351.57ms - 380.27ms slower ❌
3% - 16%
10.53ms - 52.79ms
-

slider permalink

Version Bytes Avg Time vs remote vs branch
npm latest 452 kB 514.29ms - 569.47ms - unsure 🔍
-7% - +6%
-40.58ms - +35.30ms
branch 453 kB 518.47ms - 570.57ms unsure 🔍
-7% - +8%
-35.30ms - +40.58ms
-

textfield permalink

Version Bytes Avg Time vs remote vs branch
npm latest 416 kB 239.07ms - 269.57ms - unsure 🔍
-8% - +8%
-21.31ms - +20.79ms
branch 417 kB 240.07ms - 269.09ms unsure 🔍
-8% - +8%
-20.79ms - +21.31ms
-

@Rajdeepc Rajdeepc added bug Something isn't working Component: Textfield labels Jul 14, 2023
@Rajdeepc
Copy link
Contributor

@blunteshwar Can you adjust your markdown readme properly. We want to maintain a good PR hygiene always. Thanks

@Westbrook
Copy link
Collaborator

Hopefully when you catch up to main these VRT failures will go away. 🤞🏼

While you're in this file. Would you mind taking a little time to investigate whether the Core Tokens updates unlocked allowing the quiet Textfield to actually grow in this case, unlike the previous implementation?

@Rajdeepc Rajdeepc linked an issue Jul 17, 2023 that may be closed by this pull request
1 task
@blunteshwar
Copy link
Contributor Author

blunteshwar commented Jul 17, 2023

@blunteshwar Can you adjust your markdown readme properly. We want to maintain a good PR hygiene always. Thanks

sure
done!

@Rajdeepc
Copy link
Contributor

@blunteshwar Many VRT's are failing. Can you please check.

@blunteshwar
Copy link
Contributor Author

@blunteshwar Many VRT's are failing. Can you please check.

done!
all tests are passing now

@blunteshwar
Copy link
Contributor Author

Hopefully when you catch up to main these VRT failures will go away. 🤞🏼

While you're in this file. Would you mind taking a little time to investigate whether the Core Tokens updates unlocked allowing the quiet Textfield to actually grow in this case, unlike the previous implementation?

Nope, the textarea with "quite" is not growing, its behaving the same way it was behaving before

Copy link
Collaborator

@Westbrook Westbrook left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks.

@Westbrook Westbrook merged commit a7f563a into main Jul 18, 2023
47 checks passed
@Westbrook Westbrook deleted the bug/textarea-3432 branch July 18, 2023 11:51
TarunAdobe added a commit that referenced this pull request Jul 19, 2023
* fix(meter): added role meter progressbar in meter component (#3459)

Co-authored-by: Rajdeep Chandra <rajdeepc@adobe.com>

* docs: updated swatch mixed documentations (#3455)

Co-authored-by: Rajdeep Chandra <rajdeepc@adobe.com>

* fix(textfield): update focus state when [multiline][quiet] (#3452)

* fix: menu item missing aria labels (#3417)

* testing

* fix(menu): menu-item with submenu lacks aria-haspopup and aria-expanded

* fix(menu): added aria-expanded and aria-haspopup to menu-item

* fix(menu): fixed the aria-controls attribute of menu-item

* fix(menu): reviewed changes

* fix(menu): reviewed changes-2

* fix(menu): removed the comment

* fix(menu): removed white space

* fix(menu): white space removed

* chore: bump @web/test-runner from 0.16.1 to 0.17.0

Bumps [@web/test-runner](https://github.com/modernweb-dev/web/tree/HEAD/packages/test-runner) from 0.16.1 to 0.17.0.
- [Release notes](https://github.com/modernweb-dev/web/releases)
- [Changelog](https://github.com/modernweb-dev/web/blob/master/packages/test-runner/CHANGELOG.md)
- [Commits](https://github.com/modernweb-dev/web/commits/@web/test-runner@0.17.0/packages/test-runner)

---
updated-dependencies:
- dependency-name: "@web/test-runner"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump webpack-bundle-analyzer from 4.8.0 to 4.9.0

Bumps [webpack-bundle-analyzer](https://github.com/webpack-contrib/webpack-bundle-analyzer) from 4.8.0 to 4.9.0.
- [Release notes](https://github.com/webpack-contrib/webpack-bundle-analyzer/releases)
- [Changelog](https://github.com/webpack-contrib/webpack-bundle-analyzer/blob/master/CHANGELOG.md)
- [Commits](webpack-contrib/webpack-bundle-analyzer@v4.8.0...v4.9.0)

---
updated-dependencies:
- dependency-name: webpack-bundle-analyzer
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump netlify-cli from 12.12.0 to 15.9.0

Bumps [netlify-cli](https://github.com/netlify/cli) from 12.12.0 to 15.9.0.
- [Release notes](https://github.com/netlify/cli/releases)
- [Changelog](https://github.com/netlify/cli/blob/main/CHANGELOG.md)
- [Commits](netlify/cli@v12.12.0...v15.9.0)

---
updated-dependencies:
- dependency-name: netlify-cli
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump husky from 8.0.2 to 8.0.3

Bumps [husky](https://github.com/typicode/husky) from 8.0.2 to 8.0.3.
- [Release notes](https://github.com/typicode/husky/releases)
- [Commits](typicode/husky@v8.0.2...v8.0.3)

---
updated-dependencies:
- dependency-name: husky
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump word-wrap from 1.2.3 to 1.2.4

Bumps [word-wrap](https://github.com/jonschlinkert/word-wrap) from 1.2.3 to 1.2.4.
- [Release notes](https://github.com/jonschlinkert/word-wrap/releases)
- [Commits](jonschlinkert/word-wrap@1.2.3...1.2.4)

---
updated-dependencies:
- dependency-name: word-wrap
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump @types/react from 18.0.25 to 18.2.15

Bumps [@types/react](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react) from 18.0.25 to 18.2.15.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react)

---
updated-dependencies:
- dependency-name: "@types/react"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Rajdeep Chandra <rajdeep.chandra@bsil.com>
Co-authored-by: Rajdeep Chandra <rajdeepc@adobe.com>
Co-authored-by: Piyush Vashisht <pvashish@adobe.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Component: Textfield
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Textarea with quiet and grows does not correctly indicate focused
3 participants