Consolidate encrypted file name docs and add video walkthrough#138
Consolidate encrypted file name docs and add video walkthrough#138
Conversation
mindmonk
commented
Mar 30, 2026
mindmonk
commented
- Moved the Locate Encrypted File section from accessing-vaults.md into encrypted-file-names.md, where it fits thematically
- Added a Video Walkthrough section at the end of encrypted-file-names.md demonstrating both features in sequence
- Replaced the full content in accessing-vaults.md#locate-encrypted-file with a short redirect link to the new location (preserving the anchor for any existing deep links)
WalkthroughThis PR relocates documentation content for the "Locate Encrypted File" feature. The full description, step-by-step instructions, drag-and-drop guidance, and embedded video are removed from Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/desktop/encrypted-file-names.md`:
- Line 48: Update the phrase "third party app" in the Locate Encrypted File
feature description to "third-party app" to use the correct compound adjective;
locate the string in the docs line containing "Locate Encrypted File feature" or
the exact text "then restore an older version of the encrypted file with the
third party app" and replace it with the hyphenated form.
- Around line 87-89: The <video> block uses a .mov source but declares
type="video/mp4"; convert the file /vid/file-names.mov to an MP4 (H.264/AAC) and
update the <source> element to point to /vid/file-names.mp4 while keeping
type="video/mp4" so browsers (Chrome/Edge) correctly detect the MIME type;
locate the <video> tag and its <source> element in the doc and replace only the
file extension/URI to .mp4.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: c8f01b89-745a-4564-8fb0-d1a6cbfa05a9
⛔ Files ignored due to path filters (1)
static/vid/file-names.movis excluded by!**/*.mov
📒 Files selected for processing (2)
docs/desktop/accessing-vaults.mddocs/desktop/encrypted-file-names.md
|
|
||
| ## Locate Encrypted File {#locate-encrypted-file} | ||
|
|
||
| The Locate Encrypted File feature helps users find the encrypted version of a specific file. This feature is particularly useful when vault files are versioned and the user wants to restore an older version of a file. As Cryptomator encrypts filenames and obfuscates directory structures, users first locate the encrypted file and then restore an older version of the encrypted file with the third party app. |
There was a problem hiding this comment.
Hyphenate “third-party” for correct compound adjective usage.
Please change “third party app” to “third-party app” for grammar consistency.
🧰 Tools
🪛 LanguageTool
[grammar] ~48-~48: Use a hyphen to join words.
Context: ...ion of the encrypted file with the third party app. 1. Unlock the desired vault....
(QB_NEW_EN_HYPHEN)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/desktop/encrypted-file-names.md` at line 48, Update the phrase "third
party app" in the Locate Encrypted File feature description to "third-party app"
to use the correct compound adjective; locate the string in the docs line
containing "Locate Encrypted File feature" or the exact text "then restore an
older version of the encrypted file with the third party app" and replace it
with the hyphenated form.
| <video controls autoplay loop muted style={{width: '100%', maxWidth: '1280px', height: 'auto'}}> | ||
| <source src="/vid/file-names.mov" type="video/mp4" /> | ||
| Your browser does not support the video tag. |
There was a problem hiding this comment.
🧩 Analysis chain
🌐 Web query:
For an HTML5 <video><source>, what MIME type should be used for a .mov file, and what are browser compatibility implications versus .mp4 (H.264/AAC)?
💡 Result:
The standard MIME type for a .mov file (QuickTime container) is video/quicktime. However, video/quicktime has very limited browser support in HTML5
Citations:
- 1: https://developer.mozilla.org/en-US/docs/Web/HTML/Supported_media_formats
- 2: https://stackoverflow.com/questions/57060193/browser-support-for-mov-video
- 3: https://stackoverflow.com/questions/31380695/how-to-open-mov-format-video-in-html-video-tag
- 4: https://caniuse.com/mp4
- 5: https://www.brightcove.com/blog/what-formats-do-i-need-for-html5-video
- 6: http://developer.mozilla.org/en-US/docs/Web/Media/Guides/Formats/Containers
- 7: https://developer.apple.com/documentation/quicktime-file-format/quicktime_movie_files
- 8: https://developer.mozilla.org/en-US/docs/Web/Media/Formats/Video_codecs
- 9: https://www.iana.org/assignments/media-types/video/quicktime
- 10: Don't use
<source type=video/quicktime>for .mov files. GoogleChrome/webdev-infra#8
🏁 Script executed:
#!/bin/bash
# Search for the video file reference in the docs directory
echo "=== Searching for video file references ==="
rg "file-names\.mov" docs/ 2>/dev/null || echo "No direct .mov references found"
# Check if assets directory exists and what's in vid directory
echo -e "\n=== Checking for vid directory ==="
find . -type d -name "vid" 2>/dev/null | head -20
# Search for any video file references in the repository
echo -e "\n=== Searching for any video file references ==="
rg "\.mov|\.mp4" docs/ --type md | head -20
# Check the exact file that was changed
echo -e "\n=== Checking the documentation file ==="
if [ -f "docs/desktop/encrypted-file-names.md" ]; then
sed -n '83,90p' docs/desktop/encrypted-file-names.md
fiRepository: cryptomator/docs
Length of output: 1003
Fix media type mismatch: convert .mov to .mp4.
The video file is a .mov file but declared as type="video/mp4". While .mov files with H.264/AAC codecs may play in Safari and Firefox, they will fail silently in Chrome and Edge (which enforce strict MIME checking). Convert the file to .mp4 format and keep type="video/mp4" for universal browser support across all modern browsers.
Alternatively, use type="video/quicktime" for the .mov file, but this has very limited browser support and is not recommended for production documentation.
Recommended fix: convert to .mp4
- <source src="/vid/file-names.mov" type="video/mp4" />
+ <source src="/vid/file-names.mp4" type="video/mp4" />🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/desktop/encrypted-file-names.md` around lines 87 - 89, The <video> block
uses a .mov source but declares type="video/mp4"; convert the file
/vid/file-names.mov to an MP4 (H.264/AAC) and update the <source> element to
point to /vid/file-names.mp4 while keeping type="video/mp4" so browsers
(Chrome/Edge) correctly detect the MIME type; locate the <video> tag and its
<source> element in the doc and replace only the file extension/URI to .mp4.
1 participant
