Hi, On Fri, Jun 30, 2017 at 10:30 PM, Surinder Kumar < surinder.ku...@enterprisedb.com> wrote:
> Hi, > > Review comments: > > 1) I see the font used for class .query-history .entry is monospace, > shouldn't it be Helvetica as per style guide ? the font for Messages > text > is also monospace. > > 2) Can 1px top border be added above first entry in left panel to > differentiate from above panels ? > > 3) No query execution message appears if i run same query second time, > please refer screenshot. > I think this isn't related to this patch, just now I have sent patch for similar issue. > > Apart from this patch looks good. > > Thanks, > Surinder > > > > On Fri, Jun 30, 2017 at 9:58 PM, Robert Eckhardt <reckha...@pivotal.io> > wrote: > >> Surinder, >> >> It would be great if you could take a look at this an see if the style >> changes are more inline with what you were thinking when you commented on >> the first patch. >> >> -- Rob >> >> On Fri, Jun 30, 2017 at 11:55 AM, Shruti Iyer <si...@pivotal.io> wrote: >> >>> Hi Hackers, >>> >>> Attached is the patch that refactors the react components that supports >>> the history detail pane. We moved inline styling. whenever possible, to >>> scss. >>> >>> Thanks, >>> Joao and Shruti >>> >> >> >
