launchpad-dev team mailing list archive

[Henning Eggers <henning.eggers@xxxxxxxxxxxxx>]

Am 29.07.2010 01:12, schrieb Paul Hummer:
> I'm not sure why you're surprised this passed UI review, and that may
> be because there's a bit of confusion as to what a UI review is.  A UI
> is not a QA review.  It's simply not realistic to go through every
> affected page and see what might be changed.  I trust the submitter
> to have thought about that, and demonstrate it in the UI review.  As a
> UI reviewer, I look at the UI a submitter presents to me, and I think
> about things that would make this change specifically bad. If I can't
> think of anything, I think of something that might make it better.  In
> this case, I didn't have either.  That's how it passed UI review.

Hm, I have never been trained as a UI reviewer so I will have to trust you on
that. I always thought that UI reviews were mainly about keeping the UI
consistent across all pages and maintaining usability. That would include
checking how other pages are affected by the change.

Cheers,
Henning