Hi On Tue, Feb 28, 2017 at 1:32 PM, Murtuza Zabuawala <murtuza.zabuaw...@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 -- Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgadmin-hackers