Skip to content
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

Update report_template.html #175

Closed
wants to merge 1 commit into from
Closed

Conversation

svaens
Copy link

@svaens svaens commented Jul 25, 2014

"Show activity log" link causes a blank html page to open with the single word 'false'.
This has been mentioned in at least one website, for example,
http://projecthamster.wordpress.com/2010/11/08/report-love-something-for-freelancers/
and of course, by myself.

What should happen?

On click of the link, the "Activity Log" part of the report visibility should toggle (visible or not visible depending on current state).

"Show activity log" link causes a blank html page to open with the single word 'false'. 
This has been mentioned in at least one website, for example, 
http://projecthamster.wordpress.com/2010/11/08/report-love-something-for-freelancers/
and of course, by myself. 

What should happen? 

On click of the link, the "Activity Log" part of the report visibility should toggle (visible or not visible depending on current state).
@toupeira
Copy link
Contributor

toupeira commented Sep 5, 2014

Hmm this seems to work fine here in Firefox and Chrome, what browser are you using?
The "Show activity log" link is bound to a click handler here.

@toupeira toupeira added the bug label Sep 5, 2014
@svaens
Copy link
Author

svaens commented Sep 8, 2014

Strange; I re-tested again with the original template, and now it is working for me too. I can't think that anything else has changed on my environment since I had the problem.

I was testing on Firefox 26.0, on Linux Mint 16, 3.11.0-12-generic.

But, that I can't now reproduce, it is a bit embarrassing.
I will test again tonight, on my other machine, and confirm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants