-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Icons not working properly on demo site #41
Comments
What browser? Matt Taylor On Thu, Jul 23, 2015 at 3:39 PM, Jared Weiss notifications@github.com
|
Chrome version 44.0.2403.107 |
I'm on Version 44.0.2403.107 (64-bit) and it looks great. Hey @brev I hate to drag you into this, but bootstrap glyphs aren't working on Jared's Chrome, but they are on mine. Do you know why this might happen? |
@rhyolight broken for me too, I'm seeing the below font+domain error in console:
Can't figure out why yours would be working OK, though. |
interestingly enough, the symbols show up nicely on http://data.numenta.org/index.html |
in the partial, header.tpl, why not use a relative path for scripts, CSS, and such? You can see that they are absolute paths to data.numenta.org. |
Oh, I thought this was a browser issue, but it's just a path issue.
Because relative paths must be in relation to the page being rendered (not the partial template), and the partial doesn't know what its path is going to be until runtime. |
so you have {{baseurl}} there in case it is not installed at the root of the web server? |
Yes. It could be installed as an express middleware, in which case "/" will Matt Taylor On Tue, Jul 28, 2015 at 3:42 PM, Jeff Fohl notifications@github.com wrote:
|
I can implement this middleware. Assign this to me if you would like me to handle it. |
No, there is no middleware necessary to fix this issue. I was just justifying why templates are given a |
see http://river-view.herokuapp.com/index.html
The small icons on the buttons (i.e. Details, JSON API, and Source Code) are showing up as boxes rather than the proper icons.
The text was updated successfully, but these errors were encountered: