Hi Dave,

Please find patch to fix the font on alertify header.
Please review.

--
Regards,
Murtuza Zabuawala

On Fri, Sep 15, 2017 at 7:10 PM, Murtuza Zabuawala <
murtuza.zabuaw...@enterprisedb.com> wrote:

> Hi Atul,
>
> I think we still missed alertify header fonts :)
> Check screenshot.
>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
> [image: https://community.postgresrocks.net/]
> <https://community.postgresrocks.net/>
>
> On Fri, Sep 15, 2017 at 6:38 PM, Dave Page <dp...@pgadmin.org> wrote:
>
>> Thanks, applied.
>>
>> On Fri, Sep 15, 2017 at 8:59 AM, Atul Sharma <
>> atul.sha...@enterprisedb.com> wrote:
>>
>>> Hi,
>>>
>>> PFA patch which replaces the "Roboto" font family with "Open Sans" for
>>> alertify dialogs. Regarding Query tool context menu, It seems different
>>> issue than the applied fonts.
>>>
>>> Regards,
>>> Atul
>>>
>>> On Thu, Sep 14, 2017 at 7:49 PM, Murtuza Zabuawala <
>>> murtuza.zabuaw...@enterprisedb.com> wrote:
>>>
>>>> Attaching more screenshots.
>>>>
>>>> --
>>>> Regards,
>>>> Murtuza Zabuawala
>>>> EnterpriseDB: http://www.enterprisedb.com
>>>> The Enterprise PostgreSQL Company
>>>>
>>>> [image: https://community.postgresrocks.net/]
>>>> <https://community.postgresrocks.net/>
>>>>
>>>> On Thu, Sep 14, 2017 at 7:08 PM, Murtuza Zabuawala <
>>>> murtuza.zabuaw...@enterprisedb.com> wrote:
>>>>
>>>>> Hi Atul,
>>>>>
>>>>> Please also check fonts in various the alertify dialogs.
>>>>>
>>>>> --
>>>>> Regards,
>>>>> Murtuza Zabuawala
>>>>> EnterpriseDB: http://www.enterprisedb.com
>>>>> The Enterprise PostgreSQL Company
>>>>>
>>>>> [image: https://community.postgresrocks.net/]
>>>>> <https://community.postgresrocks.net/>
>>>>>
>>>>> On Thu, Sep 14, 2017 at 7:03 PM, Dave Page <dp...@pgadmin.org> wrote:
>>>>>
>>>>>> Nope - it shouldn't look like that. Atul, Can you take a look at it
>>>>>> please?
>>>>>>
>>>>>> On Thu, Sep 14, 2017 at 2:26 PM, Murtuza Zabuawala <
>>>>>> murtuza.zabuaw...@enterprisedb.com> wrote:
>>>>>>
>>>>>>> Hi Dave,
>>>>>>>
>>>>>>> After latest pull fonts from context menu looks different, PFA
>>>>>>> screenshot.
>>>>>>> I tried on macOS: 10.10 (Web mode) on Google Chrome: Version
>>>>>>> 60.0.3112.113 (Official Build) (64-bit)
>>>>>>>
>>>>>>> Is this expected behaviour?
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Regards,
>>>>>>> Murtuza Zabuawala
>>>>>>> EnterpriseDB: http://www.enterprisedb.com
>>>>>>> The Enterprise PostgreSQL Company
>>>>>>>
>>>>>>> [image: https://community.postgresrocks.net/]
>>>>>>> <https://community.postgresrocks.net/>
>>>>>>>
>>>>>>> On Wed, Sep 13, 2017 at 8:31 PM, Dave Page <dp...@pgadmin.org>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> Bundle fonts with pgAdmin so we have a standard set for all
>>>>>>>> platforms.
>>>>>>>>
>>>>>>>> Research by Shirley at Pivotal.
>>>>>>>>
>>>>>>>> Branch
>>>>>>>> ------
>>>>>>>> master
>>>>>>>>
>>>>>>>> Details
>>>>>>>> -------
>>>>>>>> https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdif
>>>>>>>> f;h=788f7f3a69500c37008dbba459d3ea00b0b1bff8
>>>>>>>> Author: Atul Sharma <atul.sha...@enterprisedb.com>
>>>>>>>>
>>>>>>>> Modified Files
>>>>>>>> --------------
>>>>>>>> libraries.txt                                      |   4 ++-
>>>>>>>> web/pgadmin/browser/static/css/browser.css         |  33
>>>>>>>> ++++++++++++++-------
>>>>>>>> .../misc/bgprocess/static/css/bgprocess.css        |   2 +-
>>>>>>>> .../misc/file_manager/static/css/file_manager.css  |   2 +-
>>>>>>>> web/pgadmin/static/css/aci_tree.overrides.css      |   4 +--
>>>>>>>> web/pgadmin/static/css/bootstrap.overrides.css     |   5 ++--
>>>>>>>> web/pgadmin/static/css/codemirror.overrides.css    |   4 +--
>>>>>>>> web/pgadmin/static/css/pgadmin.css                 |  17
>>>>>>>> ++++++++++-
>>>>>>>> web/pgadmin/static/css/pgadmin.style.css           |  10 +++----
>>>>>>>> web/pgadmin/static/css/webcabin.overrides.css      |   6 ++--
>>>>>>>> web/pgadmin/static/fonts/OpenSans-Bold.ttf         | Bin 0 ->
>>>>>>>> 224452 bytes
>>>>>>>> web/pgadmin/static/fonts/OpenSans-Italic.ttf       | Bin 0 ->
>>>>>>>> 212760 bytes
>>>>>>>> web/pgadmin/static/fonts/OpenSans-Regular.ttf      | Bin 0 ->
>>>>>>>> 217276 bytes
>>>>>>>> web/pgadmin/static/fonts/OpenSans-SemiBold.ttf     | Bin 0 ->
>>>>>>>> 221164 bytes
>>>>>>>> web/pgadmin/static/fonts/SourceCodePro-Bold.ttf    | Bin 0 ->
>>>>>>>> 120504 bytes
>>>>>>>> web/pgadmin/static/fonts/SourceCodePro-Regular.ttf | Bin 0 ->
>>>>>>>> 120548 bytes
>>>>>>>> web/pgadmin/static/scss/_alert.scss                |   2 +-
>>>>>>>> web/pgadmin/static/scss/_alertify.overrides.scss   |   2 +-
>>>>>>>> web/pgadmin/static/scss/_typography.scss           |   2 +-
>>>>>>>> web/pgadmin/static/scss/sqleditor/_history.scss    |   8 ++---
>>>>>>>> web/pgadmin/tools/debugger/static/css/debugger.css |   6 ++--
>>>>>>>> .../tools/sqleditor/static/css/sqleditor.css       |   8 ++---
>>>>>>>> 22 files changed, 73 insertions(+), 42 deletions(-)
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> 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
>>
>
>
diff --git a/web/pgadmin/static/css/pgadmin.style.css 
b/web/pgadmin/static/css/pgadmin.style.css
index 3da1a18..d8bda69 100644
--- a/web/pgadmin/static/css/pgadmin.style.css
+++ b/web/pgadmin/static/css/pgadmin.style.css
@@ -105,6 +105,6 @@
 .opacity-5 {
   opacity: 0.5;  }
 
+.alertify .ajs-header,
 .alertify .ajs-body {
-  font-family: 'Open Sans';
-}
+  font-family: 'Open Sans'; }

Reply via email to