There is no hook to alter individual tab content after the tab type plugin renders it. The only way to modify rendered tab output is to write a custom tab renderer.
A simple alter hook would allow modules to modify tab content without subclassing the renderer.
Add a hook_quicktabs_tab_content_alter() that fires after $object->render($tab) in both the server-rendered path (QuickTabs::render()) and the AJAX
controller (QuickTabsController::ajaxContent()):
$render = $object->render($tab); $this->moduleHandler->alter('quicktabs_tab_content', $render, $tab, $instance);
New hook: hook_quicktabs_tab_content_alter(array &$render, array $tab, QuickTabsInstance $instance).
Show commandsStart 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:
changes
Comments
Comment #2
loze commentedComment #4
loze commentedAdded the new hook.
Once #3606674: hook_quicktabs_instance_alter() not invoked on AJAX tab loads is merged, we can add test coverage for this hook using the same
quicktabs_alter_testmodule introduced there. The test would add ahook_quicktabs_tab_content_alter()implementationand verify it fires on all three render paths: the default tab in AJAX mode, all tabs in non-AJAX mode, and the AJAX controller for lazily loaded tabs.
Comment #5
joelpittetThanks for the proposed hook, sounds like a d7 feature that didn’t get ported
What is the use case for this?
Every feature comes at a cost of complexity, good to weigh it.
Comment #6
loze commentedThanks for taking a look.
My use case was adding contextual links to Views-based tabs. Views blocks and pages get the "edit view" pencil natively, but that gets lost when rendered through QuickTabs.
In digging into this though, I found that the initial/default tab actually already has contextual links without the hook, the View render array goes through the normal pipeline and
contextual_preprocess()picks up#contextual_linksjust fine. The problem is only with secondary tabs loaded via AJAX.In 4.2.1,
ViewContent::doRender()pre-renders AJAX tabs withrenderRoot()and returns#markup. By the time the alter hook fires, the render array is already flat HTML, so any#contextual_linksadded by the hook never reachcontextual_preprocess(). In 4.1.0 this worked becausebuildRenderable()was returned as-is for all tabs.So I think the real fix is adding #contextual_links inside
ViewContent::doRender()itself, after$view->buildRenderable()but before therenderRoot()call. That way they're part of the render array when it gets flattened and they survive the AJAX path. I'll open a separate MR for that.Given all that, the hook probably isn't necessary, at least not for my use case. If you don't see value in keeping it as a general extension point, I understand pulling it.
Comment #7
loze commentedSo, I ended up opening a new issue #3606793: Views tabs missing contextual links on AJAX-loaded tabs
If we do that that I don't need this alter hook.
Comment #8
joelpittetI am way more comfortable baking that contextual links in, but we can leave this open if there are some strong case/support for the extension point!
Thanks for the details for why you’re looking for this!