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.

Sign up for GitHub

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

Jump to bottom

[Bug Report] Incorrect text field size calculation inside expansion panel #9299

Closed
Djaler opened this issue Oct 9, 2019 · 2 comments · Fixed by #9341 or drakeg/vue_learn#18
Closed

[Bug Report] Incorrect text field size calculation inside expansion panel #9299

Djaler opened this issue Oct 9, 2019 · 2 comments · Fixed by #9341 or drakeg/vue_learn#18
Assignees
@johnleider
Labels
C: VTextField VTextField

Comments

@Djaler
Copy link
Contributor

Djaler commented Oct 9, 2019

Environment

Vuetify Version: 2.1.2
Vue Version: 2.6.10
Browsers: Chrome 77.0.3865.90
OS: Linux x86_64

Steps to reproduce

  1. Open expansion panel
  2. Focus second field

Expected Behavior

Correct notch width in border
Correct label positioning in second field

Actual Behavior

Incorrect notch width in border
Incorrect label positioning in second field

Reproduction Link

https://codepen.io/djaler/pen/oNNXywM

Other comments

Elements width calculated while they are hidden, so offsetWidth = 0
@ghost ghost added the S: triage label Oct 9, 2019
Djaler added a commit to Djaler/vuetify that referenced this issue Oct 9, 2019
@Djaler
fix(VTextField): use intersect to recalculate elements width
c02a115 
fixes vuetifyjs#9299
@Djaler Djaler mentioned this issue Oct 9, 2019
Closed
9 tasks
Djaler added a commit to Djaler/vuetify that referenced this issue Oct 9, 2019
@Djaler
fix(VTextField): use intersect to recalculate elements width
3eea48f 
fixes vuetifyjs#9299
Djaler added a commit to Djaler/vuetify that referenced this issue Oct 10, 2019
@Djaler
fix(VTextField): use intersect to recalculate elements width
164f871 
fixes vuetifyjs#9299
Djaler added a commit to Djaler/vuetify that referenced this issue Oct 10, 2019
@Djaler
fix(VTextField): use intersect to recalculate elements width
9fe686e 
fixes vuetifyjs#9299
@haggys22
Copy link
Contributor

Related: #9165, #9156, #9225

I've been working on a PR as well: #9313

@Djaler
Copy link
Contributor Author

Djaler commented Oct 11, 2019

@haggys22 I work on this too #9300

Djaler added a commit to Djaler/vuetify that referenced this issue Oct 13, 2019
@Djaler
fix(VTextField): use intersect to recalculate elements width
925c050 
fixes vuetifyjs#9299
@Djaler Djaler mentioned this issue Oct 13, 2019
Merged
9 tasks
@johnleider johnleider added the C: VTextField VTextField label Oct 24, 2019
@johnleider johnleider self-assigned this Oct 24, 2019
Djaler added a commit to Djaler/vuetify that referenced this issue Nov 13, 2019
@Djaler
fix(VTextField): use intersect to recalculate elements width
a119753 
fixes vuetifyjs#9299
Djaler added a commit to Djaler/vuetify that referenced this issue Nov 13, 2019
@Djaler
fix(VTextField): use intersect to recalculate elements width
e71b995 
fixes vuetifyjs#9299
Djaler added a commit to Djaler/vuetify that referenced this issue Nov 21, 2019
@Djaler
fix(VTextField): use intersect to recalculate elements width
33b2f67 
fixes vuetifyjs#9299
Djaler added a commit to Djaler/vuetify that referenced this issue Nov 21, 2019
@Djaler
fix(VTextField): use intersect to recalculate elements width
2f5b9fb 
fixes vuetifyjs#9299
Djaler added a commit to Djaler/vuetify that referenced this issue Nov 29, 2019
@Djaler
fix(VTextField): use intersect to recalculate elements width
3341a83 
fixes vuetifyjs#9299
Djaler added a commit to Djaler/vuetify that referenced this issue Nov 29, 2019
@Djaler
fix(VTextField): use intersect to recalculate elements width
005e4d8 
fixes vuetifyjs#9299
johnleider pushed a commit that referenced this issue Dec 4, 2019
@Djaler @johnleider
fix(VTextField): use intersect to recalculate elements width (#9341)
c34f35c 
* feat: add new mixin to work with IntersectionObserver

* fix(VTextField): use intersect to recalculate elements width

fixes #9299
@snyk-bot snyk-bot mentioned this issue Dec 19, 2019
Closed
@snyk-bot snyk-bot mentioned this issue Jan 7, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johnleider johnleider

Labels
C: VTextField VTextField
Projects
None yet
Milestone
No milestone
3 participants
@haggys22 @Djaler @johnleider