Skip to content

Conversation

midzer
Copy link
Contributor

@midzer midzer commented Feb 14, 2016

Idee war, Javascript spezifischen Kram dort zu laden wo er benötigt wird:

  • main.js inline bei /firmware
  • das Video unter /wasistfreifunk wird mit Projekktor geladen. Leider muss jquery hier nochmal inkludiert werden, weil sonst die entsprechende Scriptpassage nicht interpretiert werden kann
  • podlove css/js sollte von /podcast geladen werden (ungetestet)

<script type="text/javascript" src="{{ "/js/projekktor-1.3.09.min.js" | prepend: site.baseurl}}"></script>

<!-- podlove stuff -->
<link rel="alternate" type="application/rss+xml" title="mp3 Podcast Feed" href="https://ffmuc.net/podcast/mp3.xml" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Wieso hast du die Podcast Feed Metadaten entfernt?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Im Quellcode von https://ffmuc.net/podcast sehe ich, dass dort ebenso das ganze podlove zeug eingebunden wird. Dachte das wär somit redundant.

Wird die .xml irgendwo auf der Hauptseite gebraucht?

Copy link
Contributor

Choose a reason for hiding this comment

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

Ja, damit RSS-Reader und Podcast-Player bei Angabe der Hauptseite direkt den Feed zum Podcast finden.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Verstehe.
Also muss <link rel="alternate" type="application/rss+xml" title="mp3 Podcast Feed" href="https://ffmuc.net/podcast/mp3.xml" /> wieder rein

@dl1com dl1com merged commit c986600 into freifunkMUC:master Apr 21, 2016
@midzer midzer deleted the patch1 branch April 21, 2016 19:19
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.

3 participants