Re: [pgAdmin4][Patch]: Backgrid StringDepsCell

Hi Ashesh,

I have changed the function name (from render_deps to updateUIDeps), which will be called on the dependent field change, as per your suggestion.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company

http://www.linkedin.com/in/asheshvashi 

Please find attached patch for the same.

Thanks,

Khushboo

On Fri, Apr 1, 2016 at 3:56 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:

Hi Ashesh,

Yes, agreed. This implementation limits us to use only String Cell.

So, I have modified the patch as you suggested.

Please find the attachment for the same.

Thanks,

Khushboo

On Fri, Apr 1, 2016 at 2:59 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

Hi Khushboo,

I am reluctant to use this code as it is.

I would prefer you create just helper functions (i.e. initialize, remove) for the dependents in Backgrid.Cell, and then use them directly from the different cell itself.
Because - this functionality limits us to us only with the string type.

Anyway - we will be using the dependent functionality for very attributes.

You can extend them directly in the Node's model (schema).   

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company

http://www.linkedin.com/in/asheshvashi

On Wed, Mar 30, 2016 at 6:39 PM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:

Hi,

Please find the attached patch for the Backgrid StringDepsCell.

The StringDepsCell displays HTML escaped strings and accepts anything typed in.
Also, Listen to the dependent fields.

Usage of the Backgrid Cell:

If the Precision cell is dependent on the Datatype then:

{
  id: 'precision', label:'{{ _('Precision') }}', type: 'test'
  cell: Backgrid.Extension.StringDepsCell, deps: ['datatype']

}

Thanks,

Khushboo 

--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers