On Tue, Feb 28, 2017 at 8:27 PM, Dave Page <dpage@pgadmin.org> wrote:
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)?