Skip to content

Conversation

@FunToHard
Copy link

  • Replace config.md with config.html for better web presentation
  • Add comprehensive configuration documentation with examples
  • Include professional styling with responsive design
  • Add interactive navigation and smooth scrolling
  • Provide detailed explanations for all TranslucentTB settings
  • Include troubleshooting section and best practices

- Replace config.md with config.html for better web presentation
- Add comprehensive configuration documentation with examples
- Include professional styling with responsive design
- Add interactive navigation and smooth scrolling
- Provide detailed explanations for all TranslucentTB settings
- Include troubleshooting section and best practices
Copilot AI review requested due to automatic review settings September 8, 2025 16:08
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR converts the TranslucentTB configuration documentation from a placeholder Markdown file to a comprehensive HTML documentation page with professional styling and interactive features.

  • Replaces the minimal config.md placeholder with a complete HTML documentation
  • Adds comprehensive configuration documentation covering all TranslucentTB settings
  • Includes interactive navigation, responsive design, and smooth scrolling functionality

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
config.md Removes placeholder Markdown file with TODO note
config.html Adds comprehensive HTML documentation with styling, navigation, and detailed configuration examples

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

config.html Outdated
<div class="config-card">
<h4>Fully Transparent Desktop</h4>
<pre><code>{
"$schema": "https://TranslucentTB.github.io/settings.schema.json",
Copy link

Copilot AI Sep 8, 2025

Choose a reason for hiding this comment

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

[nitpick] Inconsistent casing in the schema URL. The domain uses 'TranslucentTB' with mixed case, which should likely be all lowercase 'translucenttb' to follow standard GitHub Pages conventions.

Copilot uses AI. Check for mistakes.
config.html Outdated
<div class="config-card">
<h4>Application-Specific Rules</h4>
<pre><code>{
"$schema": "https://TranslucentTB.github.io/settings.schema.json",
Copy link

Copilot AI Sep 8, 2025

Choose a reason for hiding this comment

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

[nitpick] Inconsistent casing in the schema URL. The domain uses 'TranslucentTB' with mixed case, which should likely be all lowercase 'translucenttb' to follow standard GitHub Pages conventions.

Copilot uses AI. Check for mistakes.
config.html Outdated

<footer>
<div class="container">
<p>&copy; 2025 TranslucentTB Project. Configuration documentation for better Windows taskbar customization.</p>
Copy link

Copilot AI Sep 8, 2025

Choose a reason for hiding this comment

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

[nitpick] The copyright year is 2025, but the current date context suggests this may be incorrect. Consider using the actual current year or a dynamic copyright year.

Copilot uses AI. Check for mistakes.
@sylveon
Copy link
Member

sylveon commented Sep 8, 2025

I would prefer if this stays markdown, to make it easier to maintain and fit in with the Jekyll theme we use to keep a consistent style.

- Remove config.html and restore config.md.
- Follow project consistency guidelines
@FunToHard FunToHard force-pushed the feature/config-html-documentation branch from 0c5a05e to 678c9da Compare September 9, 2025 03:39
@FunToHard
Copy link
Author

✅ Revert to Markdown format for consistency

  • Remove config.html and restore config.md. ✅
  • Follow project consistency guidelines. ✅

All done as you mentioned.

config.md Outdated

### 6. Task View Opened Appearance

Applied when Task View (Alt+Tab) is open.
Copy link
Member

Choose a reason for hiding this comment

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

Task View is Win+Tab

config.md Outdated
- Example: `"#00000000"` (fully transparent black)

#### `show_peek` (boolean)
Enable/disable desktop peek functionality:
Copy link
Member

Choose a reason for hiding this comment

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

Hiding peek does not disable the functionality. This is classic taskbar only

### Common Issues

- **Configuration not loading**: Check JSON syntax and schema validation
- **Rules not working**: Verify process names and window classes using tools like Spy++
Copy link
Member

Choose a reason for hiding this comment

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

AHK's window spy is more accessible to users

config.md Outdated
1. **Check the logs**: Set `verbosity` to `"debug"` to see detailed information
2. **Validate JSON**: Use online JSON validators or VS Code with the schema
3. **Test incrementally**: Add one setting at a time to isolate issues
4. **Use process monitor**: Tools like Process Monitor can help identify file access issues
Copy link
Member

Choose a reason for hiding this comment

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

This will not help the average user

@FunToHard
Copy link
Author

give me two days.

@FunToHard
Copy link
Author

All done as requested

Exceptions

  • "AHK’s window spy is more accessible to users"
    I have not used AHK, so I cannot document this.

  • "This will not help the average user"
    This section is already marked as Advanced. For regular users, Common Issues is documented separately.

@FunToHard FunToHard requested a review from sylveon September 12, 2025 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants