-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Coordinate Trainer: option for coordinates on every square #15221
Conversation
I realize now I mistakenly tried to update a node modules file |
Glad to see someone take this on. I'll just link it to the issue with: fixes #15212 |
* master: (29 commits) move app rate limiters to web Revert necessary import for pipe ensure all rate limiters are configured move ctrl limiters to web - WIP scala tweaks report donation stats every 24h New Crowdin updates (lichess-org#15226) chessground redrawAll scss tweaks put is3d in board change event prettier Don't refetch same css, cached or no brightness/opacity on last move & check squares, but no hue rotate disable social links on kid profiles Update uap-scala to 0.17.0 Update play-json to 3.0.3 Update play-ahc-ws-standalone, ... to 2.2.7 Hide kid teams on profile comm restrictions no longer shield against comm reports remove duplicated css property ...
…into opt-coords-every-sq * 'opt-coords-every-sq' of https://github.com/BSmick6/lila:
your move |
Yep it is a different issue intentionally because both need to be implemented separately. How are your CSS skills? Based on your video, it seems like there is some styling issue to be debugged. Could be an issue with your implementation or with chessground. That's the thing to look into next I'd say. Still your move! :D |
I'm familiar with CSS so I can investigate whether it's an issue in Lila at least. |
i also ran into the same issue when trying to do this. looks like the translateXs or something are not getting applied properly. I tried on Gitpod |
obviously, it isn't a long term solution but I hacked together some CSS code to at least position the coordinates correctly. this is what it looks like: |
Working on it right now but I'm trying to get the css right before I commit anything to chessground. It's been a struggle so I might just open a draft PR and call it a day soon |
Yes it seems that we manually add chessground CSS to lila, perhaps because we want to be very deliberate. It looks like chessground works fine so you can close your PR over there. |
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.
I tried to improve/clean up the CSS code by removing unnecessary selectors and nesting others. Any and all feedback is appreciated. I plan on updating respective CSS code in the chessground repo (PR) once we've settled on the changes here.
@@ -212,3 +212,53 @@ html.transp body:not(.simple-board) cg-board { | |||
//filter: hue-rotate(calc(var(---board-hue) * 3.6deg)); | |||
} | |||
} | |||
|
|||
coords.squares { |
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.
I don't love the .squares
class name. Squares are referenced very often in chess. For example, chessground already uses a <square>
element. I anticipate conflicts one day, but for now, we can just specify the <coords>
element to mitigate that risk.
Thanks for these contributions @BSmick6! ornicar will eventually have a look and may adjust things. He is usually juggling about 13 tasks simultaneously, so there might be a small wait. |
the title looked like a button
great stuff, thanks! Next: as a global website setting. |
white normal:
black normal:
white all squares:
black all squares: