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

Added show eval toggle on broadcast multiboard page #14540

Merged
merged 16 commits into from Jan 24, 2024

Conversation

Carbrex
Copy link
Contributor

@Carbrex Carbrex commented Jan 23, 2024

Added show eval toggle on broadcast multiboard page. This currently only shows eval if it is currently available. Should this toggle request eval if it is not available? Closes #14497
Also can you help me with ctrl.trans.noarg('Show Eval'), the playing above gets translated to other languages but this does not?

@Carbrex Carbrex marked this pull request as draft January 23, 2024 13:16
@Carbrex
Copy link
Contributor Author

Carbrex commented Jan 23, 2024

image
image

@Carbrex
Copy link
Contributor Author

Carbrex commented Jan 23, 2024

I tried adding the translation to study.xml and ran docker compose run --rm ui /lila/bin/trans-dump and restarted the server but it doesnt show the translation. I am pushing these changes so that you can check whats wrong.
image

@M-DinhHoangViet
Copy link
Contributor

I tried adding the translation to study.xml and ran docker compose run --rm ui /lila/bin/trans-dump and restarted the server but it doesnt show the translation. I am pushing these changes so that you can check whats wrong. image

I think you missed in https://github.com/lichess-org/lila/blob/master/app/views/study/jsI18n.scala

@Carbrex
Copy link
Contributor Author

Carbrex commented Jan 23, 2024

I think you missed in https://github.com/lichess-org/lila/blob/master/app/views/study/jsI18n.scala

Thanks! that worked.
image
But it still doesn't get translated?

@Carbrex
Copy link
Contributor Author

Carbrex commented Jan 23, 2024

Ohh, I think the translation will be added using crowdin.

@Carbrex Carbrex marked this pull request as ready for review January 23, 2024 15:36
@Carbrex
Copy link
Contributor Author

Carbrex commented Jan 24, 2024

Shall I also fix #14320 in this pr?

@ornicar ornicar merged commit da24703 into lichess-org:master Jan 24, 2024
5 checks passed
ornicar added a commit to allanjoseph98/lila that referenced this pull request Jan 26, 2024
* master: (40 commits)
  multiboard eval tick: remove unncesseray inlined style/CSS attribute
  Added visual indicator for showing advantage in mini-board broadcast
  monitor firebase data msgs separately
  fix push notification recipient
  add source field to game json export
  only export reports created by the player
  fix typos
  scalachess 15.7.6
  Personal export: clarify that reports concerning you only include messages you posted
  Acpl chart: live update and spinner for studies
  game analysis CSP - closes lichess-org#14536
  scala tweaks
  allow partial pull/push of broadcast round PGN
  show multiboard evals even if local eval is disabled - after lichess-org#14540
  New Crowdin updates (lichess-org#14517)
  request cloud evals when toggling the option
  redesign multiboard options
  set multiboard.showEval to true by default
  tweak evaluation bars wording
  remember multiboard.showEval value using local storage
  ...
ornicar added a commit to schlawg/lila that referenced this pull request Jan 28, 2024
* master: (31 commits)
  game analysis CSP - closes lichess-org#14536
  scala tweaks
  allow partial pull/push of broadcast round PGN
  show multiboard evals even if local eval is disabled - after lichess-org#14540
  New Crowdin updates (lichess-org#14517)
  request cloud evals when toggling the option
  redesign multiboard options
  set multiboard.showEval to true by default
  tweak evaluation bars wording
  remember multiboard.showEval value using local storage
  use a toggle for multiboard showEval
  no need to reload multiboards, just redraw
  simplify puzzle dashboard css
  mobile firefox doesnt support COEP credentialless
  Removed log statement
  remember when bulk clock was started
  User.isVerifiedOrChallengeAdmin
  tweak bulk limits
  new endpoint to show a bulk
  export more games by ID
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broadcast: add eval toggle on multiboard page
3 participants