Problem/Motivation
The Varbase media header doesn't reflect on the preview
Steps to reproduce
- Create a node that has Varbase media header
- Choose media header
- Preview the node
Proposed resolution
adding the following code when ever getParameter('node') is used
if (empty($node)){
$node = $this->routeMatch->getParameter('node_preview');
}Remaining tasks
- ✅ File an issue about this project
- ✅ Addition/Change/Update/Fix to this project
- ✅ Testing to ensure no regression
- ➖ Automated unit/functional testing coverage
- ✅ Developer Documentation support on feature change/addition
- ✅ User Guide Documentation support on feature change/addition
- ✅ Accessibility and Readability
- ✅ Code review from 1 Varbase core team member
- ✅ Full testing and approval
- ✅ Credit contributors
- ✅ Review with the product owner
- ✅ Update Release Notes and Update Helper on new feature change/addition
- ✅ Release varbase-9.0.15, varbase_media_header-9.0.6, Varbase 10.0.0-beta1, varbase_media_header-10.0.0-alpha1
Varbase update type
- ✅ No Update
- ➖ Optional Update
- ➖ Forced Update
- ➖ Forced Update if Unchanged
User interface changes
Tested in Varbase
~9.0.0Test Basic page preview of the Full Content view mode - with Media Header
Test Basic page preview of the Teaser/Card view mode - with Media Header
Test Basic page - front-end view for the full content view mode - with Media Header
Test not saved yet new Basic page preview of the Full Content view - with Media Header
Test not saved yet new Basic page preview of the Teaser/Card view mode - with Media Header
![]()
Tested in Varbase
~10.0.0Test Basic page preview of the Full Content view mode - with Media Header
Test Basic page preview of the Teaser/Card view mode - with Media Header
Test Basic page - front-end view for the full content view mode - with Media Header
![]()
API changes
- N/A
Data model changes
- N/A
Release notes snippet
- Issue #3373397 by Ahmad Khader: Improved the preview option view for the
full contentview mode when a media header is present








Comments
Comment #2
ahmad khader commentedadding a patch to support the preview
Comment #3
ahmad khader commentedComment #4
ahmad khader commentedRerolling patch to 9.0.5 version
Comment #5
ahmad khader commentedComment #6
ahmad khader commentedUpdate the patch to the latest version of 10.0.x
Comment #7
ahmad khader commentedUpdate the patch to the latest version of 9.0.x
Comment #8
ahmad khader commentedComment #9
ahmad khader commentedComment #10
rajab natshahThank you, Ahmad, for reporting and patching.
To be committed to both
9.0.xand10.0.xbranches.Comment #11
rajab natshahComment #12
rajab natshahComment #15
rajab natshahIt only should show up for the
"Full Content"view modeNot other Teasers or Cards view modes
Comment #18
rajab natshahComment #19
rajab natshahComment #20
rajab natshah~9.0.0Configuring Varbase Media Header
~10.0.0Configuring Varbase Media Header
Comment #21
rajab natshah~9.0.0Configuring Varbase Media Header
~10.0.0Configuring Varbase Media Header
Comment #22
rajab natshahComment #23
rajab natshah✅ Released varbase_media_header-9.0.6
Comment #24
rajab natshah✅ Released varbase_media_header-10.0.0-alpha1
Comment #25
rajab natshah✅ Released varbase-9.0.15
Comment #26
rajab natshah✅ Released Varbase 10.0.0-beta1