Closed (fixed)
Project:
Drupal core
Version:
8.1.x-dev
Component:
entity_reference.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
3 Jul 2015 at 08:59 UTC
Updated:
14 Mar 2016 at 04:04 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #1
pguillard commentedNot a great patch, but often simple things are better...
Comment #2
pguillard commentedComment #3
lewisnymanI wouldn't like to introduce a custom element to fix this problem. It feels inconsistent.
Comment #4
pguillard commentedSome specific CSS border would be bad in form.css file.
We can put the auto_create field in a fieldset, hum.
I have no idea
Comment #5
pguillard commentedI removed the addition of a custom element, and used some CSS, using a border-top as separator.
What do you think ?
Comment #6
pguillard commentedComment #7
pguillard commentedRerolled
Comment #8
pguillard commentedComment #9
lewisnyman@pguillard I don't think it's a bad idea. I'm not sure about adding a new visual component that we haven't used elsewhere before. I'm trying to move away from modules overriding styling changes on individual pages, rather than just using the toolkit of components that Seven provides.
What if we just move that checkbox below the "reference method" field? That would separate them.
Comment #10
pguillard commentedI guess we can't separate the 2 checkboxes that belong to the same list.
This is why I tried with CSS.
Comment #11
amateescu commentedI agree with @LewisNyman, let's just move the auto-create checkbox above the bundle selection form element.
Comment #12
pguillard commentedIt is easier now (The code has changed a lot!).
This is a totally new patch, without interdiff then.
This gives :

Comment #13
jibranCan we change user facing strings now? Aren't we in string freeze?
Comment #14
pguillard commentedHum, you are right ! Depend if we keep this in 8.0.x or for 8.1.x ?
I guess this string improvement is not so important. I removed it.
Comment #15
xjmAs a UI change, this is a minor version target.
Edit: And as such, we can include string changes if we agree they are part of the best fix. Reference: https://www.drupal.org/core/d8-allowed-changes#minor
Comment #16
pguillard commentedComment #17
pguillard commented#12 is still available with the string change.
Comment #18
NikitaJain commentedTested and verified the recent patch "taxonomy_term_reference_settings_form-2522950-12.patch" and taxonomy_term_reference_settings_form-2522950-14.patch both are working fine.
After applying the patch - 2522950-12 , "Create referenced entities if they don't exist" this option appears above the "Vocabularies" option. The string change from "Vocabularies" to "Available vocabularies".
For patch - 2522950-14: Same result without the string change. So now its easy to make out the settings for the 'taxonomy term' field type.
Screenshots attached.
Comment #19
catchCommitted/pushed to 8.1.x, thanks!
We could commit the weight-only change to 8.0.x, but it's not a functional bug as-such so leaving as is for now.