861: Review dialog is XBox-huge can't be resized, and can't be moved

jameslin
Aug. 30, 2010
What's the URL of the page containing the problem?


What steps will reproduce the problem?
1. View a review request.
2. Click the Review button.

What is the expected output? What do you see instead?
The new Review dialog fills up almost my entire browser window.
1. Really large text boxes are somewhat unwieldy.
2. It obscures the change description.  Many times when I want to make
general comments that aren't specific to particular lines in the diff, I'm
addressing some point or question raised in the change description.
(Ideally I'd be able to comment on specific lines in the description as I
would for the diff, but maybe that's overkill.)

It'd be nice if this dialog could be resized and moved, at least. (Although
I suppose that raises the question about what to do about the
darkened/drop-clothed background.)

What operating system are you using? What browser?
Firefox 3.0.5 on Windows XP x64 SP2

Please provide any additional information below.
chipx86
#1 chipx86
Hrm, yeah, that's a hard one. We really can't make this a free-floating dialog with
the current design. I'll have to think about this but I won't have an immediate
solution except to open a separate browser window for the thing you want to reference.

We wanted it large so that it acted more like a review on the review request page.
There was a lot of demand for this kind of design.

I'm not going to consider this a bug, since it really is by design, but maybe we can
look into some way of "undocking" this window on demand. I don't know yet.
  • +Confirmed
  • -Type-Defect
    +Type-Enhancement
    +Component-Reviews
david
#2 david
Perhaps we could embed the change description at the top?
david
#3 david
  • +Review dialog is XBox-huge can't be resized, and can't be moved
david
#4 david
I'm going to close this out for now. We haven't heard of any other demand for changing this, so unless the situation changes, I think the current design is at least a local maximum.
  • -Confirmed
    +WontFix