-
Notifications
You must be signed in to change notification settings - Fork 70
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
base: main
Are you sure you want to change the base?
Conversation
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
I updated it to have 3 versions
|
I will expand this PR to include two more chess website : 365Chess and Chessgames |
There was a problem hiding this 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!
There was a problem hiding this 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
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