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

Offline mode (closes #1181) #2312

Merged
merged 16 commits into from
Apr 8, 2024
Merged

Conversation

george-gca
Copy link
Collaborator

@george-gca george-gca commented Apr 2, 2024

Created a plugin to tackle #1181. Currently have an issue with tikzjax since it imports some wasm file from its javascript. The rest should work as expected.

@alshedivat
Copy link
Owner

@george-gca, looks like this PR adds multiple plugins / modifies a few. could you elaborate on all added modifications?

@george-gca
Copy link
Collaborator Author

It actually only adds one plug-in. The changes made to the others was just about indentation, to keep a standard. Some of them used 4 spaces, others 2.

@george-gca
Copy link
Collaborator Author

I also renamed one plugin file to keep it consistent with the others. And moved almost all urls from libraries to _config.yml to allow downloading them from the new plugin in _plugins/download-3rd-party.rb

@george-gca george-gca merged commit 08d562a into alshedivat:master Apr 8, 2024
3 checks passed
@george-gca george-gca deleted the offline_mode branch April 8, 2024 17:51
george-gca pushed a commit that referenced this pull request Apr 15, 2024
As discussed in #1181, I suggest to make embedding videos an optional
feature.

This behavior aligns well with recently merged PR #2312.

Open questions:

1. I added a youtube link to `papers.bib`. Is this link okay?
2. I set `enable_video_embedding: false` as the default. I argue that
privacy settings should be the default. Also, the current implementation
of `video.liquid` only works for some very specific video URLs. For
example, to embed youtube, specialized links must be used to avoid
`X-Frame-Option` issues. This behavior can lead to a broken embedding,
which would not look very nice.

Feedback welcome.
george-gca pushed a commit to george-gca/multi-language-al-folio that referenced this pull request Apr 15, 2024
As discussed in alshedivat#1181, I suggest to make embedding videos an optional
feature.

This behavior aligns well with recently merged PR alshedivat#2312.

Open questions:

1. I added a youtube link to `papers.bib`. Is this link okay?
2. I set `enable_video_embedding: false` as the default. I argue that
privacy settings should be the default. Also, the current implementation
of `video.liquid` only works for some very specific video URLs. For
example, to embed youtube, specialized links must be used to avoid
`X-Frame-Option` issues. This behavior can lead to a broken embedding,
which would not look very nice.

Feedback welcome.
BoAi01 pushed a commit to BoAi01/boai01.github.io that referenced this pull request May 7, 2024
Created a plugin to tackle alshedivat#1181. Currently have an issue with tikzjax
since it imports some wasm file from its javascript. The rest should
work as expected.

---------

Signed-off-by: George Araujo <[email protected]>
BoAi01 pushed a commit to BoAi01/boai01.github.io that referenced this pull request May 7, 2024
As discussed in alshedivat#1181, I suggest to make embedding videos an optional
feature.

This behavior aligns well with recently merged PR alshedivat#2312.

Open questions:

1. I added a youtube link to `papers.bib`. Is this link okay?
2. I set `enable_video_embedding: false` as the default. I argue that
privacy settings should be the default. Also, the current implementation
of `video.liquid` only works for some very specific video URLs. For
example, to embed youtube, specialized links must be used to avoid
`X-Frame-Option` issues. This behavior can lead to a broken embedding,
which would not look very nice.

Feedback welcome.
siril-teja pushed a commit to siril-teja/siril-teja.github.io-old that referenced this pull request Jun 19, 2024
Created a plugin to tackle alshedivat#1181. Currently have an issue with tikzjax
since it imports some wasm file from its javascript. The rest should
work as expected.

---------

Signed-off-by: George Araujo <[email protected]>
siril-teja pushed a commit to siril-teja/siril-teja.github.io-old that referenced this pull request Jun 19, 2024
As discussed in alshedivat#1181, I suggest to make embedding videos an optional
feature.

This behavior aligns well with recently merged PR alshedivat#2312.

Open questions:

1. I added a youtube link to `papers.bib`. Is this link okay?
2. I set `enable_video_embedding: false` as the default. I argue that
privacy settings should be the default. Also, the current implementation
of `video.liquid` only works for some very specific video URLs. For
example, to embed youtube, specialized links must be used to avoid
`X-Frame-Option` issues. This behavior can lead to a broken embedding,
which would not look very nice.

Feedback welcome.
karapostK pushed a commit to karapostK/karapostK.github.io that referenced this pull request Jul 4, 2024
Created a plugin to tackle alshedivat#1181. Currently have an issue with tikzjax
since it imports some wasm file from its javascript. The rest should
work as expected.

---------

Signed-off-by: George Araujo <[email protected]>
karapostK pushed a commit to karapostK/karapostK.github.io that referenced this pull request Jul 4, 2024
As discussed in alshedivat#1181, I suggest to make embedding videos an optional
feature.

This behavior aligns well with recently merged PR alshedivat#2312.

Open questions:

1. I added a youtube link to `papers.bib`. Is this link okay?
2. I set `enable_video_embedding: false` as the default. I argue that
privacy settings should be the default. Also, the current implementation
of `video.liquid` only works for some very specific video URLs. For
example, to embed youtube, specialized links must be used to avoid
`X-Frame-Option` issues. This behavior can lead to a broken embedding,
which would not look very nice.

Feedback welcome.
Suraj-Bhor pushed a commit to Suraj-Bhor/suraj-bhor.github.io that referenced this pull request Aug 13, 2024
Created a plugin to tackle alshedivat#1181. Currently have an issue with tikzjax
since it imports some wasm file from its javascript. The rest should
work as expected.

---------

Signed-off-by: George Araujo <[email protected]>
Suraj-Bhor pushed a commit to Suraj-Bhor/suraj-bhor.github.io that referenced this pull request Aug 13, 2024
As discussed in alshedivat#1181, I suggest to make embedding videos an optional
feature.

This behavior aligns well with recently merged PR alshedivat#2312.

Open questions:

1. I added a youtube link to `papers.bib`. Is this link okay?
2. I set `enable_video_embedding: false` as the default. I argue that
privacy settings should be the default. Also, the current implementation
of `video.liquid` only works for some very specific video URLs. For
example, to embed youtube, specialized links must be used to avoid
`X-Frame-Option` issues. This behavior can lead to a broken embedding,
which would not look very nice.

Feedback welcome.
meiqing-wang pushed a commit to meiqing-wang/meiqing-wang.github.io that referenced this pull request Oct 13, 2024
Created a plugin to tackle alshedivat#1181. Currently have an issue with tikzjax
since it imports some wasm file from its javascript. The rest should
work as expected.

---------

Signed-off-by: George Araujo <[email protected]>
meiqing-wang pushed a commit to meiqing-wang/meiqing-wang.github.io that referenced this pull request Oct 13, 2024
As discussed in alshedivat#1181, I suggest to make embedding videos an optional
feature.

This behavior aligns well with recently merged PR alshedivat#2312.

Open questions:

1. I added a youtube link to `papers.bib`. Is this link okay?
2. I set `enable_video_embedding: false` as the default. I argue that
privacy settings should be the default. Also, the current implementation
of `video.liquid` only works for some very specific video URLs. For
example, to embed youtube, specialized links must be used to avoid
`X-Frame-Option` issues. This behavior can lead to a broken embedding,
which would not look very nice.

Feedback welcome.
avishekanand pushed a commit to avishekanand/al-folio-homepage that referenced this pull request Oct 22, 2024
Created a plugin to tackle alshedivat#1181. Currently have an issue with tikzjax
since it imports some wasm file from its javascript. The rest should
work as expected.

---------

Signed-off-by: George Araujo <[email protected]>
avishekanand pushed a commit to avishekanand/al-folio-homepage that referenced this pull request Oct 22, 2024
As discussed in alshedivat#1181, I suggest to make embedding videos an optional
feature.

This behavior aligns well with recently merged PR alshedivat#2312.

Open questions:

1. I added a youtube link to `papers.bib`. Is this link okay?
2. I set `enable_video_embedding: false` as the default. I argue that
privacy settings should be the default. Also, the current implementation
of `video.liquid` only works for some very specific video URLs. For
example, to embed youtube, specialized links must be used to avoid
`X-Frame-Option` issues. This behavior can lead to a broken embedding,
which would not look very nice.

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

Successfully merging this pull request may close these issues.

2 participants