Re: [pgadmin-hackers] [PATCH][pgAdmin4] Move Bootstrap fonts to'vendor/fonts' folder - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [pgadmin-hackers] [PATCH][pgAdmin4] Move Bootstrap fonts to'vendor/fonts' folder
Date
Msg-id CA+OCxozwQThn0TxiO3_LwkEc1vt-_7juW6EYB5qHgpLtXfxXXA@mail.gmail.com
Whole thread Raw
In response to [pgadmin-hackers] [PATCH][pgAdmin4] Move Bootstrap fonts to 'vendor/fonts' folder  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
Responses Re: [pgadmin-hackers] [PATCH][pgAdmin4] Move Bootstrap fonts to'vendor/fonts' folder
List pgadmin-hackers
Hi

On Tue, Feb 28, 2017 at 1:32 PM, Murtuza Zabuawala
<murtuza.zabuawala@enterprisedb.com> wrote:
> Hi,
>
> PFA patch to move bootstrap fonts to 'vender/fonts' folder as bootstrap.css
> looking for them as
>
>
> murtuza@vm:~/projects/pgadmin4$ grep -nr 'glyphicons'
> *web/pgadmin/static/vendor/bootstrap/bootstrap.css:270:  src:
> url('../fonts/glyphicons-halflings-regular.eot');
>
> web/pgadmin/static/vendor/bootstrap/bootstrap.css:271:  src:
> url('../fonts/glyphicons-halflings-regular.eot?#iefix')
> format('embedded-opentype'),
> url('../fonts/glyphicons-halflings-regular.woff2') format('woff2'),
> url('../fonts/glyphicons-halflings-regular.woff') format('woff'),
> url('../fonts/glyphicons-halflings-regular.ttf') format('truetype'),
> url('../fonts/glyphicons-halflings-regular.svg#glyphicons_halflingsregular')
> format('svg');

The patch just seems to remove the files altogether. I assume the
intent was to put them in web/pgadmin/static/vendor/fonts? I'm not
sure that's the best idea, as it would put them outside the bootstrap
directory. Perhaps we should update Bootstrap, and un-flatten it (i.e.
leave it in the distributed directory structure)?

> Also removed console.log from pgAgent node JS.

I've committed that bit, thanks.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: [pgadmin-hackers] pgAdmin 4 commit: Remove a debug message.
Next
From: Dave Page
Date:
Subject: [pgadmin-hackers] pgAdmin 4 commit: Improve visibility of syntax highlighting colours.Fi