refactor: DRY evidence tab page wrapper

[somethingwithproof]

Summary

Implements issue #22 by removing duplicated evidence tab page wrapper logic.

Changes

• added shared helper evidence_render_tab_page() in include/ui_helpers.php
• updated evidence_tab.php:
  • find action now uses evidence_render_tab_page('evidence_find')
  • default action now uses evidence_render_tab_page('evidence_stats')
• added regression test: tests/test_tab_wrapper.php

Validation

• php -l evidence_tab.php
• php -l include/ui_helpers.php
• php -l tests/test_tab_wrapper.php
• php tests/test_tab_wrapper.php

Issue

Closes #22

[Copilot]

Pull request overview

Refactors the Evidence tab page to remove duplicated “wrapper” rendering logic by introducing a shared helper and routing tab actions through it, with a small regression test to ensure wrapper call order and that the helper is used.

Changes:

• Added evidence_render_tab_page() helper to centralize general_header() → evidence_display_form() → content → bottom_footer() ordering.
• Updated evidence_tab.php to use the helper for the find and default actions.
• Added a lightweight regression test validating wrapper call order and asserting the new helper calls exist in source.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
include/ui_helpers.php	Adds shared evidence_render_tab_page() wrapper helper.
evidence_tab.php	Routes find and default actions through the new helper and includes it.
tests/test_tab_wrapper.php	Adds a standalone regression test for wrapper order and helper usage.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.