Merged
Conversation
Rework the patch changelist HTML for many unit entries: move ChangeTypeBadge next to unit name, replace UnitMeta with FactionRow and place UnitCode inside it. Replace inline "old → new" spans with a semantic structure using ChangeLabel and ChangeValue (with <s class="OldValue">, an arrow icon and <strong class="NewValue">) to standardize markup for buffs/nerfs/mixed changes and improve styling/accessibility across the page.
Fix markup and reorganize content: replace .Content div with a section, correct mismatched section/div tags and heading placements, and remove stray blank lines. Rename several IDs to remove double underscores (e.g. T2_Cybran__Shield_Lvl1 -> T2_Cybran_Shield_Lvl1). Convert many legacy .Card blocks into standardized article.PatchCard elements with structured PatchCardContent, ChangeGroup lists, faction badges and change values for multiple units (T3 UEF Shield, Nuke Silo, T3/T4 static artillery across factions, Eye of Rhianne, Game Enders, ACU Regen Aura, etc.). Overall improves semantic HTML and standardizes the patch-change presentation.
Revamp README to v2.3.1 with full PWA project overview, features, tech stack and project structure. Restructure pages/2019/3704.html: adjust semantic containers (Content -> div), standardize section ids (Land/Air/Navy/Structures/ACUs), rename several faction badge classes (seraphim -> sera), update image alts/unit names, and add many new patch cards (e.g. T2_TML, Advanced Regen Aura, Personal Shield, Chrono Dampener) with detailed change groups. Also introduce an aside Quick Navigation/ContentNav for improved in-page navigation and reorganize ACU/enhancement content into proper sections. These changes improve content completeness, readability and navigation for the patch notes page.
…descriptions - Updated sidebar structure to match 3829 template with modern ContentNav - Added 50 navigation items across 6 sections (Land, Air, Navy, Structures, ACU) - Fixed 30+ unit descriptions from placeholder text - Corrected 20 article-level buff/nerf badges based on actual stat changes - Fixed 49 individual change item labels (buff/nerf) to match stat direction - Updated 3 image paths to match article content (Shard, Czar, T3 Land HQ) - Fixed HTML structure: moved closing body tag, added missing ChangesList wrappers - Fixed unclosed strikethrough tag (line 976) - Verified all HTML tags are balanced (191 opening/closing pairs)
…ars to ContentSidemenu, and modernize intro sections
- Update sidebar structure to match 3830 template (modern NavSection/NavItem) - Fix duplicate ID: sera_aoe_enhc → aeon_shield_enhc - Fix broken image paths: T4Mega.pngb → T4Mega.png, aeon//shield.png - Add missing T1 UEF Bomber to sidebar - Correct unit-level badges: MMLs, Sparky, Static Flak, T3 Naval factories - Update unit descriptions for accuracy - Fix all factory health/regen increase labels from nerf→buff (17 instances) - Verify all 50 individual stat change labels for correctness - Update counts: air units 2→3, total changes 24→25
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (7)
📝 WalkthroughWalkthroughThe changes expand the project README with comprehensive documentation covering features, tech stack, and project structure while removing four archived HTML patch-note pages from the 2017 directory. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). 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 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit
Documentation
Chores