[1.0.0-alpha1] Make Island plugins configurable

It would be better to start working on this after 📌 [1.0.0-beta1] Remove Island's specific Form classes Active is merged

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

Hi all, I set this to review. It is not done but a short review would help me to know if I am on the right path. We can discuss that on wendsday

I did:

* Add PluginFormInterface, ConfigurableInterface to IslandInterface
* Implement a sample base form with a dummy label.
* Add forms.
* Pass configuration to island.

Questions:

* Should we make PluginFormInterface optional and add a basic implementation as a trait?
* Are there some basic configurations for every island.

Are there some basic configurations for every island.

it is debatable, but i don't see any configuration shared by all islands. I can picture islands with no configuration.

By the way, you didn't add those first configs:

• components providers in component library (by default, disable display_builder). Today, it is hardcoded in ::getDefinitionsForProvider()
• block providers in block library (by default, disable ui_patterns_layouts)
• display "Clear" button in history buttons (must be hidden by default)
• buttons with labels, icons, or both.

Do we focus this issue to the mechanism, and move the addition of the first configs to island plugins in a follow-up issue?

TODO in the scope of this issue:

• display "Clear" button in history buttons (must be hidden by default)

Pierre will move this to another issue:

• components providers in component library (by default, disable display_builder). Today, it is hardcoded in ::getDefinitionsForProvider()
• block providers in block library (by default,disable ui_patterns_layouts)

I finalized the island configuration part. Happy about review.

The expected config was not to display all the buttons or not, but only the "Reset" button:

display "Clear" button in history buttons (must be hidden by default)

For the the configuration form of History island raise an ajax error without message.

• Fresh install
• Go to Default islands configuration
• Click configure in front of History
• -> Ajax error

I struggled on the rebase so canceled it, would be probable easier to squash the current mr to one commit.

mogtofu33 → changed the visibility of the branch 3529067-make-island-plugins to hidden.

I created a rebased version, bug is still there, and cannot add a new DB config as well.

Fixed the configure click problem, the route {display_builder}/{instance_id} was conflicting with {display_builder}/delete
Still have a problem when creating a new config of DB islands, the route require an id which is not ready yet and will raise an error.

Fixed the new case, some style review and should be ok.