Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lyZ45-0000TN-Aq for pgadmin-hackers@arkaria.postgresql.org; Wed, 30 Jun 2021 12:11:37 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1lyZ44-0005Jt-BR for pgadmin-hackers@arkaria.postgresql.org; Wed, 30 Jun 2021 12:11:36 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lyZ43-0005Jl-51 for pgadmin-hackers@lists.postgresql.org; Wed, 30 Jun 2021 12:11:36 +0000 Received: from mail-ej1-x62a.google.com ([2a00:1450:4864:20::62a]) by makus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1lyZ3z-0008F8-H8 for pgadmin-hackers@postgresql.org; Wed, 30 Jun 2021 12:11:34 +0000 Received: by mail-ej1-x62a.google.com with SMTP id b2so3831120ejg.8 for ; Wed, 30 Jun 2021 05:11:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pgadmin.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=zXbJXFgBW+931QfbXz+cdFYef797D2ZVEKXqmLkQrUM=; b=kHY8AOXw3ZLknHAxJdi+1J/raSdiMfL9zWhbjKoYvEievbHEFw5BYZuOAiqolgPS5V 9IFvN5jiUMKTgur0y1HHt9s3qce5uvbROkVEcUBbZxFHw9h2asotzqB7F4gzXFa+Urzz SQ4Qs8PEuejudBr9Xqdo89mfIWhqW2eUQAEy4fXe+Bz1ePSMNGVw2XSyivj3ysd3CR26 KSNXEDASQI8szqVdHNglp5Pvsv32nFW+nZ36vBXw9VqJF7aReSswnyY/SMfcRZC/dn3Y 2ecbBum2JQJR/GdeKu0kt6CchowVHCXloZ4ffWsLULJEvvyoCc7RAEr9E7h4LqlXr3Fg J+Ag== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=zXbJXFgBW+931QfbXz+cdFYef797D2ZVEKXqmLkQrUM=; b=t2ca1VnSmr5i0xOmxpxNhBzI8UXM2MqLGujRq9ZfQqvkAvoPiWWcjbjT7AtIMCK0iD l9PC5UK/wEhz/zxKlO/60P8KQSYr9EtdCA3qbdtvyGRUWzaTbWeDc3f3kCgp5Ij6ukbT JRCppHJL/x02ZrGxR7Q7UDdpDLNpF4hO6KH4lj/x620D2vXix33c8P+td7z+ukLyQZ5D LJZB09FMkv4h1mZ8VLnBk5dAbhWcJhHP2xbH5zy8CUUcrrhh6PbMeTdmNa2w0wrWjTGf srD0rTnfF2W6PddS0dLVFmP6RFZdyXdtrtd72OaQl7KDhuJbfuyA60VdUR59jiX774aR TvoA== X-Gm-Message-State: AOAM532lc7N+r1MVwKKwLCGgNLEiLfH5mM3BIakWbMvV5wHme0g4gwzi m44pTBgv4LsX17RfJYPaWW/rRq1qqsyMj0eAkHNgYw== X-Google-Smtp-Source: ABdhPJyFnUQ3Y/99Jmyi3+uMMwcVPWmPkcRWRJiI7YKDCWMijrfWSruzI9dg7pRVdGQby3Mdq11QI3Gp9zSH0c+s60E= X-Received: by 2002:a17:906:b0c8:: with SMTP id bk8mr35697805ejb.412.1625055089323; Wed, 30 Jun 2021 05:11:29 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: Dave Page Date: Wed, 30 Jun 2021 13:11:17 +0100 Message-ID: Subject: Re: RM 6427 - Newlines in text columns of resultset show an allegedly "empty" field To: Khushboo Vashi Cc: pgadmin-hackers , Nikhil Mohite Content-Type: multipart/alternative; boundary="00000000000057080d05c5fa9f8d" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk --00000000000057080d05c5fa9f8d Content-Type: text/plain; charset="UTF-8" Hi On Wed, Jun 30, 2021 at 12:36 PM Khushboo Vashi < khushboo.vashi@enterprisedb.com> wrote: > Hi Dave, > > On Wed, Jun 30, 2021 at 3:35 PM Dave Page wrote: > >> Here's a simple patch for $SUBJECT. It will remove any leading spaces >> from the grid cell display and replace them with '[...] '. In edit mode, >> the data is unchanged. >> >> The patch looks good to me. > Thanks. > Do we need to specify somewhere what is [...] ? > I don't think there's much point. > > The only downside I can see to this is that the tooltip will show the >> modified data when it probably should actually show the un-modified data. >> Unfortunately that's handled by a SlickGrid plugin, which would have to be >> vendorised and modified (perhaps non-trivially). I don't think that's a big >> issue though. >> >> -- >> Dave Page >> Blog: https://pgsnake.blogspot.com >> Twitter: @pgsnake >> >> EDB: https://www.enterprisedb.com >> >> -- Dave Page Blog: https://pgsnake.blogspot.com Twitter: @pgsnake EDB: https://www.enterprisedb.com --00000000000057080d05c5fa9f8d Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hi

On Wed, Jun 30, 2021 at 12:36 PM Khushboo Vashi <= ;khushboo.vashi@enterpri= sedb.com> wrote:
Hi = Dave,

On Wed, Jun 30, 2021 at 3:35 PM Dave Page <dpage@pgadmin.org> wrote:
Here's a simple patch for $SUBJECT. It = will remove any leading spaces from the grid cell display and replace them = with '[...] '. In edit mode, the data is unchanged.

<= /div>
The patch looks good to me.=C2=A0
<= /blockquote>

Thanks.
=C2=A0
Do we need to specify = somewhere what is [...] ?

I don't think there's much point.
=C2=A0

The only downside I can see to thi= s is that the tooltip will show the modified data when it probably should a= ctually show the un-modified data. Unfortunately that's handled by a Sl= ickGrid plugin, which would have to be vendorised and modified (perhaps non= -trivially). I don't think that's a big issue though.

--
Dave Page
Bl= og: https://pgsn= ake.blogspot.com
Twitter: @pgsnake

EDB: https://www.enterprisedb.com
<= br>


--
<= /div> --00000000000057080d05c5fa9f8d--