Skip to content

Conversation

2bndy5
Copy link
Contributor

@2bndy5 2bndy5 commented Sep 29, 2025

resolves #3

  • Better color choice for anti-aliasing.
  • Better size and positioning for mobile and desktop.
  • Update mkdocs.yml config to silence warnings about unused documents (blog/**)
  • Update repo_name in mkdocs.yml to display GitHub org name instead of the generic default ("Github")
  • Disable markdownlint warnings as needed.
  • adjusted Ci to build docs during PR (dry-run) and deploy docs from main branch (using official GH actions only)

* Better color choice for anti-aliasing.
* Better size and positioning for mobile and desktop.
* Update mkdocs.yml config to silence warnings about unused documents (`blog/**`)
* Update `repo_name` in mkdocs.yml to display GitHub org name instead of the generic default ("Github")
* Disable markdownlint warnings as needed.
@2bndy5 2bndy5 added the enhancement New feature or request label Sep 29, 2025
Copy link

netlify bot commented Sep 29, 2025

Deploy Preview for cpp-linter-github-io ready!

Name Link
🔨 Latest commit 5fd1fb6
🔍 Latest deploy log https://app.netlify.com/projects/cpp-linter-github-io/deploys/68da36ec6e09530008b53f5c
😎 Deploy Preview https://deploy-preview-4--cpp-linter-github-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@shenxianpeng
Copy link
Contributor

image image

When I move the mouse to the button, the background color changes. I like the yellow color used on the first button—it closely matches our logo. Should we align the background color across all buttons?

args: [--style=Google] # Other coding style: LLVM, GNU, Chromium, Microsoft, Mozilla, WebKit.
- id: clang-tidy
args: [--checks='boost-*,bugprone-*,performance-*,readability-*,portability-*,modernize-*,clang-analyzer-*,cppcoreguidelines-*']
args:
Copy link
Contributor

@shenxianpeng shenxianpeng Sep 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The yaml format seems incorrect if users just copy and use it. Maybe id and args need to be aligned

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to YAML specs

list-name: [item]
# is equivalent to
list-name:
  - item

I hadn't noticed the misaligned indent. That should be uniformed.

@shenxianpeng
Copy link
Contributor

shenxianpeng commented Sep 29, 2025

About the title, right now this PR changes it to C/C++ Linting - cpp-linter

How about

  1. cpp-linter: Linting Solutions for C/C++
  2. cpp-linter: Linting Solutions for C/C++ Projects
  3. cpp-linter: Code Quality Solutions for C/C++

@2bndy5
Copy link
Contributor Author

2bndy5 commented Sep 29, 2025

The hover buttons color can be uniformly yellow.

I don't really care which title you want. Typically a title should be brief.

I just changed the title because the # was moved out of the index.md into home.html template.

@shenxianpeng
Copy link
Contributor

Ok let's merge and I'll see if any other changes are needed to the button background color and title

@shenxianpeng shenxianpeng merged commit 9b66e82 into main Sep 29, 2025
6 checks passed
@shenxianpeng shenxianpeng deleted the hero-style branch September 29, 2025 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

web display issue on mobile

2 participants