[Engine-devel] Gerrit NEW Change Screen

Mike Kolesnik mkolesni at redhat.com
Tue Jan 28 07:05:21 UTC 2014


----- Original Message -----
> 
> 
> ----- Original Message -----
> > From: "Roy Golan" <rgolan at redhat.com>
> > To: "Itamar Heim" <iheim at redhat.com>, vdsm-devel at lists.fedorahosted.org,
> > "engine-devel" <engine-devel at ovirt.org>
> > Sent: Sunday, January 26, 2014 9:12:38 AM
> > Subject: Re: [Engine-devel] Gerrit NEW Change Screen
> > 
> > On 01/16/2014 12:08 AM, Itamar Heim wrote:
> > > with gerrit 2.8, there is a new change screen.
> > > its not enabled by default (yet), please use and see what you think.
> > >
> > > to enable, go to settings (click the top-right arrow next to your
> > > name, and choose settings).
> > > select preferences and set "Change View:" to "New Screen".
> 
> I had found that the new window truncates the code
> See for example line 115 in
> http://gerrit.ovirt.org/#/c/23428/2/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RestartVdsCommand.java,cm
> Switching to the old view will show all the line while in the new one it is
> truncated

It's just enforcing a certain line length (which this code obviously violates) ;)

I found that if you change the zoom on the page you'll get a scrollbar at the bottom so the line can be fully viewed..

> 
> > >
> > > Thanks,
> > >    Itamar
> > > _______________________________________________
> > > Engine-devel mailing list
> > > Engine-devel at ovirt.org
> > > http://lists.ovirt.org/mailman/listinfo/engine-devel
> > +1 needs some time to get used to the new layout but overall its way
> > better to have the patch dependency ("Related")
> > and better without the cluttered patch-sets view.
> > reminds me of gitk somewhat...
> > 
> > 
> > _______________________________________________
> > Engine-devel mailing list
> > Engine-devel at ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/engine-devel
> > 
> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel
> 



More information about the Devel mailing list