Comment 82 for bug 115484

Revision history for this message
In , Friedrich-beckmann (friedrich-beckmann) wrote :

(In reply to comment #75)
> (From update of attachment 342082 [details])
> I already gave the plus for all the screenshots, as usual I expect the other
> reviewer to look at the code itself.
thank you for the review!
(In reply to comment #71)
> (From update of attachment 342083 [details])
> I don't really like how the "Allow remote images" lines up here. But it's not
> too big a deal and I'm hopeful it could be removed (bug 457296) Otherwise
> things look fine.
I have it here in this tab because this is the "email" section. I do not know a better solution.
(In reply to comment #72)
> (From update of attachment 342084 [details])
> The City text box seems larger than it needs to be, however I don't know how
> you'd arrange the extra space. It's better to use it up like this than to have
> the empty block in the middle.
I was more concerned about the smaller department and title box which I put in one line. As you mention in your comment, the width is just there to make the city box that wide.

Again, thanks for the review!

Friedrich