Skip to content

Conversation

@Jaehyuk-Lee
Copy link

@Jaehyuk-Lee Jaehyuk-Lee commented Jan 14, 2026

README.md

  • add Traditional Chinese support
  • update .NET Framework download links
  • remove DirectX 11 requirement
    • because FFXIV now supports DirectX 11 version only
  • include/exclude language parameter for better accessibility
    • include: cactbot coverage link for each languages
    • exclude: Microsoft automatically redirects users to the localized version of the site

CactbotCustomization.md

@github-actions github-actions bot added docs /docs, /screenshots, *.md 💬ko 💬cn needs-review Awaiting review labels Jan 14, 2026
@Jaehyuk-Lee Jaehyuk-Lee changed the title docs: update README docs: update README and other docs Jan 14, 2026
Comment on lines 224 to 222
[此页面](https://overlayplugin.github.io/cactbot/util/coverage/coverage.html) 列出了当前 Cactbot 中支持的副本。我们会持续添加更多支持 (随时欢迎贡献代码!) 尽管许多旧副本仍未支持。
[此页面](https://overlayplugin.github.io/cactbot/util/coverage/coverage.html?lang=cn) 列出了当前 Cactbot 中支持的副本。我们会持续添加更多支持 (随时欢迎贡献代码!) 尽管许多旧副本仍未支持。
Copy link
Collaborator

Choose a reason for hiding this comment

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

coverage doesn't appear to auto-detect language currently (which is probably a bug, given that it has the browserLanguagesToLang function but isn't using it).

Should either fix the coverage auto-detect issue (in a separate followup PR) or keep the lang parameter in the link here.

Copy link
Author

@Jaehyuk-Lee Jaehyuk-Lee Jan 15, 2026

Choose a reason for hiding this comment

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

I think quisquous just didn't notice that coverage.ts also needs the language detection.

3 years ago: browserLanguagesToLang made separated from raidemulator.ts. quisquous#4584
5 years ago: coverage.ts was added. quisquous#2098

I'll make another PR for auto-detecting the browser language.

I think the browserLanguagesToLang need a small refactor, too.

Copy link
Author

Choose a reason for hiding this comment

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

This issue will be resolved via #946.

@github-actions github-actions bot removed the needs-review Awaiting review label Jan 14, 2026
@github-actions github-actions bot added the needs-review Awaiting review label Jan 15, 2026
@xiashtra xiashtra requested a review from ShadyWhite January 16, 2026 01:01
@github-actions github-actions bot removed the needs-review Awaiting review label Jan 16, 2026
@xiashtra xiashtra merged commit 63a74bb into OverlayPlugin:main Jan 16, 2026
17 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 16, 2026
### README.md

- add Traditional Chinese support
- update .NET Framework download links
- remove DirectX 11 requirement
  - because FFXIV now supports DirectX 11 version only
- include/exclude language parameter for better accessibility
  - include: cactbot coverage link for each languages
- exclude: Microsoft automatically redirects users to the localized
version of the site

### CactbotCustomization.md

- fix links
aff3acd 63a74bb
@Jaehyuk-Lee Jaehyuk-Lee deleted the readme branch January 20, 2026 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💬cn docs /docs, /screenshots, *.md 💬ko

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants