feat(plugins): Refactor auto-hide functionality to leverage Bootstrap…

[jokob-sk]

…'s tab management for improved visibility handling

Summary by CodeRabbit

• Bug Fixes
  • Fixed tab navigation to properly activate the first visible tab when the active left-nav tab is missing, with improved initialization handling.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e6c8c663-8012-43f7-ac7f-a2c50011c412

📥 Commits

Reviewing files that changed from the base of the PR and between adc78e6 and cd0a3f6.

📒 Files selected for processing (1)

• front/pluginsCore.php

---

📝 Walkthrough

Walkthrough

Refactored the autoHideEmptyTabs function in front/pluginsCore.php to simplify tab activation logic by removing manual active class management and delegating state updates entirely to Bootstrap's .tab('show') method.

Changes

Cohort / File(s)	Summary
Tab Activation Refactoring front/pluginsCore.php	Removed manual active class assignments and targetPrefix variable from autoHideEmptyTabs. Tab switching now relies solely on Bootstrap's .tab('show') to manage both the nav item and pane active states automatically.

Possibly Related PRs

• feat(plugins): Implement auto-hide functionality for empty plugin tabs #1578: Directly modifies the same autoHideEmptyTabs function with related changes to tab-activation flow handling.

Poem

🐰 A tab switcheroo, clean and bright,
Bootstrap handles all the sleight,
No more manual class manipulation,
Just one method—pure automation!
Simpler code, a rabbit's delight! 🌟

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: refactoring the auto-hide functionality to leverage Bootstrap's tab management, which aligns with the code modifications.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch next_release

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}