Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding link support for Chess.com , 365Chess and Chessgames #5226

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

Hesketh2
Copy link
Collaborator

@Hesketh2 Hesketh2 commented Dec 20, 2024

Summary

In future for chess wiki, this is also intended to be used within the match summary popup as well (like HLTV/VLR) as well as Infobox Icon too

Update: Includes 365Chess and Chessgames, all three will be used for Chess wiki, actually it's already in use for some of pre-documentation help

|chesscom=
|365chess=
|chessgames=

How did you test this change?

How do I test link icons to begin with? I only recall the files that I need to edit

@Hesketh2 Hesketh2 marked this pull request as draft December 20, 2024 09:11
@Hesketh2 Hesketh2 requested review from hjpalpha and Rathoz and removed request for hjpalpha December 20, 2024 09:15
@Hesketh2 Hesketh2 self-assigned this Dec 20, 2024
@hjpalpha
Copy link
Collaborator

hjpalpha commented Dec 20, 2024

(I cannot edit CSS, have no access)

is to be done via PR in this repo so yes you can^^

for when inputting in Infobox gives you a different preset url, then in a match gives you another different one
@Hesketh2 Hesketh2 marked this pull request as ready for review December 21, 2024 16:26
@Hesketh2
Copy link
Collaborator Author

I updated it to have 3 versions

  • Standard: chess.com/ this is for to be able to add any chess links mostly this is for Infobox League
  • Player: chess.com/member/ this is for personal player page
  • Match: chess.com/games/view/ matchid

@Hesketh2 Hesketh2 changed the title Adding link support for Chess.com Adding link support for Chess.com , Chess365 and Chessgames Dec 24, 2024
@Hesketh2
Copy link
Collaborator Author

Hesketh2 commented Dec 24, 2024

I will expand this PR to include two more chess website : 365Chess and Chessgames
both already have saw usage in the pre-documented pages

Copy link
Collaborator

@hjpalpha hjpalpha left a comment

Choose a reason for hiding this comment

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

keep the order alphabetical!

@Hesketh2 Hesketh2 changed the title Adding link support for Chess.com , Chess365 and Chessgames Adding link support for Chess.com , 365Chess and Chessgames Dec 24, 2024
Copy link
Collaborator

@hjpalpha hjpalpha left a comment

Choose a reason for hiding this comment

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

missing the css classes for the 2 new ones

files in commons
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants