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

fix: ruby tags focus and navigation #2655

Conversation

viktar-dzmitryieu-tao
Copy link
Contributor

@viktar-dzmitryieu-tao viktar-dzmitryieu-tao commented Jan 9, 2025

Ticket: https://oat-sa.atlassian.net/browse/INF-279

What's Changed

  • Updated styling for the ruby tags appeared specifically for the Chrome (v. >=128)

How to test

  • Add any interaction in test item
  • Define prompt to it
  • Insert Furigana (Ruby) to any selection
  • See the <rt> content is aligned to the beginning of the element

Deployed to https://test-viktar-authoring.v3.playground.taocloud.org/tao/Main/login

Copy link

github-actions bot commented Jan 9, 2025

Front-end summary Node 18

💯 Total ✅ Passed ⏭️ Skipped ❌ Failed
287 287 0 0

Copy link

github-actions bot commented Jan 9, 2025

Version

Target Version 30.25.5
Last version 30.25.4

There are 0 BREAKING CHANGE, 0 feature, 2 fixes

@viktar-dzmitryieu-tao viktar-dzmitryieu-tao requested review from a team, tikhanovichA, KirylHatalski and Karol-Stelmaczonek and removed request for a team January 9, 2025 14:30
@olga-kulish olga-kulish self-requested a review January 13, 2025 08:28

ruby rt {
// Chrome >=128 and Firefox support
ruby-align: start;
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we should do it. I understand that while you are typing ruby, cursor position is weird, but:
Screenshot_62

Or at least do not apply ruby-align in test-runner (see Item Preview in the playground). These customers still use CurrentGen test-runner.

Also, if you decide to keep this, could you please add this style for A-block too, not only for Prompt?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll close this PR as it was not required and can lead to misunderstanding

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.

3 participants