Re: patchview in gerrit - everyone happy or needs some bigger revamp?

2014-05-16 Thread Miklos Vajna
On Thu, May 15, 2014 at 03:12:21PM +0200, Christian Lohmaier 
lohma...@googlemail.com wrote:
 Any burning topics that you hate about gerrit's interface?

I guess the most burning issue right now is that sometimes the
submission gets stuck and you need to manually push the change + mark
the review as abandoned, which is ugly -- and IIRC this was fine in the
past, it came in after some upgrade. I think Norbert has the exact
details, sounds like an upstream bug.

UI annoyance is much less problematic, at least for me.


signature.asc
Description: Digital signature
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


patchview in gerrit - everyone happy or needs some bigger revamp?

2014-05-15 Thread Christian Lohmaier
Hi *,

is there still demand for a customized patchset view on gerrit?

a list lists the following items:

have important things on that page without JavaScript clicking
the commit message on top
the Review, Abandon, Rebase buttons right below that (removing the two
useless Diff-Buttons)
comments against the latest patch expanded (latest comment first)
older comments collapsed (thus taking no space)
the latest patchset including a full diff and all inline comments
delegate everything else to a 300px right-hand sidebar (a la
launchpads bugview layout1):
Owner, Branch, Topic, Dates, Reviewers, older versions of the change ...
= This should allow ideally to review the change with one view, as
commit message, diff, relevant comments (if any) are all on one page
and visible from the start.

Is there demand, or are people happy with using the combination of
gitweb and gerrit's own patchview?

Any burning topics that you hate about gerrit's interface?

ciao
Christian
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice