-
Notifications
You must be signed in to change notification settings - Fork 9
Add AxCrypt and Tresorit comparison articles #133
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
Conversation
Warning Rate limit exceeded@tobihagemann has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 7 minutes and 4 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (4)
WalkthroughThis PR replaces placeholder build blocks with full article content and adds public front-matter on multiple comparison pages. Files updated include axcrypt-alternative (DE/EN) and tresorit-alternative (DE/EN), which now contain complete article bodies, front-matter fields (type: article, title, description, CTA fields, ogimage) and a footer link replacing a disabled button. boxcryptor-alternative and veracrypt-alternative (DE/EN) receive appended timestamp disclaimers and horizontal rules. Changes are content and front-matter only; no application logic or non-content code was modified. Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Suggested reviewers
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
content/comparisons/axcrypt-alternative.de.md
(2 hunks)content/comparisons/axcrypt-alternative.en.md
(2 hunks)content/comparisons/boxcryptor-alternative.de.md
(1 hunks)content/comparisons/boxcryptor-alternative.en.md
(1 hunks)content/comparisons/tresorit-alternative.de.md
(2 hunks)content/comparisons/tresorit-alternative.en.md
(2 hunks)content/comparisons/veracrypt-alternative.de.md
(1 hunks)content/comparisons/veracrypt-alternative.en.md
(1 hunks)
🔇 Additional comments (1)
content/comparisons/boxcryptor-alternative.de.md (1)
464-467
: LGTM!The timestamp disclaimer is a valuable addition to comparison articles, as it sets user expectations that features and pricing information may become outdated. The styling and placement are appropriate for a footer note.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
♻️ Duplicate comments (1)
content/comparisons/tresorit-alternative.en.md (1)
198-201
: Resolved: Heading level is now consistent (Cost Structure).The previous h4 vs h3 inconsistency is fixed. Thanks for aligning the hierarchy.
🧹 Nitpick comments (8)
content/comparisons/tresorit-alternative.en.md (3)
109-129
: Improve table accessibility: add a “Feature” header and proper scopes.
- First header cell is empty; screen readers get no context.
- Add “Feature” (or “Category”), set scope="col" on headers and scope="row" on the first cell of each row.
Apply:
- <th class="font-h3 font-normal text-left p-2 lg:p-4"></th> + <th class="font-h3 font-normal text-left p-2 lg:p-4" scope="col">Feature</th> @@ - <th class="font-h3 font-normal text-left text-xs lg:text-sm leading-relaxed p-2 lg:p-4"> + <th class="font-h3 font-normal text-left text-xs lg:text-sm leading-relaxed p-2 lg:p-4" scope="col"> @@ - <th class="font-h3 font-normal text-left text-xs lg:text-sm leading-relaxed p-2 lg:p-4"> + <th class="font-h3 font-normal text-left text-xs lg:text-sm leading-relaxed p-2 lg:p-4" scope="col">And in the body rows (example shown; repeat for each first cell):
- <td class="font-p font-bold p-2 lg:p-4">Primary Purpose</td> + <th class="font-p font-bold p-2 lg:p-4" scope="row">Primary Purpose</th>Also applies to: 205-213
28-39
: Mark decorative icons as hidden from assistive tech.Font Awesome elements will be announced unless hidden. Add aria-hidden="true" to decorative icons.
Example:
- <i class="fa-solid fa-check-circle text-xl lg:text-2xl text-primary"></i> + <i class="fa-solid fa-check-circle text-xl lg:text-2xl text-primary" aria-hidden="true"></i>Apply similarly to all check/times/info icons in feature blocks and tooltips.
Also applies to: 41-88
45-60
: Make tooltips keyboard-accessible.Icons inside tooltips aren’t focusable; keyboard users can’t access the info. Consider adding tabindex="0" to the trigger, aria-describedby linking to the tooltip id, and hide tooltip content by default for SRs unless focused/hovered.
I can propose a small progressive enhancement snippet if desired.
Also applies to: 64-81
content/comparisons/tresorit-alternative.de.md (5)
5-17
: Consider adding a German alias for direct access.Add an alias like /de/tresorit-alternative to mirror EN discoverability.
title: "Alternative zu Tresorit: Cryptomator" description: "Du willst deine Daten in der Cloud verschlüsseln und suchst die beste Alternative? Wir vergleichen für dich Tresorit mit Cryptomator." +aliases: ["/de/tresorit-alternative"]
106-129
: Verbessere die Tabellen-Barrierefreiheit: Header-Text und Scopes.
- Erste Kopfzelle ist leer; füge “Kriterium” hinzu.
- Setze scope="col" auf Kopfzellen und scope="row" auf die erste Zelle jeder Zeile.
- <th class="font-h3 font-normal text-left p-2 lg:p-4"></th> + <th class="font-h3 font-normal text-left p-2 lg:p-4" scope="col">Kriterium</th> @@ - <th class="font-h3 font-normal text-left text-xs lg:text-sm leading-relaxed p-2 lg:p-4"> + <th class="font-h3 font-normal text-left text-xs lg:text-sm leading-relaxed p-2 lg:p-4" scope="col"> @@ - <th class="font-h3 font-normal text-left text-xs lg:text-sm leading-relaxed p-2 lg:p-4"> + <th class="font-h3 font-normal text-left text-xs lg:text-sm leading-relaxed p-2 lg:p-4" scope="col">And in body rows (repeat pattern):
- <td class="font-p font-bold p-2 lg:p-4">Primärer Zweck</td> + <th class="font-p font-bold p-2 lg:p-4" scope="row">Primärer Zweck</th>Also applies to: 203-211
157-159
: Deutsch konsistenter formulieren (“self-hosted”).Ersetze Anglizismus oder kennzeichne ihn typografisch; Vorschlag: „selbst gehostet“ bzw. „Self‑Hosted“.
-Desktop-Apps kostenlos (Donationware); Mobile Apps einmaliger Kauf; Hub verfügbar als self-hosted (kostenlos bis 5 Nutzer) oder managed (kostenpflichtig) +Desktop-Apps kostenlos (Donationware); Mobile Apps einmaliger Kauf; Hub verfügbar als selbst gehostete („Self‑Hosted“) Lösung (kostenlos bis 5 Nutzer) oder als Managed‑Lösung (kostenpflichtig)Und entsprechend in der Detailsektion:
-... als self-hosted Lösung (kostenlos für bis zu 5 Nutzer, ... +... als selbst gehostete („Self‑Hosted“) Lösung (kostenlos für bis zu 5 Nutzer, ...Also applies to: 198-198
27-39
: Dekorative Icons für Screenreader verbergen.Füge aria-hidden="true" zu allen dekorativen -Icons hinzu.
Example:
- <i class="fa-solid fa-check-circle text-xl lg:text-2xl text-primary"></i> + <i class="fa-solid fa-check-circle text-xl lg:text-2xl text-primary" aria-hidden="true"></i>Also applies to: 40-87
54-60
: Tooltips tastaturbedienbar machen.Mache den Auslöser fokussierbar (tabindex="0") und verknüpfe per aria-describedby mit dem Tooltip-Inhalt; Tooltip standardmäßig verstecken und bei Fokus/Hover einblenden.
Gerne liefere ich einen kleinen HTML/CSS-Snippet dafür.
Also applies to: 63-81
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
content/comparisons/tresorit-alternative.de.md
(2 hunks)content/comparisons/tresorit-alternative.en.md
(2 hunks)
🔇 Additional comments (1)
content/comparisons/tresorit-alternative.de.md (1)
196-199
: Bestätigung: Einheitliche Überschriftsebene (Kostenstruktur).Die Sektion verwendet jetzt h3 wie die übrigen Unterabschnitte. Danke.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
♻️ Duplicate comments (1)
content/comparisons/tresorit-alternative.de.md (1)
216-216
: Satzzeichen ergänzen.Der Satz endet ohne Punkt, während die übrigen Tabellenzeilen abgeschlossen sind. Bitte einen Punkt ergänzen.
- <td class="font-p p-2 lg:p-4">Du suchst eine All-in-One-Lösung mit sicherem Cloud-Speicher</td> + <td class="font-p p-2 lg:p-4">Du suchst eine All-in-One-Lösung mit sicherem Cloud-Speicher.</td>
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/comparisons/tresorit-alternative.de.md
(2 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-07-08T07:52:25.438Z
Learnt from: iammajid
PR: cryptomator/cryptomator.github.io#129
File: i18n/de.yaml:1237-1238
Timestamp: 2025-07-08T07:52:25.438Z
Learning: The Cryptomator website uses informal address ("du" form) consistently in German translations, as they are "on first name terms" with their users. Formal address ("Sie") should be avoided to maintain the friendly, approachable tone.
Applied to files:
content/comparisons/tresorit-alternative.de.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
♻️ Duplicate comments (1)
content/comparisons/tresorit-alternative.de.md (1)
212-229
: Satzenden vereinheitlichen.Bitte alle Szenario-Sätze mit einem Punkt abschließen, damit die Tabelle konsistent formatiert ist. Der fehlende Punkt bei „All-in-One-Lösung …“ war bereits angemerkt; gleich alle Zeilen korrigieren.
- <td class="font-p p-2 lg:p-4">Du benutzt bereits Cloud-Dienste und möchtest deine Dateien verschlüsseln</td> + <td class="font-p p-2 lg:p-4">Du benutzt bereits Cloud-Dienste und möchtest deine Dateien verschlüsseln.</td> @@ - <td class="font-p p-2 lg:p-4">Du suchst eine All-in-One-Lösung mit sicherem Cloud-Speicher</td> + <td class="font-p p-2 lg:p-4">Du suchst eine All-in-One-Lösung mit sicherem Cloud-Speicher.</td> @@ - <td class="font-p p-2 lg:p-4">Du benötigst eine kostenlose Lösung</td> + <td class="font-p p-2 lg:p-4">Du benötigst eine kostenlose Lösung.</td> @@ - <td class="font-p p-2 lg:p-4">Du möchtest eine sichere Datenfreigabe und Teamverwaltung</td> + <td class="font-p p-2 lg:p-4">Du möchtest eine sichere Datenfreigabe und Teamverwaltung.</td> @@ - <td class="font-p p-2 lg:p-4">Du bist ein Unternehmen und benötigst umfassende Sicherheitsfeatures</td> + <td class="font-p p-2 lg:p-4">Du bist ein Unternehmen und benötigst umfassende Sicherheitsfeatures.</td>
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
content/comparisons/tresorit-alternative.de.md
(2 hunks)content/comparisons/tresorit-alternative.en.md
(2 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-07-08T07:52:25.438Z
Learnt from: iammajid
PR: cryptomator/cryptomator.github.io#129
File: i18n/de.yaml:1237-1238
Timestamp: 2025-07-08T07:52:25.438Z
Learning: The Cryptomator website uses informal address ("du" form) consistently in German translations, as they are "on first name terms" with their users. Formal address ("Sie") should be avoided to maintain the friendly, approachable tone.
Applied to files:
content/comparisons/tresorit-alternative.de.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
content/comparisons/axcrypt-alternative.en.md (1)
28-42
: Fix AxCrypt cloud-optimization flagThe comparison metadata claims AxCrypt is “cloud storage optimized,” yet the article correctly explains that AxCrypt has no direct cloud integration and only encrypts files locally before you manually sync them. This conflicting signal will mislead readers in the feature overview generated from this front matter. Please align the icon with the narrative (e.g., switch to the red ❌ state and clarify the tooltip).
- cloudStorageOptimized: | - <i class="fa-solid fa-check-circle text-xl lg:text-2xl text-primary"></i> + cloudStorageOptimized: | + <i class="fa-solid fa-times-circle text-xl lg:text-2xl text-red-600"></i>
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
content/comparisons/axcrypt-alternative.de.md
(2 hunks)content/comparisons/axcrypt-alternative.en.md
(2 hunks)content/comparisons/boxcryptor-alternative.de.md
(1 hunks)content/comparisons/boxcryptor-alternative.en.md
(1 hunks)content/comparisons/tresorit-alternative.de.md
(2 hunks)content/comparisons/tresorit-alternative.en.md
(2 hunks)content/comparisons/veracrypt-alternative.de.md
(2 hunks)content/comparisons/veracrypt-alternative.en.md
(2 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- content/comparisons/boxcryptor-alternative.en.md
- content/comparisons/boxcryptor-alternative.de.md
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-07-08T07:52:25.438Z
Learnt from: iammajid
PR: cryptomator/cryptomator.github.io#129
File: i18n/de.yaml:1237-1238
Timestamp: 2025-07-08T07:52:25.438Z
Learning: The Cryptomator website uses informal address ("du" form) consistently in German translations, as they are "on first name terms" with their users. Formal address ("Sie") should be avoided to maintain the friendly, approachable tone.
Applied to files:
content/comparisons/tresorit-alternative.de.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
content/comparisons/axcrypt-alternative.de.md (1)
6-6
: Bitte „du“ konsequent kleinschreiben.Hier tauchen noch großgeschriebene Varianten („Du“/„Du möchtest“) auf, während der Rest des Artikels das gewünschte klein geschriebene „du“ nutzt. Bitte die genannten Stellen ebenfalls auf „du/dein“ anpassen, damit die Ansprache durchgehend konsistent bleibt (z. B. Zeile 6 „du willst …“, Zeile 202 „du kannst …“, Tabellenzellen ab Zeile 227).
Based on learningsAlso applies to: 202-202, 227-239
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/comparisons/axcrypt-alternative.de.md
(2 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-07-08T07:52:25.438Z
Learnt from: iammajid
PR: cryptomator/cryptomator.github.io#129
File: i18n/de.yaml:1237-1238
Timestamp: 2025-07-08T07:52:25.438Z
Learning: The Cryptomator website uses informal address ("du" form) consistently in German translations, as they are "on first name terms" with their users. Formal address ("Sie") should be avoided to maintain the friendly, approachable tone.
Applied to files:
content/comparisons/axcrypt-alternative.de.md
Add two new comparison articles for AxCrypt and Tresorit (English and German versions).