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

FR: Explicilty set rikaikun to horizontal writing mode to avoid inheriting veritcal text styles from host page #613

Closed
melink14 opened this issue Jul 18, 2021 · 3 comments · Fixed by #624

Comments

@melink14
Copy link
Owner

See discussion here: #363 (reply in thread)

Can repro at online ebook reader: https://ttu-ebook.web.app/

Possible solution:
writing-mode: horizontal-tb; on #rikaichan-window

This is an easier pre-fix before migrating to shadow dom for the popup: #221

@melink14
Copy link
Owner Author

(Maybe change positioning to be right of vertical text as well.

@melink14
Copy link
Owner Author

Just realized I need to do this in each theme's CSS so I might try to combine them into one first. (#222)

@melink14 melink14 self-assigned this Jul 20, 2021
melink14 added a commit that referenced this issue Jul 20, 2021
Avoids inheriting veritcal text styles from host page when they're overly broad.

Long term, shadow dom should help, and we should also make a popup position that works well for vertical text.

Fixes #613
melink14 added a commit that referenced this issue Jul 20, 2021
Avoids inheriting veritcal text styles from host page when they're overly broad.

Long term, shadow dom should help, and we should also make a popup position that works well for vertical text.

Fixes #613
melink14 pushed a commit that referenced this issue Jul 20, 2021
## [2.1.0](v2.0.1...v2.1.0) (2021-07-20)

### Features

* **css:** Move popup themes into a single file using CSS variables ([#623](#623)) ([75241b5](75241b5)), closes [#222](#222)
* **popup:** Explicilty sets rikaikun to horizontal writing mode ([#624](#624)) ([5fae815](5fae815)), closes [#613](#613)
* Set `offline_enabled` to true in extension manifest ([#612](#612)) ([202aeac](202aeac)), closes [#429](#429)

### Bug Fixes

* **dict:** Update dictionaries to latest versions. ([#620](#620)) ([9a67cb1](9a67cb1))
@melink14
Copy link
Owner Author

🎉 This issue has been resolved in version 2.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant