commonlib: Output panel indicator updates #6215
Conversation
- Show the unread indicator on the main output panel as well. The indicator goes away when all tabs (including detached ones) are read. - Mark tab as read if the mouse enters the toolbar region. Earlier, it was possible to clear a panel using the broom button while it was still marked unread. This behaviour is fixed now. - Reduce the size of the green-badge-corner.png image by removing unnecessary metadata or attributes from it (creation time, resolution, and background color). - Rename "outputPanel" variables to "outputTab" for clarity. Signed-off-by: ricekot <git@ricekot.com>
|
Great job, no security vulnerabilities found in this Pull Request |
addOns/commonlib/src/main/java/org/zaproxy/addon/commonlib/ui/TabbedOutputPanel.java
Show resolved
Hide resolved
Do you mean an option to:
Any specific use-case you are thinking of for this option? On a side note I've been thinking about the pros / cons of exposing options to the user vs making the decision for them (especially for UI related things). Adding more options gives more choice to the user, but it could also make the tool feel more overwhelming. |
|
I was thinking point 1. I'm just considering that while output for errors etc might be helpful there may be situations where users don't need to know about every update to the tab. Ex: the current just general output tab, I hardly ever look at unless I'm after something specific. I dunno, you're right there definitely is a balance between configurability and complexity. I'm only one voice, we could wait and see if anyone else brings it up. |
Overview
Related Issues
Follow up to #6198.
Checklist
./gradlew spotlessApplyfor code formatting