Re: [pgAdmin][RM4348] Theme options in pgAdmin and dark theme

From: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
To: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
Subject: Re: [pgAdmin][RM4348] Theme options in pgAdmin and dark theme
Date: 2019-11-12 13:46:53
Message-ID: CAM9w-_=tKBBMaJmKSaRnhrdCWjk-D+SWd=BxwxYuHJwCv8nrmw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Hackers,

Attached is the patch to customize the scrollbar styling. A new SASS
variable added - $scrollbar-base-color is added to change the color as per
theme.
Please note, customization of scrollbar works only in webkit browsers like
Chrome, Safari. Mozilla does not have any way to customize.

Kindly review.

On Tue, Nov 12, 2019 at 5:55 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Thanks, patch applied.
>
> On Tue, Nov 12, 2019 at 3:41 PM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>>
>>
>> On Tue, Nov 12, 2019 at 3:25 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> On Tue, Nov 12, 2019 at 9:50 AM Aditya Toshniwal <
>>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>>
>>>>
>>>>> Attached. It may just be that it looks a little sharp because it's a
>>>>> long line. Maybe we should just make it a darker shade entirely.
>>>>>
>>>>> [image: Screenshot 2019-11-12 at 09.09.42.png]
>>>>>
>>>>
>>>> Why can't I see it :/
>>>> Maybe it got fixed along with the dark drop down for no limit. Could
>>>> you please check again.
>>>> [image: image.png]
>>>>
>>>
>>> Nope, it's definitely still there:
>>>
>> I checked on 2-3 systems. It looks fine. May be the permanent enabled
>> scrollbar causing it look like that.
>>
>>>
>>> [image: Screenshot 2019-11-12 at 09.53.02.png]
>>>
>>>
>>>
>>>> BTW, what was the reason for removing the styling from the scroll bars?
>>>>>
>>>> We had given gray shades to scrollbar which were not clearly visible
>>>> for some tables. Plus, for systems were scrollbar is not always enabled was
>>>> showing scrollbar. It should appear only on scroll on those systems.
>>>>
>>>
>>> And you're working on that? We can't keep the ugly grey scroll bars!
>>>
>> OK. I'll take it as priority.
>>
>>>
>>> Thanks!
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>
>>
>> --
>> Thanks and Regards,
>> Aditya Toshniwal
>> Sr. Software Engineer | EnterpriseDB India | Pune
>> "Don't Complain about Heat, Plant a TREE"
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>

--
Thanks and Regards,
Aditya Toshniwal
Sr. Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"

Attachment Content-Type Size
RM4348.part4.patch application/octet-stream 4.8 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2019-11-12 13:59:16 Re: [pgAdmin][RM4348] Theme options in pgAdmin and dark theme
Previous Message Shubham Agarwal 2019-11-12 12:57:22 Re: Fixes for pgAdmin feature tests