Thanks, patch applied. On Wed, Aug 9, 2017 at 12:19 PM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote:
> Hi, > > Please find the updated patch with new UI style in dashboard subnode > control based on Chethana's design suggestion. > > @Shirley, > Yes, DBA needs to click on the drop down to expand. > - If we make whole row clickable then how are we going to identify if user > is trying to expand the row or trying to cancel the active session? I am > asking this because table row <tr> tag comes before table column tag <td> > in HTML DOM, so if we have click event listers on both of the tags then how > are we going to differentiate the user operation? > - In my opinion, another reason we can not do that on a dashboard grid > because it will break the subnode control consistency, If user sees that > grid expands on clicking the row they will expect the same behaviour in > other grids as well. > > -- > Regards, > Murtuza Zabuawala > EnterpriseDB: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > [image: https://community.postgresrocks.net/] > <https://community.postgresrocks.net/> > > On Tue, Aug 8, 2017 at 7:47 PM, Shirley Wang <sw...@pivotal.io> wrote: > >> Nice! >> >> Would a DBA need to click on the drop down to expand? I think that drop >> down icon might be too small. Perhaps the whole row could be a clickable >> area to show further details. This is a good thing to test with some people >> >> >> On Aug 8, 2017, at 12:52, Ashesh Vashi <ashesh.va...@enterprisedb.com> >> wrote: >> >> On Mon, Aug 7, 2017 at 9:06 PM, Dave Page <dp...@pgadmin.org> wrote: >> >>> >>> >>> On Mon, Aug 7, 2017 at 3:49 PM, Chethana Kumar < >>> chethana.ku...@enterprisedb.com> wrote: >>> >>>> Hi Dave, >>>> >>>> Could you comment on the new design update for subnode control ? >>>> >>>> Attached designs - >>>> >>>> 1. subnode_current.png - The existing view >>>> 2. subnode_new.png - The updated view >>>> 3. current_new.png - Existing and new design placed together for >>>> comparison purpose >>>> >>>> Below changes done - >>>> >>>> 1. Reduced number of gray color variations >>>> 2. Made more simplified by removing unwanted borders >>>> 3. Merged "Arrow Down" background and the body background for more >>>> clarity >>>> >>>> Please feel free to provide your input on the same. >>>> >>> >>> Much nicer in my opinion! Good work. >>> >>> Anyone else have any comments? >>> >> It looks much nicer. >> I would like to see the effect on another dialogs like 'Table >> properties', where we have multiple level of subgrid controls with tabs. >> >> -- Thanks, Ashesh >> >>> >>> >>>> >>>> Regards, >>>> Chethana kumar >>>> >>>> >>>> >>>> On Fri, Jul 28, 2017 at 4:11 PM, Dave Page <dp...@pgadmin.org> wrote: >>>> >>>>> Hi >>>>> >>>>> I took a quick look at this and have a couple of thoughts: >>>>> >>>>> - Instead of the "edit" icon to open the subnode, we should use >>>>> something more appropriate - a "properties" icon perhaps. >>>>> >>>>> - There seems to be a lot of different shades of grey on there (maybe >>>>> a subnode design in general that just shows up with the disabled >>>>> controls), >>>>> and the subnode control looks a bit messy as a result. >>>>> >>>>> Can you work with Chethana to improve the look and feel please? >>>>> >>>>> Input from others welcome of course - screenshot attached. >>>>> >>>>> Thanks. >>>>> >>>>> On Fri, Jul 28, 2017 at 11:33 AM, Murtuza Zabuawala < >>>>> murtuza.zabuaw...@enterprisedb.com> wrote: >>>>> >>>>>> ++ Attaching the patch >>>>>> >>>>>> -- >>>>>> Regards, >>>>>> Murtuza Zabuawala >>>>>> EnterpriseDB: http://www.enterprisedb.com >>>>>> The Enterprise PostgreSQL Company >>>>>> >>>>>> On Fri, Jul 28, 2017 at 4:02 PM, Murtuza Zabuawala < >>>>>> murtuza.zabuaw...@enterprisedb.com> wrote: >>>>>> >>>>>>> Hi Dave, >>>>>>> >>>>>>> PFA patch to display additional information from pg_stat_activity >>>>>>> table using subnode control. >>>>>>> RM#2597 >>>>>>> >>>>>>> Please review. >>>>>>> >>>>>>> -- >>>>>>> Regards, >>>>>>> Murtuza Zabuawala >>>>>>> EnterpriseDB: http://www.enterprisedb.com >>>>>>> The Enterprise PostgreSQL Company >>>>>>> >>>>>>> On Tue, Jul 25, 2017 at 10:56 AM, Shirley Wang <sw...@pivotal.io> >>>>>>> wrote: >>>>>>> >>>>>>>> >>>>>>>>> On Mon, Jul 24, 2017 at 8:11 PM, Dave Page <dp...@pgadmin.org> >>>>>>>>> wrote: >>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> On Mon, Jul 24, 2017 at 3:28 PM, Shirley Wang <sw...@pivotal.io> >>>>>>>>>> wrote: >>>>>>>>>> >>>>>>>>>>> 2-3 days is a lot of valuable engineering time. Is this a 'drop >>>>>>>>>>> everything now' kind of feature or can this wait for some user >>>>>>>>>>> validation >>>>>>>>>>> on a mock up first? >>>>>>>>>>> >>>>>>>>>> >>>>>>>>>> Most of the time will likely be on the infrastructure to change >>>>>>>>>> the display to a subnode control. If you have some cycles to mockup >>>>>>>>>> potential layouts for the subnode view and have them validated, >>>>>>>>>> please feel >>>>>>>>>> free, however, that seems like an awful lot of work to me to display >>>>>>>>>> some >>>>>>>>>> missing SQL using a standard control. >>>>>>>>>> >>>>>>>>> Regarding SQL display: Developing simple control to show >>>>>>>>> codemirror in disabled state (for now) wont take that much time. >>>>>>>>> >>>>>>>>> >>>>>>>> Part of a product designer's job is to make sure there is a >>>>>>>> definitive need for a feature and that the interface for the feature is >>>>>>>> designed in such a way that the user gets all intended value from it. >>>>>>>> Time >>>>>>>> spent validating now will decrease the time spent later on redesigning >>>>>>>> / >>>>>>>> reimplementing. >>>>>>>> >>>>>>>> If everyone is aware of what that value is and confident that how >>>>>>>> it'll be displayed is right, there's little risk in starting to >>>>>>>> develop it. >>>>>>>> If we're wrong, it'll add to feature bloat and detract from the >>>>>>>> experience. >>>>>>>> >>>>>>>> Would Chethana be able to take on some of the design work? It would >>>>>>>> be valuable for the dev team to also be part of design process. >>>>>>>> >>>>>>> >>>>>>> >>>>>> >>>>> >>>>> >>>>> -- >>>>> Dave Page >>>>> Blog: http://pgsnake.blogspot.com >>>>> Twitter: @pgsnake >>>>> >>>>> EnterpriseDB UK: http://www.enterprisedb.com >>>>> The Enterprise PostgreSQL Company >>>>> >>>> >>>> >>>> >>>> -- >>>> Chethana Kumar >>>> Principal UI/UX Designer >>>> EnterpriseDB Corporation >>>> >>>> >>>> The Postgres Database Company >>>> >>>> P: +91 86981 57146 <+91%2086981%2057146> >>>> www.enterprisedb.com >>>> >>> >>> >>> >>> -- >>> Dave Page >>> Blog: http://pgsnake.blogspot.com >>> Twitter: @pgsnake >>> >>> EnterpriseDB UK: http://www.enterprisedb.com >>> The Enterprise PostgreSQL Company >>> >> >> > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company