Problem/Motivation
UI Patterns Overrides is a temporary module to prototype some UI Patterns additions before pushing them there:
├── src
│ ├── Plugin
│ │ └── UiPatterns
│ │ └── Source
│ │ ├── BlockSource.php
│ │ ├── ComponentSource.php
│ │ ├── ContentEntitySource.php
│ │ ├── LocalActionsSource.php
│ │ ├── LocalTasksSource.php
│ │ ├── MainPageContentSource.php
│ │ ├── PageTitleSource.php
│ │ └── WysiwygWidget.php
│ └── SourcesBundlerInterface.php
├── ui_patterns_overrides.info.yml
├── ui_patterns_overrides.module
└── ui_patterns_overrides.ui_patterns.inc
Proposed resolution
- BlockSource, ComponentSource, SourcesBundlerInterface & ui_patterns_overrides.module can be removed when #3529260: Flatten the Fields list (pending ui_patterns 2.0.8) and #3513568: [2.0.7] Source usability : better interfaces for sources relying on child plugins are done.
- WysiwygWidget (which is implementing SourcesBundlerInterface for the
::getDataSkeleton()method may need a special treatment - MainPageContentSource, PageTitleSource, LocalActionsSource and LocalTasksSource can be moved to
ui_patterns_page_layout - ui_patterns_overrides.ui_patterns.inc can be moved to main
ui_patternsmodule? We should be able to have it in display_builder without failing whendisplay_builder_uiis enabled. - ContentEntitySource can be removed. We will get back when #3505811: [2.1.0] Add content source will be done
Issue fork display_builder-3529468
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
Comment #2
pdureau commentedComment #3
mogtofu33 commentedComment #4
pdureau commentedLooks like a cleaning task, so a beta phase task
Comment #5
pdureau commentedBy the way, some of the logic expected in
Drupal\ui_patterns_overrides\Plugin\UiPatterns\Source\ComponentSourceis still inComponentLibraryPanel.Comment #6
pdureau commentedComment #7
pdureau commentedFollowing #3529260: Flatten the Fields list (pending ui_patterns 2.0.8) , the tasks described in the issue summary will change a bit.
Comment #8
pdureau commentedComment #9
pdureau commentedLet's also fix that:
And do this remaining task from #3534579: Align source contexts:
Comment #11
pdureau commented