Skip to content

feat(tui): improve Events screen hierarchy - #103

Open
ionfwsrijan wants to merge 7 commits into
0-Shimanshu:mainfrom
ionfwsrijan:feat/tui-events-screen-hierarchy
Open

feat(tui): improve Events screen hierarchy#103
ionfwsrijan wants to merge 7 commits into
0-Shimanshu:mainfrom
ionfwsrijan:feat/tui-events-screen-hierarchy

Conversation

@ionfwsrijan

@ionfwsrijan ionfwsrijan commented May 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Improves the Events screen visual hierarchy and scanability by restructuring the layout to clearly separate:

  • the review queue table (left)
  • selected event details + identity context (right)
  • the operator action bar (right, fixed at bottom)

Adds an Events header notice and stronger section boundaries so verdict/score/identity/endpoint/actions are easier to scan at a glance.

Why

The Events screen already had the right information, but it was visually “flat” and required extra eye effort to distinguish the queue, details/context, and actions. This change aligns the Events screen with the Audit screen’s clearer two-column structure for faster operator scanning.

Scope

  • bug fix
  • documentation
  • tests
  • TUI / CLI improvement
  • framework integration
  • signal change
  • configuration or route-policy change

Verification

Commands run:

python -m pytest -q

Notes:

  • I ran the relevant tests locally
  • I updated docs if behavior changed
  • I updated screenshots if the TUI changed materially

Signal-specific notes

Signal type:

  • hard signal (trackA)
  • soft signal (trackB)
  • not applicable

Screenshots

Before:

Screenshot 2026-05-20 110345

After:

Screenshot 2026-05-21 103242

TUI notes

  • before/after screenshots attached
  • not applicable

Local-only file check

  • I did not accidentally stage local-only files such as adiuvare.yaml or local conftest.py changes

Related issue

Closes #19

@github-actions github-actions Bot added the enhancement New feature or request label May 19, 2026
@ionfwsrijan
ionfwsrijan force-pushed the feat/tui-events-screen-hierarchy branch from 4d3fe92 to 32b199a Compare May 19, 2026 06:13
@ionfwsrijan

Copy link
Copy Markdown
Contributor Author

@0-Shimanshu

Review and merge sir.

@0-Shimanshu

Copy link
Copy Markdown
Owner

tui need ss to before after so i can see the change

@ionfwsrijan

Copy link
Copy Markdown
Contributor Author

@0-Shimanshu I've added the screenshots

@0-Shimanshu

Copy link
Copy Markdown
Owner

@ionfwsrijan i cant clearly understand what u have done with the image full screen the termina than take ss and maybe mark the changes

@ionfwsrijan

Copy link
Copy Markdown
Contributor Author

@0-Shimanshu Sir there's whole UI change as you can see in the screenshots. It is mentioned in the PR description as well. Pls read that -

• the review queue table (left)
•selected event details + identity context (right)
•the operator action bar (right, fixed at bottom)

@0-Shimanshu

Copy link
Copy Markdown
Owner

@ionfwsrijan bro I ment add full screen screen shot so atleast I can see the detail the section just show title I can't see any detail

The before screen shoot looks better to me then the after🤦

@ionfwsrijan

Copy link
Copy Markdown
Contributor Author

@0-Shimanshu I updated the after screenshot. I hope the changes are clear now

@0-Shimanshu

Copy link
Copy Markdown
Owner

The after image now show me the problem properly the detail don't fully show now as u can see it half rendering and half is not rendering because of the spacing

@ionfwsrijan

Copy link
Copy Markdown
Contributor Author

@0-Shimanshu You may review it now. The changes have been made.

@ionfwsrijan

Copy link
Copy Markdown
Contributor Author

@0-Shimanshu Sir you may review and merge

@Wahid7852

Copy link
Copy Markdown
Collaborator

hm, increased structural complexity without much validation in rendering stability, overflow handling, or smaller terminal behavior yet, should do this and good to merge, rest @0-Shimanshu do lemme know if anything in review was messed or if you wanna add anything else

Comment thread adiuvare/tui/screens/events.py
Comment thread adiuvare/tui/replit.tcss
Comment thread adiuvare/tui/replit.tcss
Comment thread adiuvare/tui/screens/events.py Outdated
Comment thread adiuvare/tui/screens/events.py Outdated
Comment thread adiuvare/tui/screens/events.py Outdated
Comment thread tests/test_tui.py
Comment thread adiuvare/tui/replit.tcss
@ionfwsrijan

Copy link
Copy Markdown
Contributor Author

@Wahid7852 ayo aap idhar? i was inactive for 2 days. will resolve this in a while.

@ionfwsrijan

Copy link
Copy Markdown
Contributor Author

@0-Shimanshu @Wahid7852 Sir the changes you asked have been made. You may review and merge this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tui: improve visual hierarchy and scannability in the Events screen

3 participants