Skip to content

Organize CSS and align GitDev skin colors - #9

Merged
dieghernan merged 2 commits into
mainfrom
refactor/css-sections
Oct 20, 2026
Merged

dieghernan merged 2 commits into
mainfrom
refactor/css-sections

Conversation

@dieghernan

@dieghernan dieghernan commented Oct 20, 2026 •

Copy link
Copy Markdown
Owner

Organize gitdevr's existing stylesheet into nine indexed component sections and consolidate equivalent callout, Reactable filter/search, and button parameter rules. Keep gitdevr's theme tokens, component behavior, footer layout, and syntax highlighting.

The second commit aligns the remaining palette differences with Chulapa's gitdev and gitdev-dark skins:

  • Set the light footer background to #0d1117.
  • Use the skin's dark muted text #8b949e for navbar/dropdown hover states, retaining this readable shade on the light site's dark navbar too.
  • Correct the standalone dark brand to resolve background #24292f and primary #373e47, avoiding the ambiguous dark alias.
  • Keep the standalone light brand and embedded pkgdown brand synchronized.

Only existing files are modified. No scripts, screenshots, dependencies, or new source files are added. The first commit is a rendering-preserving reorganization; the second contains the intentional skin-color corrections above. The footer's layout implementation remains unchanged.

Validation:

  • 110 package checks passed; devtools::lint(), jarl check ., formatting and whitespace checks passed.
  • For the first commit, all 250 selector/context entries retain their declarations and priorities; full-page screenshots and computed hover/focus styles are identical before/after in both themes at 360, 768 and 1280px.
  • After palette alignment, Chromium confirms white body / #22272e navbar / #0d1117 footer in light mode, and #24292f body/navbar/footer in dark mode. Navbar hover resolves to #8b949e in both modes.
  • Brand tests verify the resolved standalone dark background/primary and synchronization of the two light-brand definitions.

References: https://github.com/dieghernan/chulapa/blob/main/_sass/skins/gitdev.scss and https://github.com/dieghernan/chulapa/blob/main/_sass/skins/gitdev-dark.scss.

@dieghernan dieghernan added ai-assisted 🤖 changes include AI-assisted development or review upkeep maintenance, infrastructure, and similar labels Oct 20, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-20T21:38:15.151699Z 06f4299 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@codecov

codecov Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (bf27cc3) to head (06f4299).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines            7         7           
=========================================
  Hits             7         7           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@coveralls

coveralls commented Oct 20, 2026 •

Copy link
Copy Markdown

Coverage Report for CI Build 39501602793

Coverage remained the same at 100.0%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 7
Covered Lines: 7
Line Coverage: 100.0%
Coverage Strength: 4.0 hits per line

💛 - Coveralls

@dieghernan dieghernan changed the title Organize CSS into sections and consolidate equivalent rules Organize CSS and align GitDev skin colors Oct 20, 2026
@dieghernan
dieghernan merged commit 52cc6f3 into main Oct 20, 2026
10 checks passed
@dieghernan
dieghernan deleted the refactor/css-sections branch October 20, 2026 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-assisted 🤖 changes include AI-assisted development or review upkeep maintenance, infrastructure, and similar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants