MIME-Version: 1.0
In-Reply-To: 
 <CAM5-9D__jp-b=M72NDNzN-405BTtAp99j_4Hhv268a_bzYayHg@mail.gmail.com>
References: 
 <CACrUwh+GDxb0mMtJkF-EoNu6L8hzAgN4qUp5fEn_sdWH9i3KeQ@mail.gmail.com>
 <CA+OCxoypX_QdJjhmkAiJtg34oOP7GAwb58WQYQX0O-h_0uT8MA@mail.gmail.com>
 <CAM5-9D8sSJL3mFV65MnwTtgdW=ySUROgp01G+=oHeeTUxPpa8g@mail.gmail.com>
 <CA+OCxozQSbXRsjRqVxRQ8Bm_PYSv2_YOpcw7EwzBLVdmVOtvLw@mail.gmail.com>
 <CAM5-9D97E8_H1_9OHmVCvP8KBPCwwXMZN_CmUHuPgf72m_ju1Q@mail.gmail.com>
 <CAAtBm9UjhRP9Un=KsjMr11fELMBphQ9u9jDa7u9hj=J8ORDmBQ@mail.gmail.com>
 <CAM5-9D__jp-b=M72NDNzN-405BTtAp99j_4Hhv268a_bzYayHg@mail.gmail.com>
From: Dave Page <dpage@pgadmin.org>
Date: Wed, 7 Jun 2017 16:31:57 +0100
Message-ID: 
 <CA+OCxox024g-VkUTGetuZkFa-yT8pt_mxqUtkZcg0fNABqLTWg@mail.gmail.com>
Subject: Re: [pgAdmin4][PATCH] Consolidating gray colors in
 the application
To: Surinder Kumar <surinder.kumar@enterprisedb.com>
Cc: Robert Eckhardt <reckhardt@pivotal.io>, Shruti B Iyer <siyer@pivotal.io>,
	pgadmin-hackers <pgadmin-hackers@postgresql.org>,
 Matthew Kleiman <mkleiman@pivotal.io>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk
Sender: pgadmin-hackers-owner@postgresql.org

Thanks, applied.

On Wed, Jun 7, 2017 at 3:48 PM, Surinder Kumar
<surinder.kumar@enterprisedb.com> wrote:
> Hi
>
> I have used classes 'bg-gray-2 border-gray-3' for styling header and footer
> of properties panel which are also used in other places.
>
> Please find attached patch.
>
>
> On Wed, Jun 7, 2017 at 6:59 PM, Robert Eckhardt <reckhardt@pivotal.io>
> wrote:
>>
>> Thank you.
>>
>> -- Rob
>>
>> On Wed, Jun 7, 2017 at 9:28 AM, Surinder Kumar
>> <surinder.kumar@enterprisedb.com> wrote:
>>>
>>> Sure I will send.
>>>
>>> On Wed, Jun 7, 2017 at 6:54 PM, Dave Page <dpage@pgadmin.org> wrote:
>>>>
>>>> On Wed, Jun 7, 2017 at 2:22 PM, Surinder Kumar
>>>> <surinder.kumar@enterprisedb.com> wrote:
>>>> > Hi
>>>> >
>>>> > With the latest pull, I found:
>>>> > The header color of Properties panel disappears and background color
>>>> > of
>>>> > footer area in Edit Properties panel is changed.
>>>>
>>>> Oh, I missed that :-(. Can you come up with a quick fix using the new
>>>> classes?
>>>>
>>>> --
>>>> 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


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