Coming from the review in #1847596-119: Deprecate Taxonomy term reference field in favor of Entity-reference and the following discussions, I think the easiest actionable item for the moment is to not select all bundles by default and make the element required.

Files: 
CommentFileSizeAuthor
#1 1953444.patch2.28 KBamateescu
PASSED: [[SimpleTest]]: [MySQL] 53,362 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new2.28 KB
PASSED: [[SimpleTest]]: [MySQL] 53,362 pass(es).
[ View ]

Status:Needs review» Reviewed & tested by the community

So, I agree that this field needs to be required, technically speaking, since a reference field that doesn't reference anything doesn't make sense. I'm still concerned that this is "visibility through validation error" and doesn't entirely solve the original UX problem, but that should be addressed separately.

Thanks @amateescu!

Title:Users can easily skip the 'target bundles' section from the Entity reference instance settings formMake 'target bundles' required on the the Entity reference instance settings form

Issue tags:+Quick fix

Title:Make 'target bundles' required on the the Entity reference instance settings formMake 'target bundles' required on the Entity reference instance settings form
Status:Reviewed & tested by the community» Fixed

Ok, so I guess the point in doing this is so to address "This is the most important part of the form and I skipped right past it" and "It's not the 80% use case to search all elements"? I can get behind that. I agree that it's definitely not a full solution, but I think it's probably good enough for a quick fix to those particular issues.

Committed and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.