[Webkit-unassigned] [Bug 45870] [reviewtool] should always show overall comments text box

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Dec 8 19:14:33 PST 2010


https://bugs.webkit.org/show_bug.cgi?id=45870





--- Comment #6 from Adam Barth <abarth at webkit.org>  2010-12-08 19:14:33 PST ---
I'm slightly hesitating to r+ this patch because the new design makes the toolbar larger.  I have a dream that one day we're merge Review Patch and Formatted Diff into one screen, and having a heavy toolbar works against that goal.

I definitely think you're solving and important problem.  Have you considered a design where the overall comments box appears above the toolbar when you mouse over the toolbar (and disappears when you mouse out)?  That way you can have access to it wherever you are in the review but it doesn't contribute weight to the toolbar when you're not using it.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list