[2.0.0-beta1] UX of EntityFieldSource

Christian.wiedemann → made their first commit to this issue’s fork.

When i say:

Remove "Field" from fields select list

I mean, going from "Field Title" to "Title", from "Field Body" to "Body". We already know it is a field.

* I removed the title from the fieldset
* Sort the field.
* Remove the field from the list

hello,
about the change in " src/Plugin/Derivative/EntityFieldSourceDeriverBase.php " in the current MR

-            'label' => $this->t("Field @field", ["@field" => $field_data["label"]]),
+            'label' => $this->t("@field", ["@field" => $field_data["label"]]),

it may be better to update the label inside child class of EntityFieldSourceDeriverBase class instead of this base class,
because it may have implications on all source plugin derivers.
Here you just wanted to update the "DerivableContextDeriver" right? then you can update the label of derived plugins in that class directly :)

And that would be interesting also to remove the fieldset (e.g. in that usage of slotForm or propForm)
the simpler the better :) ?

I am only care about the wording results, if you agree on a simpler way of doing that, let's go :)

@just_like_good_vibes
* I added functions which can be overwritten by sub classes. Not sure if it is too much. But now it is easy to change the title, description in sub classes.
* I also prefer container but we should do this in the big UI refactor update.

I also want to review before the merge

Hello, i did not understand your previous comment Pierre,
maybe your assigned me as a mistake?
anyway, i checked the content of the MR. what is done seems visually ok, although i did not tested myself the code.

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