-
Notifications
You must be signed in to change notification settings - Fork 20
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
Gallery error on Akvo Pages #473
Comments
@adriancollier The gallery on Pages seems to work fine for me in Chrome, Firefox and Safari. Any clue as to when this would occur? |
I cannot replicate this either... @CharlotteSoed @lauraroverts @Lissyvn - do any of you know who originated this report? I cannot find any trace of it. |
no idea |
no idea either... |
nope! |
Still cannot replicate directly, but I have received the following information from Winona:
|
Still can't replicate the issue either, but one website suggested loading the CSS in the head (as mentioned as an optimisation here; http://galleria.io/docs/references/optimize/#include-your-theme-in-the-head), so will try that and see if the problem persists. |
If the galleria files are loaded in the head, shouldn't they be removed from somewhere too? |
Hm, tried removing it on the page itself, but then it always breaks, so that doesn't work :(.. And also still can't reproduce the error on my laptop, but Anna-Marthe has the same problem for one project on hers. Maybe we can try to merge the current PR and see if it works on her laptop? |
Sure, sounds good. |
@zzgvh I cannot replicate this on any browser of mine...which machine are you seeing this on? |
Locally, when running the feature/473_gallery_error branch. |
OK, so as decided this issue should not be included within the current milestone as we are not sure if the fix is needed or not. Let's pick this up in the next release https://github.com/akvo/akvo-rsr/issues?milestone=27&state=open |
Had to update the location of the CSS and JS files after the Django upgrade, that's probably why it didn't work last time.. Seems to work now, even though, as before, I still can't replicate the error 😬 |
[#473] Included Galleria theme in the head
Merged in #560 |
Test planHard to test. I can't replicate the behaviour since it seems to differ per computer. |
I've seen this before, but I thought it was resolved. Apparently the error only happens when using Chrome, but the Gallery is not available for some users on Pages:
The text was updated successfully, but these errors were encountered: