The two Xiangqi Diagrams used the same value for satellite=xiang. This thwarts the purpose of allowing external elements to recognize which Diagram they belong to by discriminating them by satellite.
I never knew what the satellite value was for before. I have now changed it to xiangqi.
Your Diagram was cloned from mine, which was again cloned from the very old Xiangqi Diagram. In that thime Diagrams were recognized by the betza.js script by having id="diagram". Both clones still had that. But in HTML the ids are supposed to be unique, and with getElementById() it would only find the first one. This is why my Diagram was ignored. For allowing multiple Diagrams on one page I later switched to recognizing Diagrams by class="idiagram"; with geteElementsByClassName() you can then get an array of all Diagrams on the page. For backward compatibility it still also looks for an id="diagram", and if it finds one it adds that to the array as last element. This explained why the first Diagram got the highest index in the array.
I have now added CLASS="idiagram" and removed ID="diagram".
BTW, I noticed you were using very many buttons for different styles. I don't know what your objections were to the layout of the auto-generated buttons, but it is possible to control that layout somewhat: If a set name starts with an asterisk, like set=*Oriental, the asterisk would be stripped off the name, and the button would be placed on a new line. That way you can prevent uncontrolled wrapping in a large row of buttons.
That's not the issue. I have two sets of buttons. One is for themes, and one is for boards only, as there are more boards than themes, and this allows someone to choose any combination of set and board. With automatically generated buttons, I cannot separate the two sets and label each one differently. Also, to reduce the space they take, I prefer to not give them a fixed width.
If you don't want to use auto-generated buttons, you can still divide up the Diagram definition in sets, by using numeric values for the set indicators; this suppresses creation of buttons. You can then make your own buttons select a set by having them call TwitchDiag(diagNr, setNr). The diagNr you could get in a reliable way as sats[satelliteValue]. But for the main Diagram on an article page, where no one can pre-empt you with other Diagrams, it would always be 0. (If you identify the Diagram by class, that is.)
Doing it this way, is it possible to have a button that changes the board and not the set? I previously saw you using TwitchDiag() with three arguments. What is the third argument?
Like I tried last night, I added TwitchDiag(sats['xiangqi']); to the beginning of each button, and this time it worked. I also removed Display(); from my board buttons, because tests showed it was unnecessary after just changing the board value.
I never knew what the satellite value was for before. I have now changed it to xiangqi.
I have now added
CLASS="idiagram"
and removedID="diagram"
.That's not the issue. I have two sets of buttons. One is for themes, and one is for boards only, as there are more boards than themes, and this allows someone to choose any combination of set and board. With automatically generated buttons, I cannot separate the two sets and label each one differently. Also, to reduce the space they take, I prefer to not give them a fixed width.
Doing it this way, is it possible to have a button that changes the board and not the set? I previously saw you using TwitchDiag() with three arguments. What is the third argument?
Like I tried last night, I added
TwitchDiag(sats['xiangqi']);
to the beginning of each button, and this time it worked. I also removedDisplay();
from my board buttons, because tests showed it was unnecessary after just changing the board value.