Hi there,

I noticed a couple of things which I would like to see fixed. 
1. When you try to reply to a review comment, it jumps to the previous or next 
comment and you have to scroll again to get back to it. 
2. When you scroll down through a file, the Patch Set bar gets hidden. You have 
to scroll all the way up to see it again.
I saw this behavior with both chrome and firefox.

Aish.


> On Dec 22, 2015, at 7:13 AM, Jay Bryant <jsbry...@electronicjungle.net> wrote:
> 
> All,
> 
> Has anyone found a way to do code reviews on an iPad.  In the old Gerrit you 
> could.  With the new interface attempts to enter comments don't work.  It 
> looks like you should be able to enter text but when you type the page just 
> jumps around.  Very frustrating.
> 
> I have tried Safari and Chrome with no luck.  
> 
> Jay
> 
> On Tue, Dec 22, 2015 at 5:31 AM Steve Gordon <sgor...@redhat.com 
> <mailto:sgor...@redhat.com>> wrote:
> ----- Original Message -----
> > From: "Lenny Verkhovsky" <len...@mellanox.com <mailto:len...@mellanox.com>>
> > To: "OpenStack Development Mailing List (not for usage questions)" 
> > <openstack-dev@lists.openstack.org 
> > <mailto:openstack-dev@lists.openstack.org>>
> >
> > Hi,
> > After Upgrading to 2.11 seems that
> > lastcomment (
> > https://github.com/openstack/third-party-ci-tools/tree/master/monitoring/lastcomment-scoreboard
> >  
> > <https://github.com/openstack/third-party-ci-tools/tree/master/monitoring/lastcomment-scoreboard>
> > )
> >   monitoring tool stopped working
> >
> > I've posted a small patch https://review.openstack.org/#/c/259083/ 
> > <https://review.openstack.org/#/c/259083/> to fix the
> > issue,
> > But I wander if there is a proper rest query to for CI name?
> > Currently there is only account_id field in Gerrit response and it's not
> > really human readable.
> 
> I ran into this with a custom script as well, you can use the account_id in a 
> request of the form:
> 
> review.openstack.org/accounts/ 
> <http://review.openstack.org/accounts/><account_id>/name
> 
> ...to get a human readable account name.
> 
> -Steve
> 
> > Lennyb.
> >
> >
> > -----Original Message-----
> > From: Zaro [mailto:zaro0...@gmail.com <mailto:zaro0...@gmail.com>]
> > Sent: Tuesday, December 22, 2015 3:21 AM
> > To: OpenStack Development Mailing List (not for usage questions)
> > <openstack-dev@lists.openstack.org 
> > <mailto:openstack-dev@lists.openstack.org>>
> > Subject: Re: [openstack-dev] [OpenStack-Infra] Gerrit Upgrade to ver 2.11,
> > completed.
> >
> > Hit '?' and it says '/' is find, give that a try.
> >
> > Looks like in Gerrit 2.11 the 'f' to get a popup of the list of files is 
> > only
> > available in unified diff view.  I don't remember if it was available in
> > side-by-side view on Gerrit 2.8.
> >
> > -Khai
> >
> >
> > On Mon, Dec 21, 2015 at 5:04 PM, Carl Baldwin <c...@ecbaldwin.net 
> > <mailto:c...@ecbaldwin.net>> wrote:
> > > I also really miss being able to pull up the list of files in diff
> > > view with the 'f' key.  Any equivalent?
> > >
> > > Carl
> > >
> > > On Mon, Dec 21, 2015 at 4:07 PM, Carl Baldwin <c...@ecbaldwin.net 
> > > <mailto:c...@ecbaldwin.net>> wrote:
> > >> I noticed a couple of things today while reviewing a largish page [1].
> > >> First, when I search for something using the browser's builtin search
> > >> (Chrome, Mac OSX Yosemite), it doesn't seem to find occurrences that
> > >> are not in the visible portion of the page.  For example, when I
> > >> search for DNSDOMAIN, I get 1 hit from the top of the file.  The file
> > >> actually has almost 30 hits for this string.  For example, scroll
> > >> down to about L310 in the file.  You'll see them all over the place
> > >> (and now Chrome's search finds these hits if you try again).  I use
> > >> to use search within a file with the old gerrit and never noticed
> > >> this problem.
> > >>
> > >> The other thing that I found annoying is when I scroll the page with
> > >> my trackpad, it now jumps around sometimes to a different part of the
> > >> page.  For example, I'll scroll up to find a spot in the file and
> > >> when I think I've arrived, it will jump back down the file a bit.  It
> > >> is disorienting.  Of course, now it isn't doing it anymore so it
> > >> doesn't seem to be all the time.  It was behaving this way for a good
> > >> 20 minutes trying to get through this file.
> > >>
> > >> Carl
> > >>
> > >> Carl
> > >>
> > >> [1] https://review.openstack.org/#/c/212213/36/neutron/db/dns_db.py 
> > >> <https://review.openstack.org/#/c/212213/36/neutron/db/dns_db.py>
> > >>
> > >> On Thu, Dec 17, 2015 at 8:17 PM, Brian Haley <brian.ha...@hpe.com 
> > >> <mailto:brian.ha...@hpe.com>> wrote:
> > >>> On 2015-12-16 16:24, openstack-dev-requ...@lists.openstack.org 
> > >>> <mailto:openstack-dev-requ...@lists.openstack.org> wrote:
> > >>>>
> > >>>> Thanks to everyone for their patience while we upgraded to Gerrit
> > >>>> 2.11.  I'm happy to announce that we were able to successfully
> > >>>> completed this task at around 21:00 UTC.  You may hack away once more.
> > >>>>
> > >>>> If you encounter any problems, please let us know here or in
> > >>>> #openstack-infra on Freenode.
> > >>>
> > >>>
> > >>> I'm still undecided on 2.11, have to give it more time, but I have
> > >>> noticed one thing that's annoying...
> > >>>
> > >>> Trying to copy text from a review no longer works easily.  When I
> > >>> highlight text there's a little "bubble" pop-up of {press c to
> > >>> comment}, which seems to interfere with both my three-button mouse copy
> > >>> buffer, as well as Ctrl-C.
> > >>> Call me a nitpicker, but having to highlight text, right-button
> > >>> click, Copy, right-button click, Paste, is a pain.
> > >>>
> > >>> Maybe someone has a simple work-around for that.
> > >>>
> > >>> -Brian
> > >>>
> > >>> ____________________________________________________________________
> > >>> ______ OpenStack Development Mailing List (not for usage questions)
> > >>> Unsubscribe:
> > >>> openstack-dev-requ...@lists.openstack.org?subject:unsubscribe 
> > >>> <http://openstack-dev-requ...@lists.openstack.org/?subject:unsubscribe>
> > >>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev 
> > >>> <http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev>
> > >
> > > ______________________________________________________________________
> > > ____ OpenStack Development Mailing List (not for usage questions)
> > > Unsubscribe:
> > > openstack-dev-requ...@lists.openstack.org?subject:unsubscribe 
> > > <http://openstack-dev-requ...@lists.openstack.org/?subject:unsubscribe>
> > > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev 
> > > <http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev>
> >
> > __________________________________________________________________________
> > OpenStack Development Mailing List (not for usage questions)
> > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe 
> > <http://openstack-dev-requ...@lists.openstack.org/?subject:unsubscribe>
> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev 
> > <http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev>
> >
> > __________________________________________________________________________
> > OpenStack Development Mailing List (not for usage questions)
> > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe 
> > <http://openstack-dev-requ...@lists.openstack.org/?subject:unsubscribe>
> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev 
> > <http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev>
> >
> 
> --
> Steve Gordon,
> Sr. Technical Product Manager,
> Red Hat Enterprise Linux OpenStack Platform
> 
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe 
> <http://openstack-dev-requ...@lists.openstack.org/?subject:unsubscribe>
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev 
> <http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Reply via email to