Re: Require suggestions for Issue #2874

public inbox for [email protected]  
help / color / mirror / Atom feed

From: Anil Sahoo <[email protected]>
To: Dave Page <[email protected]>
Cc: Rohit Bhati <[email protected]>
Cc: [email protected]
Subject: Re: Require suggestions for Issue #2874
Date: Mon, 9 Sep 2024 16:06:09 +0530
Message-ID: <CAO+oWtCLRQFJ4nUM7Q4GF4g_HGBrMvWbbT8A_GuMe=gvbcHvfg@mail.gmail.com> (raw)
In-Reply-To: <CA+OCxowY_26iwaOrXE2wmuEOA_cSwNB=0PzVhTYS1hDgGoEXDw@mail.gmail.com>
References: <CABscKVgnoB8=KcYSa0tp8LAwdw4Gq4-G5ywcmWz353jtH8RH1A@mail.gmail.com>
	<CABscKVg+1gh1CjPPX8k4_9A407hzfd8DTbP7e0U+-jcoO7awQg@mail.gmail.com>
	<CA+OCxoxiy6xc9V9qUOP0f5LSA22PwHyNFWXjMvBT10EHjF5F+g@mail.gmail.com>
	<CABscKViG4=vECzrEhQNKbwGKJwPrWDn3isw-mPA5fNphw-G6=Q@mail.gmail.com>
	<CA+OCxoxOzUKjPF8pC=z4vjVKegBzNUDtGtwc5tewmtz2MfMhMw@mail.gmail.com>
	<CABscKViS50079irxJfgL5ARtd5g4D86z+OwJ7huY7nRJbB2rLQ@mail.gmail.com>
	<CABscKVjEzXqeYu4LcHhAn1uhO5o3_-ALe1_bsK9UicOcEHq47A@mail.gmail.com>
	<CA+OCxoy5Mz53vyebx+2V0U3KH9f+Pd-p-HWoM7dMnUYNrjP-nw@mail.gmail.com>
	<CAFOhELcYWCYdQ9jF6sV=PkZJ-hkF7Pxt-a3Gi61JzFx=AoGb6w@mail.gmail.com>
	<CABscKViGC-C6dBx6c=M3TQvCXrM+KGXWED7cj9BX2uB7qNb0Tg@mail.gmail.com>
	<CA+OCxoxc1uu5yddUc8SOrkj8-O5ZY4HUtJCjqXt2H292ema1nQ@mail.gmail.com>
	<CABscKVihMj687AzJh3mWt6Dt+EiO=qanhp3bQUtQmqqv3Hg07g@mail.gmail.com>
	<CA+OCxoxNJuNuqhcOpzQUdDFTsirGpZKHGp0ooc0LxpkrCXU21Q@mail.gmail.com>
	<CABscKVhYEkFCuF+MT0XBumbVn9NsNseBLjKS=mxiMenv1sRxwA@mail.gmail.com>
	<CA+OCxowY_26iwaOrXE2wmuEOA_cSwNB=0PzVhTYS1hDgGoEXDw@mail.gmail.com>

Hi Rohit/Dave,

If we can remove the alternate background color from the row number column
and give light grey or different color to the row number column and the
column header, that will make the data distinct with the alternate
background color from the headers and row number column.

Any thoughts on this?

Thanks
Anil
--

<http://www.enterprisedb.com;

*Anil Sahoo*

Software Engineer

www.enterprisedb.com

Power to Postgres

<https://www.linkedin.com/company/edbpostgres;
<https://twitter.com/edbpostgres?lang=en;
<https://www.facebook.com/EDBpostgres;
<https://www.instagram.com/EDBpostgres/;


On Mon, Sep 9, 2024 at 3:30 PM Dave Page <[email protected]> wrote:

> Hi
>
> On Mon, 9 Sept 2024 at 10:18, Rohit Bhati <[email protected]>
> wrote:
>
>> Hi Dave,
>>
>> I have made changes as per your suggestions.
>> I have attached screenshots showing how the updates look across all
>> themes for your reference.
>>
>> If the current changes does not fit well with the theme or if you have
>> any suggestions for improvements, please let me know.
>>
>
> The colours etc. look correct now (ie. there are no weird borders), but I
> still don't think the row number looks visually distinct from the rest of
> the data. The bold text simply isn't very bold - I think it still needs
> more weight.
>
>
>
>>
>> Thanks
>> Rohit Bhati
>> EnterpriseDB
>>
>>
>> On Fri, Sep 6, 2024 at 2:01 PM Dave Page <[email protected]> wrote:
>>
>>>
>>>
>>> On Fri, 6 Sept 2024 at 09:25, Rohit Bhati <[email protected]>
>>> wrote:
>>>
>>>> I have made some more changes with changing background color.
>>>>
>>>
>>> Yeah, that looks weird again, specifically on the selected rows. I
>>> believe the issue is that we should not highlight the row number column
>>> when a row is selected at all. That makes sense, as we should only be
>>> highlighting the columns that would get copied in a copy/paste operation
>>> anyway.
>>>
>>> Also, the bold still doesn't look bold. I think the weight needs to be
>>> increased.
>>>
>>>
>>>>
>>>> I have attached screenshots showing how the updates look across all
>>>> themes.
>>>>
>>>> On Fri, Sep 6, 2024 at 1:51 PM Dave Page <[email protected]> wrote:
>>>>
>>>>> Hi
>>>>>
>>>>> On Fri, 6 Sept 2024 at 09:19, Rohit Bhati <
>>>>> [email protected]> wrote:
>>>>>
>>>>>> Hi,
>>>>>>
>>>>>> I have removed the border and background color, just made the row
>>>>>> numbers bold.
>>>>>>
>>>>>> I have attached screenshots showing how the updates look across all
>>>>>> themes.
>>>>>>
>>>>>
>>>>> That doesn't look bold at all to me - and I think we still need some
>>>>> other visual distinction, such as the column background colour.
>>>>>
>>>>>
>>>>>>
>>>>>>
>>>>>> On Fri, Sep 6, 2024 at 12:35 PM Khushboo Vashi <
>>>>>> [email protected]> wrote:
>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> On Thu, Sep 5, 2024 at 7:11 PM Dave Page <[email protected]> wrote:
>>>>>>>
>>>>>>>> Hi
>>>>>>>>
>>>>>>>> On Thu, 5 Sept 2024 at 14:38, Rohit Bhati <
>>>>>>>> [email protected]> wrote:
>>>>>>>>
>>>>>>>>> Hi,
>>>>>>>>>
>>>>>>>>> I have made changes for issue #7863
>>>>>>>>> <https://github.com/pgadmin-org/pgadmin4/issues/7863; based on
>>>>>>>>> theme requirements.
>>>>>>>>>
>>>>>>>>> I have attached screenshots showing how the updates look across
>>>>>>>>> all themes for your reference.
>>>>>>>>>
>>>>>>>>> If the current color does not fit well with the theme or if you
>>>>>>>>> have any suggestions for improvements, please let me know.
>>>>>>>>>
>>>>>>>>
>>>>>>>> Is the colour contrast high enough on the row numbers for selected
>>>>>>>> rows in the high contrast theme? The black on grey seems like it might not
>>>>>>>> be to me, but I haven't tested.
>>>>>>>>
>>>>>>>> I wonder also if we should make the row numbers bold all the time.
>>>>>>>> Thoughts?
>>>>>>>>
>>>>>>> We can check how it looks. Also the cell border doesn't look good to
>>>>>>> me.
>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> --
>>>>>>>> Dave Page
>>>>>>>> pgAdmin: https://www.pgadmin.org
>>>>>>>> PostgreSQL: https://www.postgresql.org
>>>>>>>> EDB: https://www.enterprisedb.com
>>>>>>>>
>>>>>>>> PGDay UK 2024, 11th September, London: https://2024.pgday.uk/
>>>>>>>>
>>>>>>>>
>>>>>
>>>>> --
>>>>> Dave Page
>>>>> pgAdmin: https://www.pgadmin.org
>>>>> PostgreSQL: https://www.postgresql.org
>>>>> EDB: https://www.enterprisedb.com
>>>>>
>>>>> PGDay UK 2024, 11th September, London: https://2024.pgday.uk/
>>>>>
>>>>>
>>>
>>> --
>>> Dave Page
>>> pgAdmin: https://www.pgadmin.org
>>> PostgreSQL: https://www.postgresql.org
>>> EDB: https://www.enterprisedb.com
>>>
>>> PGDay UK 2024, 11th September, London: https://2024.pgday.uk/
>>>
>>>
>
> --
> Dave Page
> pgAdmin: https://www.pgadmin.org
> PostgreSQL: https://www.postgresql.org
> EDB: https://www.enterprisedb.com
>
> PGDay UK 2024, 11th September, London: https://2024.pgday.uk/
>
>

view thread (33+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Require suggestions for Issue #2874
  In-Reply-To: <CAO+oWtCLRQFJ4nUM7Q4GF4g_HGBrMvWbbT8A_GuMe=gvbcHvfg@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox