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

Auto-open tagest fields #12248

Merged
merged 1 commit into from
Jun 4, 2018
Merged

Auto-open tagest fields #12248

merged 1 commit into from
Jun 4, 2018

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Jun 1, 2018

Overview

Makes it easier to see what's in a tagset by opening the widget when clicking on it, as opposed to waiting till the user types.

Before

When clicking on a tagset field, you are prompted to type somthing before any tags display. This screenshot is from the "New Individual" form:
screenshot from 2018-06-01 21 31 11

After

When clicking on a tagset field, the first 10 tags appear (alphabetically). Scrolling down will show more than 10, allowing you to browse the entire list via the "infinite scroll" feature of select2. You can still type in the search box to filter the results, as before.
screenshot from 2018-06-01 21 32 15

Comments

I don't know why I didn't think to do this in the first place; it's much nicer.

https://civicrm.stackexchange.com/questions/20688/how-can-i-make-tags-in-tagsets-visible-to-users-who-want-to-tag-contacts/

@homotechsual
Copy link
Contributor

This is a nice Quality of Life change, I can't see any obvious downsides to making this change.

@colemanw
Copy link
Member Author

colemanw commented Jun 3, 2018

@lcdservices can you give a thumbs up or down on this change?

@monishdeb
Copy link
Member

Nice improvement and worked as mentioned in the description. Merging now.

@monishdeb monishdeb merged commit 26bd959 into civicrm:master Jun 4, 2018
@colemanw colemanw deleted the tagsets branch June 4, 2018 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants