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

Jump to bottom

When embedding full Piwik in another application via iframe, "View detailed visitor log" link does not work #10079

Closed
mattab opened this issue Apr 20, 2016 · 2 comments
Closed

When embedding full Piwik in another application via iframe, "View detailed visitor log" link does not work #10079

mattab opened this issue Apr 20, 2016 · 2 comments
Assignees
@diosmosis
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone
3.9.0

Comments

@mattab
Copy link
Member

mattab commented Apr 20, 2016

Reproduce

  • Embed Piwik Dashboard via iframe
  • In widget "Real time visitors" click on the footer link "View detailed visitor log"
  • Got: the page becomes white (see screenshot)
  • Expected instead: the visitor log should load and replace the dashboard.

Screenshot

empty

Reproduce on demo: https://demo.piwik.org/index.php?module=Widgetize&action=iframe&moduleToWidgetize=Dashboard&actionToWidgetize=index&idSite=1&period=week&date=yesterday

Reported in forums: https://forum.piwik.org/t/piwik-and-iframe/18726/7
@mattab mattab added the Bug For errors / faults / flaws / inconsistencies etc. label Apr 20, 2016
@mattab mattab added this to the 2.16.x (LTS) milestone Apr 20, 2016
@sgiehl
Copy link
Member

sgiehl commented Apr 25, 2016

Wouldn't it be a cleaner solution to remove those links in Widgetize mode?
If someone widgetizes the dashboard or any other site, I don't think the intention is to let the user switch between the pages. If we would make those links working there wouldn't be a way back to the previous "page".

@mattab mattab modified the milestones: Mid term, 2.16.x (LTS) May 27, 2016
@mattab
Copy link
Member Author

mattab commented Sep 25, 2018

sounds good to remove those links in widgetized mode 👍

@mattab mattab modified the milestones: Backlog (Help wanted), 3.8.0 Sep 25, 2018
@diosmosis diosmosis self-assigned this Feb 5, 2019
@diosmosis diosmosis mentioned this issue Feb 6, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@diosmosis diosmosis

Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Milestone
3.9.0
Development

No branches or pull requests
3 participants
@diosmosis @mattab @sgiehl