Skip to content
This repository has been archived by the owner on Dec 13, 2020. It is now read-only.

Combined Fields/ Lookup adjustment #839

Closed
metas-mk opened this issue Jun 5, 2017 · 2 comments
Closed

Combined Fields/ Lookup adjustment #839

metas-mk opened this issue Jun 5, 2017 · 2 comments

Comments

@metas-mk
Copy link
Member

metas-mk commented Jun 5, 2017

Type of issue

Feature request

Current behavior

In Issue #600 we made the assumption, that default fields shall be directly selected when available and the user shall use the mouse to change if needed.

Expected behavior

After receiving the feedback and to make the dropdown behavior more predictable for the user the behavior shall be adjusted to the following.

Always show the dropdown, also when there is a default property. Although default property is set, the frontend shall open the dropdown and select the default entry. The user just as to press [enter] to confirm. In case she/ he needs another selection different from default in those case just has to do the selection with arrow keys.
It's just important that the default shall be first in Dropdown and be selected initially when opening.

@metas-mk metas-mk added this to the 2017-24 milestone Jun 5, 2017
@cadavre
Copy link
Contributor

cadavre commented Jun 5, 2017

Just popped on my mind: what if there is just one suggestion available. I would say to also display it even if it's only possibility to select. To be unified.

@Dunkat Dunkat self-assigned this Jun 6, 2017
Dunkat added a commit that referenced this issue Jun 8, 2017
Dunkat added a commit that referenced this issue Jun 8, 2017
Dunkat added a commit that referenced this issue Jun 8, 2017
damianprzygodzki added a commit that referenced this issue Jun 8, 2017
Combined Fields/ Lookup adjustment #839
damianprzygodzki added a commit that referenced this issue Jun 8, 2017
Revert "Combined Fields/ Lookup adjustment #839"
Dunkat added a commit that referenced this issue Jun 8, 2017
damianprzygodzki added a commit that referenced this issue Jun 9, 2017
@metas-mk metas-mk self-assigned this Jun 12, 2017
@metas-mk
Copy link
Member Author

Tested. I like it :-)
Works nicely as required. Closing.

metas-mk added a commit to metasfresh/metasfresh that referenced this issue Jun 12, 2017
Adding the release notes from issue metasfresh/metasfresh-webui-frontend-legacy#839 Combined Fields/ Lookup adjustment to this week's release.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants