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

Unable to unfocus input when clicking on an icon inside #2005

Closed
jean-moldovan opened this issue May 3, 2018 · 0 comments · Fixed by #2215
Closed

Unable to unfocus input when clicking on an icon inside #2005

jean-moldovan opened this issue May 3, 2018 · 0 comments · Fixed by #2215
Milestone
v0.17.0

Comments

@jean-moldovan
Copy link

jean-moldovan commented May 3, 2018 •
edited

Hello,

When you have an icon inside an input and click on it, it stays focused all the time:
https://jsfiddle.net/waugrryy/1979/

  1. Click inside the input, then outside. Works as expected
  2. Click on an icon inside the input, then outside. Input remains focused

Thanks!
kpwbo added a commit to kpwbo/quasar that referenced this issue May 5, 2018
@kpwbo
fix: input unfocused when before/after icon is clicked (fix quasarfra…
2b8e370 
…mework#2005)
pdanpdan added a commit to pdanpdan/quasar that referenced this issue Jun 28, 2018
@pdanpdan
- Don't show clear button when current value is the same with clearValue
26ad0a0 
  - For QColor and QDateTime if there is no clearValue we use defaultValue
- QSelect: use className from option item in list
- QSearch: combine before/after with search and clear icons (can be disabled)
- QInput and friends: focus input field in __onFocus

close quasarframework#2017, close quasarframework#2005, close quasarframework#1953
pdanpdan added a commit to pdanpdan/quasar that referenced this issue Jun 28, 2018
@pdanpdan
ClearValue, className for QSelect, QSearch before|after, QInput focus…
4063663 
… on marginalia

- Don't show clear button when current value is the same with clearValue
  - For QColor and QDateTime if there is no clearValue we use defaultValue
- QSelect: use className from option item in list
- QSearch: combine before/after with search and clear icons (can be disabled)
- QInput and friends: focus input field in __onFocus

close quasarframework#2017, close quasarframework#2005, close quasarframework#1953
pdanpdan added a commit to pdanpdan/quasar that referenced this issue Jun 29, 2018
@pdanpdan
className for QSelect, QSearch before|after, QInput focus on marginalia
0424d9a 
- QSelect: use className from option item in list
- QSearch: combine before/after with search and clear icons (can be disabled)
- QInput and friends: focus input field in __onFocus

close quasarframework#2017, close quasarframework#2005, close quasarframework#1953
pdanpdan added a commit to pdanpdan/quasar that referenced this issue Jun 29, 2018
@pdanpdan
className for QSelect, QSearch before|after, QInput focus on marginalia
6948dcc 
- QSelect: use className from option item in list
- QSearch: combine before/after with search and clear icons (can be disabled)
- QInput and friends: focus input field in __onFocus

close quasarframework#2017, close quasarframework#2005, close quasarframework#1953
@pdanpdan pdanpdan mentioned this issue Jun 29, 2018
Merged
rstoenescu pushed a commit that referenced this issue Jun 29, 2018
@pdanpdan @rstoenescu
className for QSelect, QSearch before|after, QInput focus on marginal…
33ea19c 
…ia (#2215)

* className for QSelect, QSearch before|after, QInput focus on marginalia

- QSelect: use className from option item in list
- QSearch: combine before/after with search and clear icons (can be disabled)
- QInput and friends: focus input field in __onFocus

close #2017, close #2005, close #1953

* Some demos for navigation in dropdown

* Update QInput.vue
@rstoenescu rstoenescu added this to the v0.16.1 milestone Jun 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.17.0
Development

Successfully merging a pull request may close this issue.

className for QSelect, QSearch before|after, QInput focus on marginalia pdanpdan/quasar
2 participants
@jean-moldovan @rstoenescu