On Mon, Mar 21, 2016 at 5:01 AM, Surinder Kumar
<surinder.kumar@enterprisedb.com> wrote:
> Hi,
>
> Please find updated patch.
>
> On Fri, Mar 18, 2016 at 9:18 PM, Dave Page <dpage@pgadmin.org> wrote:
>>
>> On Fri, Mar 18, 2016 at 2:36 PM, Surinder Kumar
>> <surinder.kumar@enterprisedb.com> wrote:
>> > Hi,
>> >
>> > I have updated the backform control for sql-field.
>> >
>> > Following are the enhancements:
>> > 1. Added border to control which was missing earlier.
>> > 2. Allows user to provide height to sql-field control by specifying
>> > extraClasses field in model schema.
>> > 3. Defined new classes for sql-field control height in overrides.css
>> > 4. Added scrollbar to the control.
>> >
>> > Usage:
>> >
>> > {
>> > id: 'definition', label:'{{ _("Definition") }}', cell: 'string',
>> > type: 'text', mode: ['create', 'edit'], group: 'Definition',
>> > control: Backform.SqlFieldControl,
>> > extraClasses:['custom_height_css_class']
>> > }
>> >
>> >
>> > Please review the attached patch and commit if it looks good.
>>
>> Thanks. Why is it full width, unlike other controls? That seems like
>> it needs fixing.
>
> It doesn't need full width. so I have fixed it.
Thanks - applied with some minor style changes for consistency.
There were a couple of minor issues that I didn't have time to fix -
I've added a TODO for that to our internal Kanban chart:
- The field is slightly wider than others, approx 3px at each end.
Ideally this just needs a margin adjustment, but that's messing up the
wrapping.
- The corners look a little odd.
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company