Rename the per-orchestrator expander to Steps so it stops colliding with Advanced mode
This commit is contained in:
@@ -175,7 +175,7 @@ $runningScripts = array_unique($runningScripts);
|
|||||||
</label>
|
</label>
|
||||||
<span class="vv-job-dot"></span>
|
<span class="vv-job-dot"></span>
|
||||||
<?php if (!empty($orch['children'])): ?>
|
<?php if (!empty($orch['children'])): ?>
|
||||||
<button class="vv-advanced-toggle" style="margin-left:auto;width:110px" onclick="vvToggleAdvanced(this)">Advanced ▸</button>
|
<button class="vv-advanced-toggle" style="margin-left:auto;width:110px" onclick="vvToggleSteps(this)">Steps ▸</button>
|
||||||
<?php endif; ?>
|
<?php endif; ?>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
@@ -1630,12 +1630,16 @@ function vvSaveChild(el) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
function vvToggleAdvanced(btn) {
|
// Named Steps, not Advanced. The page already carries a global "Advanced" mode button that
|
||||||
|
// unlocks script source and raw conf editing — an unrelated control with a much sharper edge.
|
||||||
|
// Two buttons sharing a name on one page meant "go to Scheduler and click Advanced" could land
|
||||||
|
// someone in raw conf editing when all they wanted was to see what an orchestrator runs.
|
||||||
|
function vvToggleSteps(btn) {
|
||||||
const card = btn.closest('.vv-sched-card');
|
const card = btn.closest('.vv-sched-card');
|
||||||
const children = card.querySelector('.vv-children');
|
const children = card.querySelector('.vv-children');
|
||||||
const visible = children.style.display !== 'none';
|
const visible = children.style.display !== 'none';
|
||||||
children.style.display = visible ? 'none' : 'block';
|
children.style.display = visible ? 'none' : 'block';
|
||||||
btn.textContent = visible ? 'Advanced ▸' : 'Advanced ▾';
|
btn.textContent = visible ? 'Steps ▸' : 'Steps ▾';
|
||||||
}
|
}
|
||||||
|
|
||||||
function vvToggleTools(btn) {
|
function vvToggleTools(btn) {
|
||||||
|
|||||||
Reference in New Issue
Block a user