[MGNLTAGS-38] Cannot open http://www.google.com or Google Analytics dashboard in dashboard subapp, though other URLs work Created: 09/Jun/14 Updated: 12/Jun/14 Resolved: 11/Jun/14 |
|
| Status: | Closed |
| Project: | Marketing Tags Manager |
| Component/s: | None |
| Affects Version/s: | 1.0 |
| Fix Version/s: | 1.0 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Cheng Hu | Assignee: | Peili Liang |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| Template: |
|
| Acceptance criteria: |
Empty
|
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
| Date of First Response: |
| Description |
|
| Comments |
| Comment by Peili Liang [ 10/Jun/14 ] |
|
When I open the console of developer tools in chrome browser, I found an error info: Maybe we should open the dashboard in a new browser tab to prohibit the similar issue. but it need to investigate how to open the dashboard in a new browser by vaadin. |
| Comment by Cheng Hu [ 10/Jun/14 ] |
|
From the experience of others on the web, it seems like there may be no way to embed the Google Analytics dashboard in a subapp iFrame, since it directly forbids it in the header. From work on a previous issue, there is also no straightforward way to open a new browser window/tab with the URL without it being considered a pop-up, because the Action buttons in the Action bar are automatically generated on the client side where there is no straightforward way to assign JavaScript click listeners to them. So the only option may be not to have the dashboard concept at all. |
| Comment by Andreas Weder [ 11/Jun/14 ] |
|
Nice suggestion. Or rather: nice addition, actually, since this also adds value for cases where a dashboard properly displays inside an iframe. I have three requests still:
|
| Comment by Christoph Meier [ 11/Jun/14 ] |
|
review: instead of |