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

[Cal-5] Added the calendar-modal #318

Closed
wants to merge 2 commits into from

Conversation

Marius1501
Copy link
Contributor

@Marius1501 Marius1501 commented Nov 28, 2022

Closes #289

grafik

I added the modal for the calendar when clicking on the calendar icon. I used an iframe to display the TUMOnline website but the URL is not found. I don't know what's the problem because it is the same URL which is used to open the website.

</div>
<div class="modal-body">
<div class="content" id="modalCalendarContent">
<iframe src="view_data.props.calendar_url"></iframe>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<iframe src="view_data.props.calendar_url"></iframe>
<iframe v-bind:src="view_data.props.calendar_url"></iframe>

I think you need to use v-bind here

@Marius1501
Copy link
Contributor Author

@octycs Thank you very much for the hint! Now is the problem that the iframe hasn't the permission to show the website. 😞
grafik

@CommanderStorm
Copy link
Member

image
Hmm…
Yea... It looks like the easy path via an iframe is not an option here…

I have an API for the TUMonline calendar and will ask them if I can share it openly.

@CommanderStorm
Copy link
Member

blocked until #324 is done and merged

@CommanderStorm CommanderStorm added frontend Related to the frontend blocked-by-upstream labels Dec 9, 2022
@CommanderStorm CommanderStorm changed the title Added the calendar-modal [Cal-5] Added the calendar-modal Jan 2, 2023
@CommanderStorm
Copy link
Member

CommanderStorm commented Jan 11, 2023

@Marius1501 I am closing this issue, as I don't see a way we can get this approach working.
I think a different approach is more prommissing (see #324).

Any approaches in this way should be based on #200 and build as a component though, as that will be merge-ready soon and the calendar is a thing I think would be easier to develop+test this way.

@Marius1501 Marius1501 deleted the calendar_modal branch January 11, 2023 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked-by-upstream frontend Related to the frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Better Calendar support
3 participants