Problem/Motivation

Island Plugin deserve a clean and rationalization.

Current `IslandPluginBase` is a bit big, with 5 services, some islands do not need so much, let introduce a lighter base class if possible and a more complete abstract `IslandPluginAdvanced`.

As well new service need to be evaluate to reduce complexity if possible.

Done

Extensive code removed from `defaultConfiguration`. => Done

Trait removed and creation of a service. => partial, service to evaluate

Rationalize injection and construction of islands => Done

To do

Invert the providers list #3540727: Source library panels: from allowed to excluded providers => Done

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

mogtofu33 created an issue. See original summary.

mogtofu33’s picture

Assigned: Unassigned » mogtofu33

mogtofu33’s picture

Issue summary: View changes
mogtofu33’s picture

Issue tags: +beta blocker
mogtofu33’s picture

Issue summary: View changes
Status: Active » Needs work
mogtofu33’s picture

Title: Island lighter base class » Island Plugin refactor and rationalization
mogtofu33’s picture

  • mogtofu33 committed 81d66a5c on 1.0.x
    [#3545303] feat: Island class clean, exclude instead of allowed
    
    By:...

mogtofu33 changed the visibility of the branch 3545303-island-pass-1 to hidden.

  • mogtofu33 committed 6f73930a on 1.0.x
    [#3545303] Island Plugin refactor and rationalization
    
    By: mogtofu33
    
mogtofu33’s picture

Assigned: mogtofu33 » Unassigned
Status: Needs work » Fixed

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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